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

[Mark Brown]

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.