linux-kernel - [PATCH v3] regulator: axp20x: Fix DLDO2 voltage control register mask for AXP22x

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-Id: <20201201001000.22302-1-dinghua.ma.sz@gmail.com>
Date:   Tue,  1 Dec 2020 08:10:00 +0800
From:   dinghua.ma@...s.csie.org, dinghua.ma.sz@...il.com
To:     Chen-Yu Tsai <wens@...e.org>
Cc:     Mark Brown <broonie@...nel.org>,
        Liam Girdwood <lgirdwood@...il.com>,
        linux-kernel@...r.kernel.org,
        "DingHua Ma" <dinghua.ma.sz@...il.com>, stable@...r.kernel.org
Subject: [PATCH v3] regulator: axp20x: Fix DLDO2 voltage control register mask for AXP22x

From: "DingHua Ma" <dinghua.ma.sz@...il.com>

When I use the axp20x chip to power my SDIO device on the 5.4 kernel, 
the output voltage of DLDO2 is wrong. After comparing the register 
manual and source code of the chip, I found that the mask bit of the 
driver register of the port was wrong. I fixed this error by modifying 
the mask register of the source code. This error seems to be a copy 
error of the macro when writing the code. Now the voltage output of 
the DLDO2 port of axp20x is correct. My development environment is 
Allwinner A40I of arm architecture, and the kernel version is 5.4.

Signed-off-by: DingHua Ma <dinghua.ma.sz@...il.com>
Reviewed-by: Chen-Yu Tsai <wens@...e.org> 
Cc: <stable@...r.kernel.org>
Fixes: db4a555f7c4c ("regulator: axp20x: use defines for masks")
---
Changes since v2:
- Modify topic description
---
Changes since v1:
- More accurate description for this patch
---
 drivers/regulator/axp20x-regulator.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/axp20x-regulator.c b/drivers/regulator/axp20x-regulator.c
index cd1224182ad7..90cb8445f721 100644
--- a/drivers/regulator/axp20x-regulator.c
+++ b/drivers/regulator/axp20x-regulator.c
@@ -594,7 +594,7 @@ static const struct regulator_desc axp22x_regulators[] = {
 		 AXP22X_DLDO1_V_OUT, AXP22X_DLDO1_V_OUT_MASK,
 		 AXP22X_PWR_OUT_CTRL2, AXP22X_PWR_OUT_DLDO1_MASK),
 	AXP_DESC(AXP22X, DLDO2, "dldo2", "dldoin", 700, 3300, 100,
-		 AXP22X_DLDO2_V_OUT, AXP22X_PWR_OUT_DLDO2_MASK,
+		 AXP22X_DLDO2_V_OUT, AXP22X_DLDO2_V_OUT_MASK,
 		 AXP22X_PWR_OUT_CTRL2, AXP22X_PWR_OUT_DLDO2_MASK),
 	AXP_DESC(AXP22X, DLDO3, "dldo3", "dldoin", 700, 3300, 100,
 		 AXP22X_DLDO3_V_OUT, AXP22X_DLDO3_V_OUT_MASK,
-- 
2.25.1