[PATCH v1 4/9] ARM: dts: imx8mm-verdin: follow changed pmic

Oleksandr Suvorov oleksandr.suvorov at toradex.com
Thu Oct 29 17:19:38 CET 2020


On Thu, Oct 29, 2020 at 4:44 PM Marcel Ziswiler
<marcel.ziswiler at toradex.com> wrote:
>
> On Wed, 2020-10-28 at 11:58 +0200, Igor Opaniuk wrote:
> > From: Max Krummenacher <max.krummenacher at toradex.com>
> >
> > The used PMIC has been changed from RHOM BD71837 to NXP PCA9450A.
> > Adjust the device tree accordingly.
> > Remove the old ADC node as the ADC has been changed and has no longer
> > a separate power rail.
> >
> > Signed-off-by: Max Krummenacher <max.krummenacher at toradex.com>
> > Signed-off-by: Igor Opaniuk <igor.opaniuk at toradex.com>
>
> Acked-by: Marcel Ziswiler <marcel.ziswiler at toradex.com>

Reviewed-by: Oleksandr Suvorov <oleksandr.suvorov at toradex.com>

>
> > ---
> >
> >  arch/arm/dts/imx8mm-verdin-u-boot.dtsi |   5 +-
> >  arch/arm/dts/imx8mm-verdin.dts         | 154 +++++++++++++------------
> >  2 files changed, 80 insertions(+), 79 deletions(-)
> >
> > diff --git a/arch/arm/dts/imx8mm-verdin-u-boot.dtsi b/arch/arm/dts/imx8mm-verdin-u-boot.dtsi
> > index fe6bb9bf03..249b0f8f66 100644
> > --- a/arch/arm/dts/imx8mm-verdin-u-boot.dtsi
> > +++ b/arch/arm/dts/imx8mm-verdin-u-boot.dtsi
> > @@ -2,7 +2,6 @@
> >  /*
> >   * Copyright 2020 Toradex
> >   */
> > -
> >  / {
> >         wdt-reboot {
> >                 compatible = "wdt-reboot";
> > @@ -90,11 +89,11 @@
> >         u-boot,dm-spl;
> >  };
> >
> > -&{/soc at 0/bus at 30800000/i2c at 30a20000/pmic at 4b} {
> > +&{/soc at 0/bus at 30800000/i2c at 30a20000/pmic} {
> >         u-boot,dm-spl;
> >  };
> >
> > -&{/soc at 0/bus at 30800000/i2c at 30a20000/pmic at 4b/regulators} {
> > +&{/soc at 0/bus at 30800000/i2c at 30a20000/pmic/regulators} {
> >         u-boot,dm-spl;
> >  };
> >
> > diff --git a/arch/arm/dts/imx8mm-verdin.dts b/arch/arm/dts/imx8mm-verdin.dts
> > index 1c67c08c88..fb0756d6e1 100644
> > --- a/arch/arm/dts/imx8mm-verdin.dts
> > +++ b/arch/arm/dts/imx8mm-verdin.dts
> > @@ -203,115 +203,123 @@
> >         pinctrl-0 = <&pinctrl_i2c1>;
> >         status = "okay";
> >
> > -       pmic at 4b {
> > -               compatible = "rohm,bd71840", "rohm,bd71837";
> > -               bd71837,pmic-buck2-uses-i2c-dvs;
> > -               bd71837,pmic-buck2-dvs-voltage = <1000000>, <900000>, <0>; /* VDD_ARM: Run-Idle */
> > -               gpio_intr = <&gpio1 3 GPIO_ACTIVE_LOW>;
> > -               /* PMIC BD71837 PMIC_nINT GPIO1_IO3 */
> > +       /* Assembled on V1.1 HW and later */
> > +       pmic {
> > +               reg = <0x25>;
> > +               u-boot,dm-spl;
> > +               compatible = "nxp,pca9450a";
> > +               /* PMIC PCA9450 PMIC_nINT GPIO1_IO3 */
> >                 pinctrl-0 = <&pinctrl_pmic>;
> > -               reg = <0x4b>;
> > -
> > -               gpo {
> > -                       rohm,drv = <0x0C>;      /* 0b0000_1100 all gpos with cmos output mode */
> > -               };
> > +               gpio_intr = <&gpio1 3 GPIO_ACTIVE_LOW>;
> >
> >                 regulators {
> > -                       buck1_reg: BUCK1 {
> > -                               regulator-always-on;
> > -                               regulator-boot-on;
> > +                       u-boot,dm-spl;
> > +                       #address-cells = <1>;
> > +                       #size-cells = <0>;
> > +
> > +                       pca9450,pmic-buck2-uses-i2c-dvs;
> > +                       /* Run/Standby voltage */
> > +                       pca9450,pmic-buck2-dvs-voltage = <950000>, <850000>;
> > +
> > +                       buck1_reg: regulator at 0 {
> > +                               reg = <0>;
> >                                 regulator-compatible = "buck1";
> > -                               regulator-max-microvolt = <1300000>;
> > -                               regulator-min-microvolt = <700000>;
> > -                               regulator-ramp-delay = <1250>;
> > +                               regulator-min-microvolt = <600000>;
> > +                               regulator-max-microvolt = <2187500>;
> > +                               regulator-boot-on;
> > +                               regulator-always-on;
> > +                               regulator-ramp-delay = <3125>;
> >                         };
> >
> > -                       buck2_reg: BUCK2 {
> > -                               regulator-always-on;
> > -                               regulator-boot-on;
> > +                       buck2_reg: regulator at 1 {
> > +                               reg = <1>;
> >                                 regulator-compatible = "buck2";
> > -                               regulator-max-microvolt = <1300000>;
> > -                               regulator-min-microvolt = <700000>;
> > -                               regulator-ramp-delay = <1250>;
> > +                               regulator-min-microvolt = <600000>;
> > +                               regulator-max-microvolt = <2187500>;
> > +                               regulator-boot-on;
> > +                               regulator-always-on;
> > +                               regulator-ramp-delay = <3125>;
> >                         };
> >
> > -                       buck5_reg: BUCK5 {
> > -                               regulator-always-on;
> > +                       buck3_reg: regulator at 2 {
> > +                               reg = <2>;
> > +                               regulator-compatible = "buck3";
> > +                               regulator-min-microvolt = <600000>;
> > +                               regulator-max-microvolt = <2187500>;
> >                                 regulator-boot-on;
> > -                               regulator-compatible = "buck5";
> > -                               regulator-max-microvolt = <1350000>;
> > -                               regulator-min-microvolt = <700000>;
> > +                               regulator-always-on;
> >                         };
> >
> > -                       buck6_reg: BUCK6 {
> > -                               regulator-always-on;
> > +                       buck4_reg: regulator at 3 {
> > +                               reg = <3>;
> > +                               regulator-compatible = "buck4";
> > +                               regulator-min-microvolt = <600000>;
> > +                               regulator-max-microvolt = <3400000>;
> >                                 regulator-boot-on;
> > -                               regulator-compatible = "buck6";
> > -                               regulator-max-microvolt = <3300000>;
> > -                               regulator-min-microvolt = <3000000>;
> > +                               regulator-always-on;
> >                         };
> >
> > -                       buck7_reg: BUCK7 {
> > -                               regulator-always-on;
> > +                       buck5_reg: regulator at 4 {
> > +                               reg = <4>;
> > +                               regulator-compatible = "buck5";
> > +                               regulator-min-microvolt = <600000>;
> > +                               regulator-max-microvolt = <3400000>;
> >                                 regulator-boot-on;
> > -                               regulator-compatible = "buck7";
> > -                               regulator-max-microvolt = <1995000>;
> > -                               regulator-min-microvolt = <1605000>;
> > +                               regulator-always-on;
> >                         };
> >
> > -                       buck8_reg: BUCK8 {
> > -                               regulator-always-on;
> > +                       buck6_reg: regulator at 5 {
> > +                               reg = <5>;
> > +                               regulator-compatible = "buck6";
> > +                               regulator-min-microvolt = <600000>;
> > +                               regulator-max-microvolt = <3400000>;
> >                                 regulator-boot-on;
> > -                               regulator-compatible = "buck8";
> > -                               regulator-max-microvolt = <1400000>;
> > -                               regulator-min-microvolt = <800000>;
> > +                               regulator-always-on;
> >                         };
> >
> > -                       ldo1_reg: LDO1 {
> > -                               regulator-always-on;
> > -                               regulator-boot-on;
> > +                       ldo1_reg: regulator at 6 {
> > +                               reg = <6>;
> >                                 regulator-compatible = "ldo1";
> > +                               regulator-min-microvolt = <1600000>;
> >                                 regulator-max-microvolt = <3300000>;
> > -                               regulator-min-microvolt = <3000000>;
> > +                               regulator-boot-on;
> > +                               regulator-always-on;
> >                         };
> >
> > -                       ldo2_reg: LDO2 {
> > -                               regulator-always-on;
> > -                               regulator-boot-on;
> > +                       ldo2_reg: regulator at 7 {
> > +                               reg = <7>;
> >                                 regulator-compatible = "ldo2";
> > -                               regulator-max-microvolt = <900000>;
> > -                               regulator-min-microvolt = <900000>;
> > +                               regulator-min-microvolt = <800000>;
> > +                               regulator-max-microvolt = <1150000>;
> > +                               regulator-boot-on;
> > +                               regulator-always-on;
> >                         };
> >
> > -                       ldo3_reg: LDO3 {
> > -                               regulator-always-on;
> > -                               regulator-boot-on;
> > +                       ldo3_reg: regulator at 8 {
> > +                               reg = <8>;
> >                                 regulator-compatible = "ldo3";
> > +                               regulator-min-microvolt = <800000>;
> >                                 regulator-max-microvolt = <3300000>;
> > -                               regulator-min-microvolt = <1800000>;
> > +                               regulator-boot-on;
> > +                               regulator-always-on;
> >                         };
> >
> > -                       ldo4_reg: LDO4 {
> > -                               regulator-always-on;
> > -                               regulator-boot-on;
> > +                       ldo4_reg: regulator at 9 {
> > +                               reg = <9>;
> >                                 regulator-compatible = "ldo4";
> > -                               regulator-max-microvolt = <1800000>;
> > -                               regulator-min-microvolt = <900000>;
> > +                               regulator-min-microvolt = <800000>;
> > +                               regulator-max-microvolt = <3300000>;
> > +                               regulator-boot-on;
> > +                               regulator-always-on;
> >                         };
> >
> > -                       ldo5_reg: LDO5 {
> > +                       ldo5_reg: regulator at 10 {
> > +                               reg = <10>;
> >                                 regulator-compatible = "ldo5";
> > +                               regulator-min-microvolt = <800000>;
> >                                 regulator-max-microvolt = <3300000>;
> > -                               regulator-min-microvolt = <3300000>;
> >                         };
> >
> > -                       ldo6_reg: LDO6 {
> > -                               regulator-always-on;
> > -                               regulator-boot-on;
> > -                               regulator-compatible = "ldo6";
> > -                               regulator-max-microvolt = <1800000>;
> > -                               regulator-min-microvolt = <900000>;
> > -                       };
> >                 };
> >         };
> >
> > @@ -321,12 +329,6 @@
> >                 reg = <0x32>;
> >         };
> >
> > -       adc at 34 {
> > -               compatible = "maxim,max11607";
> > -               reg = <0x34>;
> > -               vcc-supply = <&ldo5_reg>;
> > -       };
> > -
> >         eeprom_module: eeprom at 50 {
> >                 compatible = "st,24c02", "atmel,24c02", "i2c-eeprom";
> >                 pagesize = <16>;



-- 
Best regards
Oleksandr Suvorov

Toradex AG
Ebenaustrasse 10 | 6048 Horw | Switzerland | T: +41 41 500 48 00


More information about the U-Boot mailing list