[alsa-devel] [PATCH] ASoC: da7210: Add HeadPhone Playback Volume control

Mark Brown broonie at opensource.wolfsonmicro.com
Tue Jul 20 10:37:55 CEST 2010


On Tue, Jul 20, 2010 at 11:24:17AM +0900, Kuninori Morimoto wrote:

> +	/*
> +	 * HeadPhone PlayBack Volume 40 is given here
> +	 * because DA7210 doesn't have default volume.
> +	 */
> +	snd_soc_update_bits(codec, DA7210_HP_L_VOL, 0x3F, 40);
> +	snd_soc_update_bits(codec, DA7210_HP_R_VOL, 0x3F, 40);
> +

I find this very surprising - the chip actually comes up with an
indeterminate value at start?  In any case, if this is required I'd
suggest picking mute as the default value since then there's no chance
that the signal will be excessively loud by default on some systems.


More information about the Alsa-devel mailing list