[alsa-devel] [PATCH] ASoC: Add max98371 codec driver

[anish kumar]

On Tue, Apr 19, 2016 at 2:46 AM, Mark Brown <broonie at kernel.org> wrote:
> On Mon, Apr 18, 2016 at 01:20:35PM -0700, anish kumar wrote:
>> On Mon, Apr 18, 2016 at 9:47 AM, Mark Brown <broonie at kernel.org> wrote:
>
>> > As I said last time I would expect this to be a TLV control.  As far as
>> > I can tell this is basically functioning as a maximum volume.  Otherwise
>> > this looks good.
>
>> I responded to your comment earlier but there was no reply so I thought
>> it is accepted this way. Excuse my understanding. It is not actually a volume
>> but described as such.
>
> If it's measured in dB I'd still expect a TLV control.
>
>> It is "Speaker No-Load Output Voltage Maximum". It is
>> described here on page 50.
>> https://datasheets.maximintegrated.com/en/ds/MAX98371.pdf
>
> Hrm, that looks like the driver may be able to set it automatically
> using the regulator API and just have a switch for the guaranteed.

Should i pass it via device tree then? I didn't get your statement about
"switch".