* [PATCH 0/6] Use correct LDO5 control registers for PCA9450
@ 2025-08-11 13:11 Frieder Schrempf
2025-08-11 13:11 ` [PATCH 1/6] Revert "pmic: pca9450: Add optional SD_VSEL GPIO for LDO5" Frieder Schrempf
` (7 more replies)
0 siblings, 8 replies; 14+ messages in thread
From: Frieder Schrempf @ 2025-08-11 13:11 UTC (permalink / raw)
To: u-boot, Fabio Estevam, Fabio Estevam, Jaehoon Chung, Marek Vasut,
Stefano Babic, Tom Rini, u-boot
Cc: Frieder Schrempf, Joy Zou, NXP i.MX U-Boot Team, Peng Fan, Ye Li
From: Frieder Schrempf <frieder.schrempf@kontron.de>
This is a port for the following series in Linux, which is already
available in mainline:
https://patchwork.kernel.org/project/linux-arm-kernel/cover/20241218152842.97483-1-frieder@fris.de/
This aligns the PMIC driver with current Linux and allows to use
upstream devicetrees in U-Boot.
Frieder Schrempf (6):
Revert "pmic: pca9450: Add optional SD_VSEL GPIO for LDO5"
pmic: pca9450: Fix enable register for LDO5
pmic: pca9450: Fix control register for LDO5
pmic: pca9450: Handle hardware with fixed SD_VSEL for LDO5
arm: dts: imx8mp-data-modul-edm-sbc: Remove deprecated sd-vsel-gpios
arm: dts: imx8mp-dhcom-som: Remove deprecated sd-vsel-gpios
arch/arm/dts/imx8mp-data-modul-edm-sbc.dts | 1 -
arch/arm/dts/imx8mp-dhcom-som.dtsi | 1 -
drivers/power/pmic/pca9450.c | 22 -------------
drivers/power/regulator/pca9450.c | 37 ++++++++++++++++++++--
4 files changed, 34 insertions(+), 27 deletions(-)
--
2.50.1
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 1/6] Revert "pmic: pca9450: Add optional SD_VSEL GPIO for LDO5"
2025-08-11 13:11 [PATCH 0/6] Use correct LDO5 control registers for PCA9450 Frieder Schrempf
@ 2025-08-11 13:11 ` Frieder Schrempf
2025-08-11 13:11 ` [PATCH 2/6] pmic: pca9450: Fix enable register for LDO5 Frieder Schrempf
` (6 subsequent siblings)
7 siblings, 0 replies; 14+ messages in thread
From: Frieder Schrempf @ 2025-08-11 13:11 UTC (permalink / raw)
To: u-boot, Jaehoon Chung, Tom Rini
Cc: Frieder Schrempf, Joy Zou, Peng Fan, Ye Li
From: Frieder Schrempf <frieder.schrempf@kontron.de>
This reverts commit 2add0511757e2c5897a88b57c5ea8c912140e60f.
It turns out that all boards using the PCA9450 actually have the
SD_VSEL input connected to the VSELECT signal of the SoCs SD/MMC
interface. Therefore we don't need manual control for this signal
via GPIO and there aren't any users.
This is equivalent to the following change in Linux:
c73be62caabb ("Revert "regulator: pca9450: Add SD_VSEL GPIO for LDO5"")
Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
drivers/power/pmic/pca9450.c | 22 ----------------------
1 file changed, 22 deletions(-)
diff --git a/drivers/power/pmic/pca9450.c b/drivers/power/pmic/pca9450.c
index 9d875f8bdbe..0e3d14abf15 100644
--- a/drivers/power/pmic/pca9450.c
+++ b/drivers/power/pmic/pca9450.c
@@ -6,12 +6,9 @@
#include <fdtdec.h>
#include <errno.h>
#include <dm.h>
-#include <dm/device_compat.h>
#include <i2c.h>
-#include <linux/err.h>
#include <log.h>
#include <asm/global_data.h>
-#include <asm-generic/gpio.h>
#include <linux/printk.h>
#include <power/pmic.h>
#include <power/regulator.h>
@@ -29,10 +26,6 @@ static const struct pmic_child_info pmic_children_info[] = {
{ },
};
-struct pca9450_priv {
- struct gpio_desc *sd_vsel_gpio;
-};
-
static int pca9450_reg_count(struct udevice *dev)
{
return PCA9450_REG_NUM;
@@ -85,21 +78,7 @@ static int pca9450_bind(struct udevice *dev)
static int pca9450_probe(struct udevice *dev)
{
- struct pca9450_priv *priv = dev_get_priv(dev);
unsigned int reset_ctrl;
- int ret = 0;
-
- if (CONFIG_IS_ENABLED(DM_GPIO) && CONFIG_IS_ENABLED(DM_REGULATOR_PCA9450)) {
- priv->sd_vsel_gpio = devm_gpiod_get_optional(dev, "sd-vsel",
- GPIOD_IS_OUT |
- GPIOD_IS_OUT_ACTIVE);
- if (IS_ERR(priv->sd_vsel_gpio)) {
- ret = PTR_ERR(priv->sd_vsel_gpio);
- dev_err(dev, "Failed to request SD_VSEL GPIO: %d\n", ret);
- if (ret)
- return ret;
- }
- }
if (ofnode_read_bool(dev_ofnode(dev), "nxp,wdog_b-warm-reset"))
reset_ctrl = PCA9450_PMIC_RESET_WDOG_B_CFG_WARM;
@@ -132,5 +111,4 @@ U_BOOT_DRIVER(pmic_pca9450) = {
.bind = pca9450_bind,
.probe = pca9450_probe,
.ops = &pca9450_ops,
- .priv_auto = sizeof(struct pca9450_priv),
};
--
2.50.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 2/6] pmic: pca9450: Fix enable register for LDO5
2025-08-11 13:11 [PATCH 0/6] Use correct LDO5 control registers for PCA9450 Frieder Schrempf
2025-08-11 13:11 ` [PATCH 1/6] Revert "pmic: pca9450: Add optional SD_VSEL GPIO for LDO5" Frieder Schrempf
@ 2025-08-11 13:11 ` Frieder Schrempf
2025-08-11 13:11 ` [PATCH 3/6] pmic: pca9450: Fix control " Frieder Schrempf
` (5 subsequent siblings)
7 siblings, 0 replies; 14+ messages in thread
From: Frieder Schrempf @ 2025-08-11 13:11 UTC (permalink / raw)
To: u-boot, Fabio Estevam, Jaehoon Chung, Marek Vasut, Tom Rini
Cc: Frieder Schrempf, Joy Zou, Peng Fan, Ye Li
From: Frieder Schrempf <frieder.schrempf@kontron.de>
The LDO5 regulator has two configuration registers, but only
LDO5CTRL_L contains the bits for enabling/disabling the regulator.
This is equivalent to the following change in Linux:
f5aab0438ef1 ("regulator: pca9450: Fix enable register for LDO5")
Fixes: 326337fb005f ("pmic: pca9450: Add regulator driver")
Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
drivers/power/regulator/pca9450.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/regulator/pca9450.c b/drivers/power/regulator/pca9450.c
index a2a34244723..27db55e688d 100644
--- a/drivers/power/regulator/pca9450.c
+++ b/drivers/power/regulator/pca9450.c
@@ -144,7 +144,7 @@ static struct pca9450_plat pca9450_reg_data[] = {
PCA_DATA("LDO4", PCA9450_LDO4CTRL, HW_STATE_CONTROL,
PCA9450_LDO4CTRL, PCA9450_LDO34_MASK,
pca9450_ldo34_vranges),
- PCA_DATA("LDO5", PCA9450_LDO5CTRL_H, HW_STATE_CONTROL,
+ PCA_DATA("LDO5", PCA9450_LDO5CTRL_L, HW_STATE_CONTROL,
PCA9450_LDO5CTRL_H, PCA9450_LDO5_MASK,
pca9450_ldo5_vranges),
};
--
2.50.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 3/6] pmic: pca9450: Fix control register for LDO5
2025-08-11 13:11 [PATCH 0/6] Use correct LDO5 control registers for PCA9450 Frieder Schrempf
2025-08-11 13:11 ` [PATCH 1/6] Revert "pmic: pca9450: Add optional SD_VSEL GPIO for LDO5" Frieder Schrempf
2025-08-11 13:11 ` [PATCH 2/6] pmic: pca9450: Fix enable register for LDO5 Frieder Schrempf
@ 2025-08-11 13:11 ` Frieder Schrempf
2025-08-11 13:12 ` [PATCH 4/6] pmic: pca9450: Handle hardware with fixed SD_VSEL " Frieder Schrempf
` (4 subsequent siblings)
7 siblings, 0 replies; 14+ messages in thread
From: Frieder Schrempf @ 2025-08-11 13:11 UTC (permalink / raw)
To: u-boot, Jaehoon Chung, Tom Rini
Cc: Frieder Schrempf, Joy Zou, Peng Fan, Ye Li
From: Frieder Schrempf <frieder.schrempf@kontron.de>
For LDO5 we need to be able to check the status of the SD_VSEL input in
order to know which control register is used. Read the status of the
SD_VSEL signal via GPIO and use the correct register accordingly.
To use this, the LDO5 node in the devicetree needs the sd-vsel-gpios
property to reference the GPIO that is used to read back the SD_VSEL
status internally. Please note that the SION bit in the IOMUX must be
set if the signal is muxed as VSELECT and controlled by the USDHC
controller.
This is equivalent to the following change in Linux:
3ce6f4f943dd ("regulator: pca9450: Fix control register for LDO5")
Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
drivers/power/regulator/pca9450.c | 31 +++++++++++++++++++++++++++++--
1 file changed, 29 insertions(+), 2 deletions(-)
diff --git a/drivers/power/regulator/pca9450.c b/drivers/power/regulator/pca9450.c
index 27db55e688d..9d8d142b464 100644
--- a/drivers/power/regulator/pca9450.c
+++ b/drivers/power/regulator/pca9450.c
@@ -7,9 +7,12 @@
* ROHM BD71837 regulator driver
*/
+#include <asm-generic/gpio.h>
#include <dm.h>
+#include <dm/device_compat.h>
#include <log.h>
#include <linux/bitops.h>
+#include <linux/err.h>
#include <power/pca9450.h>
#include <power/pmic.h>
#include <power/regulator.h>
@@ -52,6 +55,7 @@ struct pca9450_plat {
u8 volt_mask;
struct pca9450_vrange *ranges;
unsigned int numranges;
+ struct gpio_desc *sd_vsel_gpio;
};
#define PCA_RANGE(_min, _vstep, _sel_low, _sel_hi) \
@@ -222,13 +226,23 @@ static int pca9450_set_enable(struct udevice *dev, bool enable)
val);
}
+static u8 pca9450_get_vsel_reg(struct pca9450_plat *plat)
+{
+ if (!strcmp(plat->name, "LDO5") &&
+ (plat->sd_vsel_gpio && !dm_gpio_get_value(plat->sd_vsel_gpio)) {
+ return PCA9450_LDO5CTRL_L;
+ }
+
+ return plat->volt_reg;
+}
+
static int pca9450_get_value(struct udevice *dev)
{
struct pca9450_plat *plat = dev_get_plat(dev);
unsigned int reg, tmp;
int i, ret;
- ret = pmic_reg_read(dev->parent, plat->volt_reg);
+ ret = pmic_reg_read(dev->parent, pca9450_get_vsel_reg(plat));
if (ret < 0)
return ret;
@@ -274,7 +288,7 @@ static int pca9450_set_value(struct udevice *dev, int uvolt)
if (!found)
return -EINVAL;
- return pmic_clrsetbits(dev->parent, plat->volt_reg,
+ return pmic_clrsetbits(dev->parent, pca9450_get_vsel_reg(plat),
plat->volt_mask, sel);
}
@@ -335,6 +349,19 @@ static int pca9450_regulator_probe(struct udevice *dev)
*plat = pca9450_reg_data[i];
+ if (!strcmp(plat->name, "LDO5")) {
+ if (CONFIG_IS_ENABLED(DM_GPIO) && CONFIG_IS_ENABLED(DM_REGULATOR_PCA9450)) {
+ plat->sd_vsel_gpio = devm_gpiod_get_optional(dev, "sd-vsel",
+ GPIOD_IS_IN);
+ if (IS_ERR(plat->sd_vsel_gpio)) {
+ ret = PTR_ERR(plat->sd_vsel_gpio);
+ dev_err(dev, "Failed to request SD_VSEL GPIO: %d\n", ret);
+ if (ret)
+ return ret;
+ }
+ }
+ }
+
return 0;
}
--
2.50.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 4/6] pmic: pca9450: Handle hardware with fixed SD_VSEL for LDO5
2025-08-11 13:11 [PATCH 0/6] Use correct LDO5 control registers for PCA9450 Frieder Schrempf
` (2 preceding siblings ...)
2025-08-11 13:11 ` [PATCH 3/6] pmic: pca9450: Fix control " Frieder Schrempf
@ 2025-08-11 13:12 ` Frieder Schrempf
2025-08-11 13:12 ` [PATCH 5/6] arm: dts: imx8mp-data-modul-edm-sbc: Remove deprecated sd-vsel-gpios Frieder Schrempf
` (3 subsequent siblings)
7 siblings, 0 replies; 14+ messages in thread
From: Frieder Schrempf @ 2025-08-11 13:12 UTC (permalink / raw)
To: u-boot, Jaehoon Chung, Tom Rini
Cc: Frieder Schrempf, Joy Zou, Peng Fan, Ye Li
From: Frieder Schrempf <frieder.schrempf@kontron.de>
There are two ways to set the output voltage of the LD05
regulator. First by writing to the voltage selection registers
and second by toggling the SD_VSEL signal.
Usually board designers connect SD_VSEL to the VSELECT signal
controlled by the USDHC controller, but in some cases the
signal is hardwired to a fixed low level (therefore selecting
3.3V as initial value for allowing to boot from the SD card).
In these cases, the voltage is only determined by the value
of the LDO5CTRL_L register. Introduce a property
nxp,sd-vsel-fixed-low to let the driver know that SD_VSEL
is low and there is no GPIO to actually get that
information from dynamically.
This is equivalent to the following change in Linux:
c8c1ab2c5cb7 ("regulator: pca9450: Handle hardware with fixed SD_VSEL for LDO5")
Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
drivers/power/regulator/pca9450.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/drivers/power/regulator/pca9450.c b/drivers/power/regulator/pca9450.c
index 9d8d142b464..3bb0c71dde7 100644
--- a/drivers/power/regulator/pca9450.c
+++ b/drivers/power/regulator/pca9450.c
@@ -56,6 +56,7 @@ struct pca9450_plat {
struct pca9450_vrange *ranges;
unsigned int numranges;
struct gpio_desc *sd_vsel_gpio;
+ bool sd_vsel_fixed_low;
};
#define PCA_RANGE(_min, _vstep, _sel_low, _sel_hi) \
@@ -229,7 +230,8 @@ static int pca9450_set_enable(struct udevice *dev, bool enable)
static u8 pca9450_get_vsel_reg(struct pca9450_plat *plat)
{
if (!strcmp(plat->name, "LDO5") &&
- (plat->sd_vsel_gpio && !dm_gpio_get_value(plat->sd_vsel_gpio)) {
+ ((plat->sd_vsel_gpio && !dm_gpio_get_value(plat->sd_vsel_gpio)) ||
+ plat->sd_vsel_fixed_low)) {
return PCA9450_LDO5CTRL_L;
}
@@ -360,6 +362,8 @@ static int pca9450_regulator_probe(struct udevice *dev)
return ret;
}
}
+
+ plat->sd_vsel_fixed_low = dev_read_bool(dev, "nxp,sd-vsel-fixed-low");
}
return 0;
--
2.50.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 5/6] arm: dts: imx8mp-data-modul-edm-sbc: Remove deprecated sd-vsel-gpios
2025-08-11 13:11 [PATCH 0/6] Use correct LDO5 control registers for PCA9450 Frieder Schrempf
` (3 preceding siblings ...)
2025-08-11 13:12 ` [PATCH 4/6] pmic: pca9450: Handle hardware with fixed SD_VSEL " Frieder Schrempf
@ 2025-08-11 13:12 ` Frieder Schrempf
2025-08-11 14:22 ` Marek Vasut
2025-08-11 13:12 ` [PATCH 6/6] arm: dts: imx8mp-dhcom-som: " Frieder Schrempf
` (2 subsequent siblings)
7 siblings, 1 reply; 14+ messages in thread
From: Frieder Schrempf @ 2025-08-11 13:12 UTC (permalink / raw)
To: u-boot, Fabio Estevam, Marek Vasut, Stefano Babic, Tom Rini
Cc: Frieder Schrempf, NXP i.MX U-Boot Team
From: Frieder Schrempf <frieder.schrempf@kontron.de>
The sd-vsel-gpios property in the root of the PMIC node is deprecated
and therefore not parsed by the driver anymore. We can safely remove
this as it wasn't used anyway due to the pad not having the correct
pinmux settings.
Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
arch/arm/dts/imx8mp-data-modul-edm-sbc.dts | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/dts/imx8mp-data-modul-edm-sbc.dts b/arch/arm/dts/imx8mp-data-modul-edm-sbc.dts
index 8066f7fb649..6b40106e3bd 100644
--- a/arch/arm/dts/imx8mp-data-modul-edm-sbc.dts
+++ b/arch/arm/dts/imx8mp-data-modul-edm-sbc.dts
@@ -344,7 +344,6 @@
pinctrl-0 = <&pinctrl_pmic>;
interrupt-parent = <&gpio1>;
interrupts = <3 IRQ_TYPE_LEVEL_LOW>;
- sd-vsel-gpios = <&gpio1 4 GPIO_ACTIVE_HIGH>;
/*
* i.MX 8M Plus Data Sheet for Consumer Products
--
2.50.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 6/6] arm: dts: imx8mp-dhcom-som: Remove deprecated sd-vsel-gpios
2025-08-11 13:11 [PATCH 0/6] Use correct LDO5 control registers for PCA9450 Frieder Schrempf
` (4 preceding siblings ...)
2025-08-11 13:12 ` [PATCH 5/6] arm: dts: imx8mp-data-modul-edm-sbc: Remove deprecated sd-vsel-gpios Frieder Schrempf
@ 2025-08-11 13:12 ` Frieder Schrempf
2025-08-11 14:21 ` Marek Vasut
2025-08-25 8:40 ` [PATCH 0/6] Use correct LDO5 control registers for PCA9450 Frieder Schrempf
2025-09-01 7:44 ` Peng Fan (OSS)
7 siblings, 1 reply; 14+ messages in thread
From: Frieder Schrempf @ 2025-08-11 13:12 UTC (permalink / raw)
To: u-boot, Fabio Estevam, Marek Vasut, Stefano Babic, Tom Rini,
u-boot
Cc: Frieder Schrempf, NXP i.MX U-Boot Team
From: Frieder Schrempf <frieder.schrempf@kontron.de>
The sd-vsel-gpios property in the root of the PMIC node is deprecated
and therefore not parsed by the driver anymore. We can safely remove
this as it wasn't used anyway due to the pad not having the correct
pinmux settings.
Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
arch/arm/dts/imx8mp-dhcom-som.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/dts/imx8mp-dhcom-som.dtsi b/arch/arm/dts/imx8mp-dhcom-som.dtsi
index f2d99d05854..c1ca3805737 100644
--- a/arch/arm/dts/imx8mp-dhcom-som.dtsi
+++ b/arch/arm/dts/imx8mp-dhcom-som.dtsi
@@ -245,7 +245,6 @@
pinctrl-0 = <&pinctrl_pmic>;
interrupt-parent = <&gpio1>;
interrupts = <3 IRQ_TYPE_LEVEL_LOW>;
- sd-vsel-gpios = <&gpio1 4 GPIO_ACTIVE_HIGH>;
/*
* i.MX 8M Plus Data Sheet for Consumer Products
--
2.50.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH 6/6] arm: dts: imx8mp-dhcom-som: Remove deprecated sd-vsel-gpios
2025-08-11 13:12 ` [PATCH 6/6] arm: dts: imx8mp-dhcom-som: " Frieder Schrempf
@ 2025-08-11 14:21 ` Marek Vasut
0 siblings, 0 replies; 14+ messages in thread
From: Marek Vasut @ 2025-08-11 14:21 UTC (permalink / raw)
To: Frieder Schrempf, u-boot, Fabio Estevam, Stefano Babic, Tom Rini,
u-boot
Cc: Frieder Schrempf, NXP i.MX U-Boot Team
On 8/11/25 3:12 PM, Frieder Schrempf wrote:
> From: Frieder Schrempf <frieder.schrempf@kontron.de>
>
> The sd-vsel-gpios property in the root of the PMIC node is deprecated
> and therefore not parsed by the driver anymore. We can safely remove
> this as it wasn't used anyway due to the pad not having the correct
> pinmux settings.
>
> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
Reviewed-by: Marek Vasut <marek.vasut@mailbox.org>
I'll eventually finish switch of two leftover boards to OF_UPSTREAM and
drop this whole file, thanks for the reminder.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 5/6] arm: dts: imx8mp-data-modul-edm-sbc: Remove deprecated sd-vsel-gpios
2025-08-11 13:12 ` [PATCH 5/6] arm: dts: imx8mp-data-modul-edm-sbc: Remove deprecated sd-vsel-gpios Frieder Schrempf
@ 2025-08-11 14:22 ` Marek Vasut
0 siblings, 0 replies; 14+ messages in thread
From: Marek Vasut @ 2025-08-11 14:22 UTC (permalink / raw)
To: Frieder Schrempf, u-boot, Fabio Estevam, Stefano Babic, Tom Rini
Cc: Frieder Schrempf, NXP i.MX U-Boot Team
On 8/11/25 3:12 PM, Frieder Schrempf wrote:
> From: Frieder Schrempf <frieder.schrempf@kontron.de>
>
> The sd-vsel-gpios property in the root of the PMIC node is deprecated
> and therefore not parsed by the driver anymore. We can safely remove
> this as it wasn't used anyway due to the pad not having the correct
> pinmux settings.
>
> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
Reviewed-by: Marek Vasut <marek.vasut@mailbox.org>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 0/6] Use correct LDO5 control registers for PCA9450
2025-08-11 13:11 [PATCH 0/6] Use correct LDO5 control registers for PCA9450 Frieder Schrempf
` (5 preceding siblings ...)
2025-08-11 13:12 ` [PATCH 6/6] arm: dts: imx8mp-dhcom-som: " Frieder Schrempf
@ 2025-08-25 8:40 ` Frieder Schrempf
2025-08-27 5:46 ` Peng Fan
2025-09-01 7:44 ` Peng Fan (OSS)
7 siblings, 1 reply; 14+ messages in thread
From: Frieder Schrempf @ 2025-08-25 8:40 UTC (permalink / raw)
To: Jaehoon Chung, Tom Rini
Cc: Joy Zou, NXP i.MX U-Boot Team, Stefano Babic, Peng Fan, Ye Li,
Fabio Estevam, Fabio Estevam, Frieder Schrempf, u-boot,
Marek Vasut, u-boot
Hi Jaehoon,
Am 11.08.25 um 15:11 schrieb Frieder Schrempf:
> From: Frieder Schrempf <frieder.schrempf@kontron.de>
>
> This is a port for the following series in Linux, which is already
> available in mainline:
>
> https://patchwork.kernel.org/project/linux-arm-kernel/cover/20241218152842.97483-1-frieder@fris.de/
>
> This aligns the PMIC driver with current Linux and allows to use
> upstream devicetrees in U-Boot.
>
> Frieder Schrempf (6):
> Revert "pmic: pca9450: Add optional SD_VSEL GPIO for LDO5"
> pmic: pca9450: Fix enable register for LDO5
> pmic: pca9450: Fix control register for LDO5
> pmic: pca9450: Handle hardware with fixed SD_VSEL for LDO5
> arm: dts: imx8mp-data-modul-edm-sbc: Remove deprecated sd-vsel-gpios
> arm: dts: imx8mp-dhcom-som: Remove deprecated sd-vsel-gpios
>
> arch/arm/dts/imx8mp-data-modul-edm-sbc.dts | 1 -
> arch/arm/dts/imx8mp-dhcom-som.dtsi | 1 -
> drivers/power/pmic/pca9450.c | 22 -------------
> drivers/power/regulator/pca9450.c | 37 ++++++++++++++++++++--
> 4 files changed, 34 insertions(+), 27 deletions(-)
>
I saw that there are several patches on the mailing list for
/drivers/power that have received no reply from a maintainer for several
weeks (see [1]). I just wanted to make sure that you are aware of this
and kindly ask to have a look.
Thanks
Frieder
[1]
https://patchwork.ozlabs.org/project/uboot/list/?series=&submitter=&state=1&q=&archive=both&delegate=12423
^ permalink raw reply [flat|nested] 14+ messages in thread
* RE: [PATCH 0/6] Use correct LDO5 control registers for PCA9450
2025-08-25 8:40 ` [PATCH 0/6] Use correct LDO5 control registers for PCA9450 Frieder Schrempf
@ 2025-08-27 5:46 ` Peng Fan
2025-08-27 7:37 ` Frieder Schrempf
0 siblings, 1 reply; 14+ messages in thread
From: Peng Fan @ 2025-08-27 5:46 UTC (permalink / raw)
To: Frieder Schrempf, Jaehoon Chung, Tom Rini
Cc: Joy Zou, dl-uboot-imx, Stefano Babic, Ye Li, Fabio Estevam,
Fabio Estevam, Frieder Schrempf, u-boot@lists.denx.de,
Marek Vasut, u-boot@dh-electronics.com
> Subject: Re: [PATCH 0/6] Use correct LDO5 control registers for
> PCA9450
>
> Hi Jaehoon,
>
> Am 11.08.25 um 15:11 schrieb Frieder Schrempf:
> > From: Frieder Schrempf <frieder.schrempf@kontron.de>
> >
> > This is a port for the following series in Linux, which is already
> > available in mainline:
> >
> >
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2F
> patc
> > hwork.kernel.org%2Fproject%2Flinux-arm-
> kernel%2Fcover%2F20241218152842
> > .97483-1-
> frieder%40fris.de%2F&data=05%7C02%7Cpeng.fan%40nxp.com%7C51
> 6d
> >
> 65c0706c4d71292d08dde3b31d9d%7C686ea1d3bc2b4c6fa92cd99c5c
> 301635%7C0%7C
> >
> 0%7C638917080648452454%7CUnknown%7CTWFpbGZsb3d8eyJFbXB
> 0eU1hcGkiOnRydWU
> >
> sIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%
> 3D%3D%
> >
> 7C0%7C%7C%7C&sdata=neJx6WZU4iZWjSUcsnzOVQvFiaCTKLChE8srEx
> 6cwEQ%3D&rese
> > rved=0
> >
> > This aligns the PMIC driver with current Linux and allows to use
> > upstream devicetrees in U-Boot.
> >
> > Frieder Schrempf (6):
> > Revert "pmic: pca9450: Add optional SD_VSEL GPIO for LDO5"
> > pmic: pca9450: Fix enable register for LDO5
> > pmic: pca9450: Fix control register for LDO5
> > pmic: pca9450: Handle hardware with fixed SD_VSEL for LDO5
> > arm: dts: imx8mp-data-modul-edm-sbc: Remove deprecated sd-
> vsel-gpios
> > arm: dts: imx8mp-dhcom-som: Remove deprecated sd-vsel-gpios
> >
> > arch/arm/dts/imx8mp-data-modul-edm-sbc.dts | 1 -
> > arch/arm/dts/imx8mp-dhcom-som.dtsi | 1 -
> > drivers/power/pmic/pca9450.c | 22 -------------
> > drivers/power/regulator/pca9450.c | 37
> ++++++++++++++++++++--
> > 4 files changed, 34 insertions(+), 27 deletions(-)
> >
>
> I saw that there are several patches on the mailing list for
> /drivers/power that have received no reply from a maintainer for
> several weeks (see [1]). I just wanted to make sure that you are aware
> of this and kindly ask to have a look.
I could help handle that via mmc tree, if Tom and you are ok.
Regards
Peng.
>
> Thanks
> Frieder
>
> [1]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 0/6] Use correct LDO5 control registers for PCA9450
2025-08-27 5:46 ` Peng Fan
@ 2025-08-27 7:37 ` Frieder Schrempf
2025-08-27 14:32 ` Tom Rini
0 siblings, 1 reply; 14+ messages in thread
From: Frieder Schrempf @ 2025-08-27 7:37 UTC (permalink / raw)
To: Peng Fan, Jaehoon Chung, Tom Rini
Cc: Joy Zou, dl-uboot-imx, Stefano Babic, Ye Li, Fabio Estevam,
Fabio Estevam, Frieder Schrempf, u-boot@lists.denx.de,
Marek Vasut, u-boot@dh-electronics.com
Am 27.08.25 um 07:46 schrieb Peng Fan:
>> Subject: Re: [PATCH 0/6] Use correct LDO5 control registers for
>> PCA9450
>>
>> Hi Jaehoon,
>>
>> Am 11.08.25 um 15:11 schrieb Frieder Schrempf:
>>> From: Frieder Schrempf <frieder.schrempf@kontron.de>
>>>
>>> This is a port for the following series in Linux, which is already
>>> available in mainline:
>>>
>>>
>> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2F
>> patc
>>> hwork.kernel.org%2Fproject%2Flinux-arm-
>> kernel%2Fcover%2F20241218152842
>>> .97483-1-
>> frieder%40fris.de%2F&data=05%7C02%7Cpeng.fan%40nxp.com%7C51
>> 6d
>>>
>> 65c0706c4d71292d08dde3b31d9d%7C686ea1d3bc2b4c6fa92cd99c5c
>> 301635%7C0%7C
>>>
>> 0%7C638917080648452454%7CUnknown%7CTWFpbGZsb3d8eyJFbXB
>> 0eU1hcGkiOnRydWU
>>>
>> sIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%
>> 3D%3D%
>>>
>> 7C0%7C%7C%7C&sdata=neJx6WZU4iZWjSUcsnzOVQvFiaCTKLChE8srEx
>> 6cwEQ%3D&rese
>>> rved=0
>>>
>>> This aligns the PMIC driver with current Linux and allows to use
>>> upstream devicetrees in U-Boot.
>>>
>>> Frieder Schrempf (6):
>>> Revert "pmic: pca9450: Add optional SD_VSEL GPIO for LDO5"
>>> pmic: pca9450: Fix enable register for LDO5
>>> pmic: pca9450: Fix control register for LDO5
>>> pmic: pca9450: Handle hardware with fixed SD_VSEL for LDO5
>>> arm: dts: imx8mp-data-modul-edm-sbc: Remove deprecated sd-
>> vsel-gpios
>>> arm: dts: imx8mp-dhcom-som: Remove deprecated sd-vsel-gpios
>>>
>>> arch/arm/dts/imx8mp-data-modul-edm-sbc.dts | 1 -
>>> arch/arm/dts/imx8mp-dhcom-som.dtsi | 1 -
>>> drivers/power/pmic/pca9450.c | 22 -------------
>>> drivers/power/regulator/pca9450.c | 37
>> ++++++++++++++++++++--
>>> 4 files changed, 34 insertions(+), 27 deletions(-)
>>>
>>
>> I saw that there are several patches on the mailing list for
>> /drivers/power that have received no reply from a maintainer for
>> several weeks (see [1]). I just wanted to make sure that you are aware
>> of this and kindly ask to have a look.
>
> I could help handle that via mmc tree, if Tom and you are ok.
I don't mind. If Tom agrees to take power patches through the mmc tree
from you that's fine.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 0/6] Use correct LDO5 control registers for PCA9450
2025-08-27 7:37 ` Frieder Schrempf
@ 2025-08-27 14:32 ` Tom Rini
0 siblings, 0 replies; 14+ messages in thread
From: Tom Rini @ 2025-08-27 14:32 UTC (permalink / raw)
To: Frieder Schrempf, Peng Fan, Jaehoon Chung
Cc: Joy Zou, dl-uboot-imx, Stefano Babic, Ye Li, Fabio Estevam,
Fabio Estevam, Frieder Schrempf, u-boot@lists.denx.de,
Marek Vasut, u-boot@dh-electronics.com
[-- Attachment #1: Type: text/plain, Size: 2483 bytes --]
On Wed, Aug 27, 2025 at 09:37:24AM +0200, Frieder Schrempf wrote:
> Am 27.08.25 um 07:46 schrieb Peng Fan:
> >> Subject: Re: [PATCH 0/6] Use correct LDO5 control registers for
> >> PCA9450
> >>
> >> Hi Jaehoon,
> >>
> >> Am 11.08.25 um 15:11 schrieb Frieder Schrempf:
> >>> From: Frieder Schrempf <frieder.schrempf@kontron.de>
> >>>
> >>> This is a port for the following series in Linux, which is already
> >>> available in mainline:
> >>>
> >>>
> >> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2F
> >> patc
> >>> hwork.kernel.org%2Fproject%2Flinux-arm-
> >> kernel%2Fcover%2F20241218152842
> >>> .97483-1-
> >> frieder%40fris.de%2F&data=05%7C02%7Cpeng.fan%40nxp.com%7C51
> >> 6d
> >>>
> >> 65c0706c4d71292d08dde3b31d9d%7C686ea1d3bc2b4c6fa92cd99c5c
> >> 301635%7C0%7C
> >>>
> >> 0%7C638917080648452454%7CUnknown%7CTWFpbGZsb3d8eyJFbXB
> >> 0eU1hcGkiOnRydWU
> >>>
> >> sIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%
> >> 3D%3D%
> >>>
> >> 7C0%7C%7C%7C&sdata=neJx6WZU4iZWjSUcsnzOVQvFiaCTKLChE8srEx
> >> 6cwEQ%3D&rese
> >>> rved=0
> >>>
> >>> This aligns the PMIC driver with current Linux and allows to use
> >>> upstream devicetrees in U-Boot.
> >>>
> >>> Frieder Schrempf (6):
> >>> Revert "pmic: pca9450: Add optional SD_VSEL GPIO for LDO5"
> >>> pmic: pca9450: Fix enable register for LDO5
> >>> pmic: pca9450: Fix control register for LDO5
> >>> pmic: pca9450: Handle hardware with fixed SD_VSEL for LDO5
> >>> arm: dts: imx8mp-data-modul-edm-sbc: Remove deprecated sd-
> >> vsel-gpios
> >>> arm: dts: imx8mp-dhcom-som: Remove deprecated sd-vsel-gpios
> >>>
> >>> arch/arm/dts/imx8mp-data-modul-edm-sbc.dts | 1 -
> >>> arch/arm/dts/imx8mp-dhcom-som.dtsi | 1 -
> >>> drivers/power/pmic/pca9450.c | 22 -------------
> >>> drivers/power/regulator/pca9450.c | 37
> >> ++++++++++++++++++++--
> >>> 4 files changed, 34 insertions(+), 27 deletions(-)
> >>>
> >>
> >> I saw that there are several patches on the mailing list for
> >> /drivers/power that have received no reply from a maintainer for
> >> several weeks (see [1]). I just wanted to make sure that you are aware
> >> of this and kindly ask to have a look.
> >
> > I could help handle that via mmc tree, if Tom and you are ok.
>
> I don't mind. If Tom agrees to take power patches through the mmc tree
> from you that's fine.
This would be helpful, thanks!
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 0/6] Use correct LDO5 control registers for PCA9450
2025-08-11 13:11 [PATCH 0/6] Use correct LDO5 control registers for PCA9450 Frieder Schrempf
` (6 preceding siblings ...)
2025-08-25 8:40 ` [PATCH 0/6] Use correct LDO5 control registers for PCA9450 Frieder Schrempf
@ 2025-09-01 7:44 ` Peng Fan (OSS)
7 siblings, 0 replies; 14+ messages in thread
From: Peng Fan (OSS) @ 2025-09-01 7:44 UTC (permalink / raw)
To: u-boot, Fabio Estevam, Fabio Estevam, Jaehoon Chung, Marek Vasut,
Stefano Babic, Tom Rini, u-boot, Frieder Schrempf
Cc: Peng Fan, Frieder Schrempf, Joy Zou, NXP i.MX U-Boot Team, Ye Li
From: Peng Fan <peng.fan@nxp.com>
On Mon, 11 Aug 2025 15:11:56 +0200, Frieder Schrempf wrote:
> From: Frieder Schrempf <frieder.schrempf@kontron.de>
>
> This is a port for the following series in Linux, which is already
> available in mainline:
>
> https://patchwork.kernel.org/project/linux-arm-kernel/cover/20241218152842.97483-1-frieder@fris.de/
>
> [...]
Applied, thanks!
[1/6] Revert "pmic: pca9450: Add optional SD_VSEL GPIO for LDO5"
commit: b7360bd9e8e05dcfd1b4e5fd1c996aabf919e180
[2/6] pmic: pca9450: Fix enable register for LDO5
commit: 8a04fbd9afd635d37c9afc1b56d014d134861e29
[3/6] pmic: pca9450: Fix control register for LDO5
commit: addfe4544630a96a1cd03daf822a196af42b7a73
[4/6] pmic: pca9450: Handle hardware with fixed SD_VSEL for LDO5
commit: 281829f5ca1645c3bffa130d04753a1a76b64dfb
[5/6] arm: dts: imx8mp-data-modul-edm-sbc: Remove deprecated sd-vsel-gpios
commit: 925f63b0209483cd04b014cc8b5deab1f20267b7
[6/6] arm: dts: imx8mp-dhcom-som: Remove deprecated sd-vsel-gpios
commit: dc0a12fecea789de83015319f3e9aeddd0a824a2
Best regards,
--
Peng Fan <peng.fan@nxp.com>
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2025-09-01 6:33 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-11 13:11 [PATCH 0/6] Use correct LDO5 control registers for PCA9450 Frieder Schrempf
2025-08-11 13:11 ` [PATCH 1/6] Revert "pmic: pca9450: Add optional SD_VSEL GPIO for LDO5" Frieder Schrempf
2025-08-11 13:11 ` [PATCH 2/6] pmic: pca9450: Fix enable register for LDO5 Frieder Schrempf
2025-08-11 13:11 ` [PATCH 3/6] pmic: pca9450: Fix control " Frieder Schrempf
2025-08-11 13:12 ` [PATCH 4/6] pmic: pca9450: Handle hardware with fixed SD_VSEL " Frieder Schrempf
2025-08-11 13:12 ` [PATCH 5/6] arm: dts: imx8mp-data-modul-edm-sbc: Remove deprecated sd-vsel-gpios Frieder Schrempf
2025-08-11 14:22 ` Marek Vasut
2025-08-11 13:12 ` [PATCH 6/6] arm: dts: imx8mp-dhcom-som: " Frieder Schrempf
2025-08-11 14:21 ` Marek Vasut
2025-08-25 8:40 ` [PATCH 0/6] Use correct LDO5 control registers for PCA9450 Frieder Schrempf
2025-08-27 5:46 ` Peng Fan
2025-08-27 7:37 ` Frieder Schrempf
2025-08-27 14:32 ` Tom Rini
2025-09-01 7:44 ` Peng Fan (OSS)
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.