[alsa-devel] [PATCH] ASoC: alc5632: rename volume/switch contols for master and speaker volumes.
From: Andrey Danin danindrey@mail.ru
Signed-off-by: Andrey Danin danindrey@mail.ru Signed-off-by: Leon Romanovsky leon@leon.nu --- sound/soc/codecs/alc5632.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/sound/soc/codecs/alc5632.c b/sound/soc/codecs/alc5632.c index 8a3bf71..71c8a8d 100644 --- a/sound/soc/codecs/alc5632.c +++ b/sound/soc/codecs/alc5632.c @@ -144,10 +144,10 @@ static const DECLARE_TLV_DB_SCALE(vdac_tlv, -3525, 075, 0); static const struct snd_kcontrol_new alc5632_vol_snd_controls[] = { /* left starts at bit 8, right at bit 0 */ /* 31 steps (5 bit), -46.5db scale */ - SOC_DOUBLE_TLV("Line Playback Volume", + SOC_DOUBLE_TLV("Speaker Playback Volume", ALC5632_SPK_OUT_VOL, 8, 0, 31, 1, hp_tlv), /* bit 15 mutes left, bit 7 right */ - SOC_DOUBLE("Line Playback Switch", + SOC_DOUBLE("Speaker Playback Switch", ALC5632_SPK_OUT_VOL, 15, 7, 1, 1), SOC_DOUBLE_TLV("Headphone Playback Volume", ALC5632_HP_OUT_VOL, 8, 0, 31, 1, hp_tlv), @@ -166,9 +166,9 @@ static const struct snd_kcontrol_new alc5632_snd_controls[] = { ALC5632_PHONE_IN_VOL, 8, 31, 1, vol_tlv), SOC_DOUBLE_TLV("LineIn Capture Volume", ALC5632_LINE_IN_VOL, 8, 0, 31, 1, vol_tlv), - SOC_DOUBLE_TLV("Stereo DAC Playback Volume", + SOC_DOUBLE_TLV("Master Playback Volume", ALC5632_STEREO_DAC_IN_VOL, 8, 0, 63, 1, vdac_tlv), - SOC_DOUBLE("Stereo DAC Playback Switch", + SOC_DOUBLE("Master Playback Switch", ALC5632_STEREO_DAC_IN_VOL, 15, 7, 1, 1), SOC_SINGLE_TLV("Mic1 Capture Volume", ALC5632_MIC_VOL, 8, 31, 1, vol_tlv),
On Sun, Nov 13, 2011 at 09:53:13PM +0200, Leon Romanovsky wrote:
- SOC_DOUBLE_TLV("Line Playback Volume",
- SOC_DOUBLE_TLV("Speaker Playback Volume", ALC5632_SPK_OUT_VOL, 8, 0, 31, 1, hp_tlv),
This looks really suspicious, why are we renaming the line output as a speaker output? This might be appropriate on your board but if the drivers aren't part of the CODEC it might not be appropriate on another board.
ALC5632_LINE_IN_VOL, 8, 0, 31, 1, vol_tlv),
- SOC_DOUBLE_TLV("Stereo DAC Playback Volume",
- SOC_DOUBLE_TLV("Master Playback Volume", ALC5632_STEREO_DAC_IN_VOL, 8, 0, 63, 1, vdac_tlv),
- SOC_DOUBLE("Stereo DAC Playback Switch",
- SOC_DOUBLE("Master Playback Switch",
This looks fine.
On Mon, Nov 14, 2011 at 23:39, Mark Brown broonie@opensource.wolfsonmicro.com wrote:
On Sun, Nov 13, 2011 at 09:53:13PM +0200, Leon Romanovsky wrote:
- SOC_DOUBLE_TLV("Line Playback Volume",
- SOC_DOUBLE_TLV("Speaker Playback Volume",
ALC5632_SPK_OUT_VOL, 8, 0, 31, 1, hp_tlv),
This looks really suspicious, why are we renaming the line output as a speaker output? This might be appropriate on your board but if the drivers aren't part of the CODEC it might not be appropriate on another board.
There are two main reasons: 1) We are changing SPK_OUT_VOL which on the schematics marked as a speaker, and we want to be as close as possible to the datasheet. 2) We are tired to explain to users that the speaker volume controlled by "Line Playback Volume" in alsamixer.
ALC5632_LINE_IN_VOL, 8, 0, 31, 1, vol_tlv),
- SOC_DOUBLE_TLV("Stereo DAC Playback Volume",
- SOC_DOUBLE_TLV("Master Playback Volume",
ALC5632_STEREO_DAC_IN_VOL, 8, 0, 63, 1, vdac_tlv),
- SOC_DOUBLE("Stereo DAC Playback Switch",
- SOC_DOUBLE("Master Playback Switch",
This looks fine.
On Mon, Nov 14, 2011 at 11:50:05PM +0200, Leon Romanovsky wrote:
On Mon, Nov 14, 2011 at 23:39, Mark Brown
This looks really suspicious, why are we renaming the line output as a speaker output? This might be appropriate on your board but if the drivers aren't part of the CODEC it might not be appropriate on another board.
There are two main reasons:
- We are changing SPK_OUT_VOL which on the schematics marked as a
speaker, and we want to be as close as possible to the datasheet. 2) We are tired to explain to users that the speaker volume controlled by "Line Playback Volume" in alsamixer.
The above both sound board specific. What does the CODEC itself call these outputs?
On Tue, Nov 15, 2011 at 21:25, Mark Brown broonie@opensource.wolfsonmicro.com wrote:
On Mon, Nov 14, 2011 at 11:50:05PM +0200, Leon Romanovsky wrote:
On Mon, Nov 14, 2011 at 23:39, Mark Brown
This looks really suspicious, why are we renaming the line output as a speaker output? This might be appropriate on your board but if the drivers aren't part of the CODEC it might not be appropriate on another board.
There are two main reasons:
- We are changing SPK_OUT_VOL which on the schematics marked as a
speaker, and we want to be as close as possible to the datasheet. 2) We are tired to explain to users that the speaker volume controlled by "Line Playback Volume" in alsamixer.
The above both sound board specific. What does the CODEC itself call these outputs?
SPK_OUT/SPK_OUTN http://ac100.wikispaces.com/file/view/alc5632_mixer_path.PNG/251095266/alc56...
participants (2)
-
Leon Romanovsky
-
Mark Brown