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

[Ola Lilja <ola.o.lilja@stericsson.com>]

On 05/08/2012 08:27 PM, Mark Brown wrote:

>> +static struct snd_kcontrol_new ab8500_ctrls[] = {
>> +	/* Charge pump */
>> +	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.