[PATCH 0/1] regulator: ad5398: Correct SW_POWER_DOWN bit number

[PATCH 0/1] regulator: ad5398: Correct SW_POWER_DOWN bit number

[Isaac Scott]

This patch corrects the bit number that represents the soft power down
bit of the AD5398. This was overlooked during testing as in my use case,
the regulator was enabled and disabled via multiple methods, such as
using the power down pin on the chip itself through a different driver.

I have tested this on v6.13, and the bit number selected here accurately
reflects what is listed in the datasheet for the AD5398.

Isaac Scott (1):
  regulator: ad5398: Correct the SW_POWER_DOWN bit number

 drivers/regulator/ad5398.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.43.0

[PATCH 1/1] regulator: ad5398: Correct the SW_POWER_DOWN bit number

[Isaac Scott]

The datasheet for the AD5398 regulator indicates that the Soft power
down bit is bit 15, not 16 as represented in the driver. Correct this to
ensure the functionality is implemented correctly.

Signed-off-by: Isaac Scott <isaac.scott@ideasonboard.com>
---
 drivers/regulator/ad5398.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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;
-- 
2.43.0

Re: [PATCH 0/1] regulator: ad5398: Correct SW_POWER_DOWN bit number

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 665 bytes --]

On Thu, Feb 06, 2025 at 12:24:58PM +0000, Isaac Scott wrote:
> This patch corrects the bit number that represents the soft power down
> bit of the AD5398. This was overlooked during testing as in my use case,
> the regulator was enabled and disabled via multiple methods, such as
> using the power down pin on the chip itself through a different driver.

Please don't send cover letters for single patches, if there is anything
that needs saying put it in the changelog of the patch or after the ---
if it's administrative stuff.  This reduces mail volume and ensures that 
any important information is recorded in the changelog rather than being
lost. 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

[PATCH 0/1] regulator: ad5398: Correct SW_POWER_DOWN bit number

[Isaac Scott]

This patch corrects the bit number that represents the soft power down
bit of the AD5398. This was overlooked during testing as in my use case,
the regulator was enabled and disabled via multiple methods, such as
using the power down pin on the chip itself through a different driver.

I have tested this on v6.13, and the bit number selected here accurately
reflects what is listed in the datasheet for the AD5398.

Isaac Scott (1):
  regulator: ad5398: Correct the SW_POWER_DOWN bit number

 drivers/regulator/ad5398.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.43.0