Re: [PATCH v3 1/3] riscv: dts: spacemit: k3-deepcomputing-fml13v05: add P1 PMIC regulator tree

[Sandie Cao]

Hi Maud:

> From: "Maud Spierings"<maud_spierings@xxxxxxxxx>
> Date:  Wed, Nov 4, 2026, 23:31

> > diff --git a/arch/riscv/boot/dts/spacemit/k3-deepcomputing-fml13v05.dts b/arch/riscv/boot/dts/spacemit/k3-deepcomputing-fml13v05.dts
> > index b5d5112de358..804abc85602a 100644
> > --- a/arch/riscv/boot/dts/spacemit/k3-deepcomputing-fml13v05.dts
> > +++ b/arch/riscv/boot/dts/spacemit/k3-deepcomputing-fml13v05.dts
> > @@ -22,6 +22,161 @@ memory@100000000 {
> >                  device_type = "memory";
> >                  reg = <0x1 0x00000000 0x4 0x00000000>;
> >          };
> > +
> > +        reg_aux_vcc3v3: regulator-aux-vcc3v3 {
> > +                compatible = "regulator-fixed";
> > +                regulator-name = "AUX_VCC3V3";
> > +                regulator-min-microvolt = <3300000>;
> > +                regulator-max-microvolt = <3300000>;
> > +                regulator-always-on;
> > +        };
> > +
> > +        reg_aux_vcc5v: regulator-aux-vcc5v {
> > +                compatible = "regulator-fixed";
> > +                regulator-name = "AUX_VCC5V";
> > +                regulator-min-microvolt = <5000000>;
> > +                regulator-max-microvolt = <5000000>;
> > +                regulator-always-on;
> > +        };
> > +};
> > +
> > +&i2c8 {
> > +        pinctrl-names = "default";
> > +        pinctrl-0 = <&i2c8_cfg>;
> > +        status = "okay";
> > +
> > +        p1@41 {
> 
> I believe this should be pmic@41
> If it needs a label it could be p1: pmic@41 but the node name should be 
> more descriptive.

This patch follows k3-pico-itx.dts, it uses p1@41. And k3-com260.dtsi uses pmic@41.
I will uses pmic@41 in next version.

Best regards
Sandie

> 
> Kind regards,
> Maud
>