[PATCH v3 4/7] platform: starfive: rename "stf,axp15060-regulator" -> "x-powers,axp15060"
Nam Cao
namcao at linutronix.de
Mon Feb 5 00:31:22 PST 2024
On 04/Feb/2024 Minda Chen wrote:
> >
> > OpenSBI uses the device tree compatible string "stf,axp15060-regulator"
> > for the regulator node. However, the string used by U-Boot (and Linux) is
> > actually "x-powers,axp15060". As OpenSBI gets the device tree from U-Boot,
> > this causes the regulator device to be undetected, and OpenSBI does not use
> > this device to perform board reset/shutdown.
> >
> > Rename this device tree compatible string to match U-Boot (and Linux).
> >
> > Signed-off-by: Nam Cao <namcao at linutronix.de>
> > ---
> > platform/generic/starfive/jh7110.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/platform/generic/starfive/jh7110.c
> > b/platform/generic/starfive/jh7110.c
> > index d3647c2..385e09c 100644
> > --- a/platform/generic/starfive/jh7110.c
> > +++ b/platform/generic/starfive/jh7110.c
> > @@ -224,7 +224,7 @@ static int pm_reset_init(void *fdt, int nodeoff, }
> >
> > static const struct fdt_match pm_reset_match[] = {
> > - { .compatible = "stf,axp15060-regulator", .data = (void *)true },
> > + { .compatible = "x-powers,axp15060", .data = (void *)true },
> > { },
> > };
> >
> It is OK.
> But upstream u-boot jh7110 dts file do not contain this compatible,
> Can you also upstream this in u-boot, or I will do this myself
It is in upstream U-Boot, just a few days ago:
https://source.denx.de/u-boot/u-boot/-/commit/6882255ac3107c58e1153311df8a8270087f8cb3
Best regards,
Nam
More information about the opensbi
mailing list