Subject: Re: [PATCH] ASoC: rt1308: Add RT1308 amplifier driver
On Wed, May 08, 2019 at 01:17:54PM +0000, Derek wrote:
The maximum volume can be limited with snd_soc_limit_volume() but if the limit is for the speaker then that's machine dependent anyway.
The maximum volume setting is temporarily not machine dependent. It avoids harming if any machine doesn't set the limit volume.
I'm confused about what this limit is - is it just some arbatrary limit imposed in case some board has problems or is it an actual limit that comes from the chip? If it's a "just in case" limit then the board should be the one doing the limiting, yes people can break things but it's difficult to get decision like that done sensibly in the CODEC driver in a way that works for everything and the general expectation is that this is all for use by system integrators rather than end users.
It is an actual limit that comes from the chip.
------Please consider the environment before printing this e-mail.