[PATCH v4 2/2] ASoC: cs35l41: Add bindings for CS35L41

Mark Brown broonie at kernel.org
Fri Jul 30 17:33:04 CEST 2021


On Fri, Jul 30, 2021 at 10:28:56AM -0500, David Rhodes wrote:
> On 7/27/21 10:31 AM, Mark Brown wrote:
> > > +  cirrus,right-channel-amp:

> > Does this need to be in the DT or could it be a runtime control (eg, for
> > a system that supports being rotated to adjust which speaker things go
> > out of).

> This can also be changed at runtime using set_channel_map.

Then it shouldn't be in the DT.

> > This is usually a runtime control too.  This applies to a bunch of the
> > properties TBH.

> I agree, many of these properties can be changed to controls.
> I would keep these in DT:
> boost-peak-milliamp
> boost-ind-nanohenry
> boost-cap-microfarad
> asp-sdout-hiz
> classh-bst-max-limit
> gpio-*

Those look reasonable.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20210730/eea381cd/attachment.sig>


More information about the Alsa-devel mailing list