Am Mittwoch, den 22.03.2017, 18:53 +0200 schrieb Leonard Crestez: > This enables LDO bypass by default on the imx6qdl-sabresd boards. New > dts files with -ldo suffix are added for users who want to run with LDOs > enabled on these boards anyway. Given that using LDO bypass affects the device lifetime negatively (see AN4724), I think the default should still be to use LDO enabled mode and have new DTs for people that desire to shorten the lifetime of the SoC for a minimal drop in power consumption. Regards, Lucas > > Signed-off-by: Leonard Crestez <leonard.crestez@xxxxxxx> > --- > arch/arm/boot/dts/imx6dl-sabresd-ldo.dts | 13 +++++++++++++ > arch/arm/boot/dts/imx6q-sabresd-ldo.dts | 13 +++++++++++++ > arch/arm/boot/dts/imx6qdl-sabresd.dtsi | 19 +++++++++++++++++++ > arch/arm/boot/dts/imx6qdl.dtsi | 6 +++--- > arch/arm/boot/dts/imx6qp-sabresd-ldo.dts | 13 +++++++++++++ > arch/arm/boot/dts/imx6qp-sabresd.dts | 4 ++-- > 6 files changed, 63 insertions(+), 5 deletions(-) > create mode 100644 arch/arm/boot/dts/imx6dl-sabresd-ldo.dts > create mode 100644 arch/arm/boot/dts/imx6q-sabresd-ldo.dts > create mode 100644 arch/arm/boot/dts/imx6qp-sabresd-ldo.dts > > diff --git a/arch/arm/boot/dts/imx6dl-sabresd-ldo.dts b/arch/arm/boot/dts/imx6dl-sabresd-ldo.dts > new file mode 100644 > index 0000000..1b224d6 > --- /dev/null > +++ b/arch/arm/boot/dts/imx6dl-sabresd-ldo.dts > @@ -0,0 +1,13 @@ > +/* > + * Copyright 2017 NXP > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + */ > + > +#include "imx6dl-sabresd.dts" > + > +&gpc { > + fsl,ldo-bypass = <0>; > +}; > diff --git a/arch/arm/boot/dts/imx6q-sabresd-ldo.dts b/arch/arm/boot/dts/imx6q-sabresd-ldo.dts > new file mode 100644 > index 0000000..4d2e8cc > --- /dev/null > +++ b/arch/arm/boot/dts/imx6q-sabresd-ldo.dts > @@ -0,0 +1,13 @@ > +/* > + * Copyright 2017 NXP > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + */ > + > +#include "imx6q-sabresd.dts" > + > +&gpc { > + fsl,ldo-bypass = <0>; > +}; > diff --git a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi > index 63bf95e..5a73d9d 100644 > --- a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi > @@ -182,6 +182,10 @@ > status = "okay"; > }; > > +&gpc { > + fsl,ldo-bypass = <1>; > +}; > + > &hdmi { > ddc-i2c-bus = <&i2c2>; > status = "okay"; > @@ -548,6 +552,21 @@ > status = "okay"; > }; > > +®_arm { > + vin-supply = <&sw1a_reg>; > + regulator-allow-bypass; > +}; > + > +®_pu { > + vin-supply = <&sw1c_reg>; > + regulator-allow-bypass; > +}; > + > +®_soc { > + vin-supply = <&sw1c_reg>; > + regulator-allow-bypass; > +}; > + > &snvs_poweroff { > status = "okay"; > }; > diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi > index 6d7bf64..54fe769 100644 > --- a/arch/arm/boot/dts/imx6qdl.dtsi > +++ b/arch/arm/boot/dts/imx6qdl.dtsi > @@ -677,7 +677,7 @@ > compatible = "fsl,anatop-regulator"; > regulator-name = "vddarm"; > regulator-min-microvolt = <725000>; > - regulator-max-microvolt = <1450000>; > + regulator-max-microvolt = <1300000>; > regulator-always-on; > anatop-reg-offset = <0x140>; > anatop-vol-bit-shift = <0>; > @@ -694,7 +694,7 @@ > compatible = "fsl,anatop-regulator"; > regulator-name = "vddpu"; > regulator-min-microvolt = <725000>; > - regulator-max-microvolt = <1450000>; > + regulator-max-microvolt = <1300000>; > regulator-enable-ramp-delay = <150>; > anatop-reg-offset = <0x140>; > anatop-vol-bit-shift = <9>; > @@ -711,7 +711,7 @@ > compatible = "fsl,anatop-regulator"; > regulator-name = "vddsoc"; > regulator-min-microvolt = <725000>; > - regulator-max-microvolt = <1450000>; > + regulator-max-microvolt = <1300000>; > regulator-always-on; > anatop-reg-offset = <0x140>; > anatop-vol-bit-shift = <18>; > diff --git a/arch/arm/boot/dts/imx6qp-sabresd-ldo.dts b/arch/arm/boot/dts/imx6qp-sabresd-ldo.dts > new file mode 100644 > index 0000000..c659533 > --- /dev/null > +++ b/arch/arm/boot/dts/imx6qp-sabresd-ldo.dts > @@ -0,0 +1,13 @@ > +/* > + * Copyright 2017 NXP > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + */ > + > +#include "imx6qp-sabresd.dts" > + > +&gpc { > + fsl,ldo-bypass = <0>; > +}; > diff --git a/arch/arm/boot/dts/imx6qp-sabresd.dts b/arch/arm/boot/dts/imx6qp-sabresd.dts > index b234580..a8a5004 100644 > --- a/arch/arm/boot/dts/imx6qp-sabresd.dts > +++ b/arch/arm/boot/dts/imx6qp-sabresd.dts > @@ -50,8 +50,8 @@ > compatible = "fsl,imx6qp-sabresd", "fsl,imx6qp"; > }; > > -&cpu0 { > - arm-supply = <&sw2_reg>; > +®_arm { > + vin-supply = <&sw2_reg>; > }; > > &iomuxc { -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html