Re: [PATCH] ASoC: da7210: Add HeadPhone Playback Volume control

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel


[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux