]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
regulator: spacemit-p1: Update supply names
authorGuodong Xu <guodong@riscstar.com>
Fri, 6 Feb 2026 02:32:03 +0000 (10:32 +0800)
committerMark Brown <broonie@kernel.org>
Sun, 22 Feb 2026 23:53:44 +0000 (23:53 +0000)
Update supply names to match the P1 PMIC's actual hardware pinout where
each buck has an individual VIN pin (vin1-vin6) and LDO groups have
dedicated input pins (aldoin, dldoin1, dldoin2).

This is an ABI change from the original "vin" and "buck5" supplies.
The P1/PMIC regulator has no consumers in the DTS tree yet. For the two
K1 boards in-tree (BPI-F3 and Jupiter), power settings come from
boot firmware, so a probe failure has minimal impact.

Signed-off-by: Guodong Xu <guodong@riscstar.com>
Link: https://developer.spacemit.com/documentation?token=T1Btw2BdiiSlSXkAdibcoMetnag
[1]
Reviewed-by: Alex Elder <elder@riscstar.com>
Link: https://patch.msgid.link/20260206-spacemit-p1-v4-2-8f695d93811e@riscstar.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/spacemit-p1.c

index 2b585ba01a93d0ce163c33e14c8022a506bdce11..57e6e00a73fa416fd6e557eb5bcada7dc69df444 100644 (file)
@@ -87,13 +87,16 @@ static const struct linear_range p1_ldo_ranges[] = {
        }
 
 #define P1_BUCK_DESC(_n) \
-       P1_REG_DESC(BUCK, buck, _n, "vin", 0x47, BUCK_MASK, 255, p1_buck_ranges)
+       P1_REG_DESC(BUCK, buck, _n, "vin" #_n, 0x47, BUCK_MASK, 255, p1_buck_ranges)
 
 #define P1_ALDO_DESC(_n) \
-       P1_REG_DESC(ALDO, aldo, _n, "vin", 0x5b, LDO_MASK, 128, p1_ldo_ranges)
+       P1_REG_DESC(ALDO, aldo, _n, "aldoin", 0x5b, LDO_MASK, 128, p1_ldo_ranges)
 
-#define P1_DLDO_DESC(_n) \
-       P1_REG_DESC(DLDO, dldo, _n, "buck5", 0x67, LDO_MASK, 128, p1_ldo_ranges)
+#define P1_DLDO1_DESC(_n) \
+       P1_REG_DESC(DLDO, dldo, _n, "dldoin1", 0x67, LDO_MASK, 128, p1_ldo_ranges)
+
+#define P1_DLDO2_DESC(_n) \
+       P1_REG_DESC(DLDO, dldo, _n, "dldoin2", 0x67, LDO_MASK, 128, p1_ldo_ranges)
 
 static const struct regulator_desc p1_regulator_desc[] = {
        P1_BUCK_DESC(1),
@@ -108,13 +111,13 @@ static const struct regulator_desc p1_regulator_desc[] = {
        P1_ALDO_DESC(3),
        P1_ALDO_DESC(4),
 
-       P1_DLDO_DESC(1),
-       P1_DLDO_DESC(2),
-       P1_DLDO_DESC(3),
-       P1_DLDO_DESC(4),
-       P1_DLDO_DESC(5),
-       P1_DLDO_DESC(6),
-       P1_DLDO_DESC(7),
+       P1_DLDO1_DESC(1),
+       P1_DLDO1_DESC(2),
+       P1_DLDO1_DESC(3),
+       P1_DLDO1_DESC(4),
+       P1_DLDO2_DESC(5),
+       P1_DLDO2_DESC(6),
+       P1_DLDO2_DESC(7),
 };
 
 static int p1_regulator_probe(struct platform_device *pdev)