]> git.baikalelectronics.ru Git - kernel.git/commit
ASoC: rt5651: Fix dac- and adc- vol-tlv values being off by a factor of 10
authorHans de Goede <hdegoede@redhat.com>
Fri, 26 Feb 2021 14:38:14 +0000 (15:38 +0100)
committerMark Brown <broonie@kernel.org>
Wed, 10 Mar 2021 12:49:43 +0000 (12:49 +0000)
commitc474d108b6ffe568b0f331d492b22ce4464900d2
treeebccc6549d4afb3fa2d1bb80211e1be48700dad0
parente7ecdbf0afdbeb9c30e31d466a75e5d1d2085524
ASoC: rt5651: Fix dac- and adc- vol-tlv values being off by a factor of 10

The adc_vol_tlv volume-control has a range from -17.625 dB to +30 dB,
not -176.25 dB to + 300 dB. This wrong scale is esp. a problem in userspace
apps which translate the dB scale to a linear scale. With the logarithmic
dB scale being of by a factor of 10 we loose all precision in the lower
area of the range when apps translate things to a linear scale.

E.g. the 0 dB default, which corresponds with a value of 47 of the
0 - 127 range for the control, would be shown as 0/100 in alsa-mixer.

Since the centi-dB values used in the TLV struct cannot represent the
0.375 dB step size used by these controls, change the TLV definition
for them to specify a min and max value instead of min + stepsize.

Note this mirrors commit ab3a23a7fe1c ("ASoC: rt5670: Fix dac- and adc-
vol-tlv values being off by a factor of 10") which made the exact same
change to the rt5670 codec driver.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Link: https://lore.kernel.org/r/20210226143817.84287-3-hdegoede@redhat.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/rt5651.c