[PATCH] regulator: tps65023: Fix mask for LDOs output voltage select control

[PATCH] regulator: tps65023: Fix mask for LDOs output voltage select control

[Axel Lin]

According to the datasheet:
The LDO_CTRL registers are used to set the output voltage of LDO1 and LDO2.
LDO_CTRL[7] and LDO_CTRL[3] are reserved and should always be written to 0.

Thus the mask for TPS65023_LDO_1 and TPS65023_LDO_2 should be 0x07 and 0x70
respectively.

Signed-off-by: Axel Lin <axel.lin@gmail.com>
---
 drivers/regulator/tps65023-regulator.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/tps65023-regulator.c b/drivers/regulator/tps65023-regulator.c
index df8b70d..6fab428 100644
--- a/drivers/regulator/tps65023-regulator.c
+++ b/drivers/regulator/tps65023-regulator.c
@@ -71,7 +71,7 @@
 
 /* LDO_CTRL bitfields */
 #define TPS65023_LDO_CTRL_LDOx_SHIFT(ldo_id)	((ldo_id)*4)
-#define TPS65023_LDO_CTRL_LDOx_MASK(ldo_id)	(0x0F << ((ldo_id)*4))
+#define TPS65023_LDO_CTRL_LDOx_MASK(ldo_id)	(0x07 << ((ldo_id)*4))
 
 /* Number of step-down converters available */
 #define TPS65023_NUM_DCDC		3
-- 
1.7.9.5

Re: [PATCH] regulator: tps65023: Fix mask for LDOs output voltage select control

[Mark Brown]

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

On Thu, Jun 14, 2012 at 01:55:25PM +0800, Axel Lin wrote:
> According to the datasheet:
> The LDO_CTRL registers are used to set the output voltage of LDO1 and LDO2.
> LDO_CTRL[7] and LDO_CTRL[3] are reserved and should always be written to 0.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]