power: rk8xx: fix swapped mask and value in init registers for RK806
authorQuentin Schulz <quentin.schulz@cherry.de>
Wed, 28 May 2025 12:07:27 +0000 (14:07 +0200)
committerKever Yang <kever.yang@rock-chips.com>
Fri, 6 Jun 2025 09:23:34 +0000 (17:23 +0800)
The val (the bits to set) is the second member of the reg_data structure
and mask the third one. We obviously want to clear bits 6 and 7 in order
to only set bit 7 in there instead of only clearing bit 7 in order to
write bits 6 and 7 (which makes no sense).

Fortunately, according to the datasheet, bit 6 value doesn't matter when
bit 7 is set so this is essentially just a cosmetic change, no intended
change in behavior.

Fixes: f172575d92cd ("power: rk8xx: add support for RK806")
Signed-off-by: Quentin Schulz <quentin.schulz@cherry.de>
Reviewed-by: Kever Yang <kever.yang@rock-chips.com>
drivers/power/pmic/rk8xx.c

index a14555c..3bc696d 100644 (file)
@@ -91,7 +91,7 @@ void rk8xx_off_for_plugin(struct udevice *dev)
 
 static struct reg_data rk806_init_reg[] = {
        /* RST_FUN */
-       { RK806_REG_SYS_CFG3, GENMASK(7, 6), BIT(7)},
+       { RK806_REG_SYS_CFG3, BIT(7), GENMASK(7, 6)},
 };
 
 static struct reg_data rk817_init_reg[] = {