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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux