[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