diff options
| author | Dheeraj Reddy Jonnalagadda <dheeraj.linuxdev@gmail.com> | 2025-02-06 16:01:53 +0530 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2025-02-06 11:27:24 +0000 |
| commit | 0a7c85b516830c0bb088b0bdb2f2c50c76fc531a (patch) | |
| tree | 1bc585d434c33896ffde3d318220688f32c91e57 | |
| parent | 3c32a4386909e8023b3c49253fec33d267be16bb (diff) | |
regulator: ad5398: Fix incorrect power down bit mask
AD5398_SW_POWER_DOWN was defined with a bit position outside the valid
range of the device's 16-bit register. The bitwise operation with an
unsigned short would always evaluate to 0, making the power down check
ineffective.
Update AD5398_SW_POWER_DOWN to use a valid bit position within the
16-bit range of the register.
Fixes: 19d022d67d73 ("regulator: ad5398: change enable bit name to improve readibility")
Signed-off-by: Dheeraj Reddy Jonnalagadda <dheeraj.linuxdev@gmail.com>
Link: https://patch.msgid.link/20250206103153.59114-1-dheeraj.linuxdev@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
| -rw-r--r-- | drivers/regulator/ad5398.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/regulator/ad5398.c b/drivers/regulator/ad5398.c index 0274f41d0233..eb2a666a45cb 100644 --- a/drivers/regulator/ad5398.c +++ b/drivers/regulator/ad5398.c @@ -16,7 +16,7 @@ #include <linux/regulator/machine.h> #include <linux/regulator/of_regulator.h> -#define AD5398_SW_POWER_DOWN BIT(16) +#define AD5398_SW_POWER_DOWN BIT(15) struct ad5398_chip_info { struct i2c_client *client; |