On Wed, May 11, 2022 at 03:49:12PM +0200, Takashi Iwai wrote:
On Mon, 09 May 2022 23:46:47 +0200, Vitaly Rodionov wrote:
From: Stefan Binding sbinding@opensource.cirrus.com
The cs35l41 part contains a DSP which is able to run firmware. The cs_dsp library can be used to control the DSP. These controls can be exposed to userspace using ALSA controls. This library adds apis to be able to interface between cs_dsp and hda drivers and expose the relevant controls as ALSA controls.
Hmm, quite lots of things aren't explained here.
First off, as far as I see, the control elements that are implemented in this patch are pretty unique, they don't follow the standard way. Admittedly, ASoC core (ab)uses the TLV read/write for the arbitrary data bytes, and this seems following that instead. If so, it needs more clear explanation in the comments add/or commit logs.
Hm... yes I think we should be very clear on if the TLV support is needed here. This should only be needed if the firmware has controls (and we need to use those controls) over 512 bytes, and most firmwares don't. Can we check this and remove the TLV support if we can? We probably don't want to add more users of that stuff if we can help it.
Thanks, Charles