Re: [PATCH] ARM: dts: imx6: correct anatop regulators range

[Lucas Stach]

Am Freitag, den 29.06.2018, 15:54 +0800 schrieb Anson Huang:
> Correct some i.MX6 SoCs anatop regulators range
> according to reference manual.
> 
> Signed-off-by: Anson Huang <Anson.Huang@xxxxxxx>

NACK.

This is wrong. While the regulator supports this range, the SoC
doesn't.ÂThe current settings are derived from the allowed operating
ranges of the SoC (at least for the MX6QDL), which is in accordance
with how regulator constraints in DT are meant to be used.

Regards,
Lucas

> ---
> Âarch/arm/boot/dts/imx6qdl.dtsi | 12 ++++++------
> Âarch/arm/boot/dts/imx6sl.dtsiÂÂ|ÂÂ4 ++--
> Âarch/arm/boot/dts/imx6sx.dtsiÂÂ|ÂÂ4 ++--
> Â3 files changed, 10 insertions(+), 10 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi
> index 911141e..d9f98dd 100644
> --- a/arch/arm/boot/dts/imx6qdl.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl.dtsi
> @@ -689,8 +689,8 @@
> > Â				regulator-1p1 {
> > Â					compatible = "fsl,anatop-regulator";
> > Â					regulator-name = "vdd1p1";
> > -					regulator-min-microvolt = <1000000>;
> > -					regulator-max-microvolt = <1200000>;
> > +					regulator-min-microvolt = <800000>;
> > +					regulator-max-microvolt = <1375000>;
> > Â					regulator-always-on;
> > Â					anatop-reg-offset = <0x110>;
> > Â					anatop-vol-bit-shift = <8>;
> @@ -704,8 +704,8 @@
> > Â				regulator-3p0 {
> > Â					compatible = "fsl,anatop-regulator";
> > Â					regulator-name = "vdd3p0";
> > -					regulator-min-microvolt = <2800000>;
> > -					regulator-max-microvolt = <3150000>;
> > +					regulator-min-microvolt = <2625000>;
> > +					regulator-max-microvolt = <3400000>;
> > Â					regulator-always-on;
> > Â					anatop-reg-offset = <0x120>;
> > Â					anatop-vol-bit-shift = <8>;
> @@ -719,8 +719,8 @@
> > Â				regulator-2p5 {
> > Â					compatible = "fsl,anatop-regulator";
> > Â					regulator-name = "vdd2p5";
> > -					regulator-min-microvolt = <2250000>;
> > -					regulator-max-microvolt = <2750000>;
> > +					regulator-min-microvolt = <2100000>;
> > +					regulator-max-microvolt = <2875000>;
> > Â					regulator-always-on;
> > Â					anatop-reg-offset = <0x130>;
> > Â					anatop-vol-bit-shift = <8>;
> diff --git a/arch/arm/boot/dts/imx6sl.dtsi b/arch/arm/boot/dts/imx6sl.dtsi
> index 81f48116..3d7931b 100644
> --- a/arch/arm/boot/dts/imx6sl.dtsi
> +++ b/arch/arm/boot/dts/imx6sl.dtsi
> @@ -539,8 +539,8 @@
> > Â				regulator-3p0 {
> > Â					compatible = "fsl,anatop-regulator";
> > Â					regulator-name = "vdd3p0";
> > -					regulator-min-microvolt = <2800000>;
> > -					regulator-max-microvolt = <3150000>;
> > +					regulator-min-microvolt = <2625000>;
> > +					regulator-max-microvolt = <3400000>;
> > Â					regulator-always-on;
> > Â					anatop-reg-offset = <0x120>;
> > Â					anatop-vol-bit-shift = <8>;
> diff --git a/arch/arm/boot/dts/imx6sx.dtsi b/arch/arm/boot/dts/imx6sx.dtsi
> index 7130ab8..f26f611 100644
> --- a/arch/arm/boot/dts/imx6sx.dtsi
> +++ b/arch/arm/boot/dts/imx6sx.dtsi
> @@ -607,8 +607,8 @@
> > Â				regulator-3p0 {
> > Â					compatible = "fsl,anatop-regulator";
> > Â					regulator-name = "vdd3p0";
> > -					regulator-min-microvolt = <2800000>;
> > -					regulator-max-microvolt = <3150000>;
> > +					regulator-min-microvolt = <2625000>;
> > +					regulator-max-microvolt = <3400000>;
> > Â					regulator-always-on;
> > Â					anatop-reg-offset = <0x120>;
> > Â					anatop-vol-bit-shift = <8>;