Skip to content

Commit 5cacc6f

Browse files
jwrdegoedebroonie
authored andcommitted
ASoC: rt5670: Correct RT5670_LDO_SEL_MASK
The RT5670_PWR_ANLG1 register has 3 bits to select the LDO voltage, so the correct mask is 0x7 not 0x3. Because of this wrong mask we were programming the ldo bits to a setting of binary 001 (0x05 & 0x03) instead of binary 101 when moving to SND_SOC_BIAS_PREPARE. According to the datasheet 001 is a reserved value, so no idea what it did, since the driver was working fine before I guess we got lucky and it does something which is ok. Fixes: 5e8351d ("ASoC: add RT5670 CODEC driver") Signed-off-by: Hans de Goede <[email protected]> Cc: [email protected] Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Mark Brown <[email protected]>
1 parent 0ceb8a3 commit 5cacc6f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

sound/soc/codecs/rt5670.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -757,7 +757,7 @@
757757
#define RT5670_PWR_VREF2_BIT 4
758758
#define RT5670_PWR_FV2 (0x1 << 3)
759759
#define RT5670_PWR_FV2_BIT 3
760-
#define RT5670_LDO_SEL_MASK (0x3)
760+
#define RT5670_LDO_SEL_MASK (0x7)
761761
#define RT5670_LDO_SEL_SFT 0
762762

763763
/* Power Management for Analog 2 (0x64) */

0 commit comments

Comments
 (0)