[alsa-devel] [PATCH 10/11] ASoC: codecs: Add AB8500 codec-driver

[Mark Brown]

On Wed, May 23, 2012 at 09:04:49AM +0200, Ola Lilja wrote:
> On 05/08/2012 08:27 PM, Mark Brown wrote:

> >> +	SOC_ENUM("Charge Pump High Threshold For Low Voltage",
> >> +		soc_enum_envdeththre),
> >> +	SOC_ENUM("Charge Pump Low Threshold For Low Voltage",
> >> +		soc_enum_envdetlthre),
> >> +	SOC_ENUM("Charge Pump Envelope Detection", soc_enum_envdetcpen),
> >> +	SOC_ENUM("Charge Pump Envelope Detection Decay Time",
> >> +		soc_enum_envdettime),

> > Again, why is this all exposed to userspace?

> These one we need as ALSA-controls. They are parameters that we can tune during
> runtime and are not hard-wired in any way.

This seems incredibly surprising - are you sure the configuration isn't
simply a case of enabling and disabling the charge pump voltage scaling?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
Url : http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20120523/87238696/attachment.sig