Unverified Commit fcb3b5a5 authored by Charles Keepax's avatar Charles Keepax Committed by Mark Brown
Browse files

ASoC: cs42l51: Correct minimum value for SX volume control 



The minimum value for the PGA Volume is given as 0x1A, however the
values from there to 0x19 are all the same volume and this is not
represented in the TLV structure. The number of volumes given is correct
so this leads to all the volumes being shifted. Move the minimum value
up to 0x19 to fix this.

Signed-off-by: default avatarCharles Keepax <ckeepax@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20220602162119.3393857-7-ckeepax@opensource.cirrus.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent a8928ada

sound/soc/codecs/cs42l51.c

+1 −1
Original line number Diff line number Diff line
@@ -143,7 +143,7 @@ static const struct snd_kcontrol_new cs42l51_snd_controls[] = { 
			0, 0xA0, 96, adc_att_tlv),

	SOC_DOUBLE_R_SX_TLV("PGA Volume",

			CS42L51_ALC_PGA_CTL, CS42L51_ALC_PGB_CTL,

			0, 0x1A, 30, pga_tlv),

			0, 0x19, 30, pga_tlv),

	SOC_SINGLE("Playback Deemphasis Switch", CS42L51_DAC_CTL, 3, 1, 0),

	SOC_SINGLE("Auto-Mute Switch", CS42L51_DAC_CTL, 2, 1, 0),

	SOC_SINGLE("Soft Ramp Switch", CS42L51_DAC_CTL, 1, 1, 0),