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

[Lucas Stach]

Am Montag, den 02.07.2018, 10:04 +0800 schrieb Anson Huang:
> According to i.MX6 datasheet, the LDO_1P1's typical
> programming operating range is 1.0V to 1.2V, and
> the LDO_2P5's typical programming operating range
> is 2.25V to 2.75V, correct LDO_1P1 and LDO_2P5's
> regulator range settings for i.MX6 SoCs.
> 
> Signed-off-by: Anson Huang <Anson.Huang@xxxxxxx>

Reviewed-by: Lucas Stach <l.stach@xxxxxxxxxxxxxx>

> ---
> changes since V1:
> 	Correct the regulator range according to datasheet's statement.
> Âarch/arm/boot/dts/imx6sl.dtsi | 8 ++++----
> Âarch/arm/boot/dts/imx6sx.dtsi | 8 ++++----
> Â2 files changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/imx6sl.dtsi b/arch/arm/boot/dts/imx6sl.dtsi
> index 81f48116..c64bd90 100644
> --- a/arch/arm/boot/dts/imx6sl.dtsi
> +++ b/arch/arm/boot/dts/imx6sl.dtsi
> @@ -524,8 +524,8 @@
> > Â				regulator-1p1 {
> > Â					compatible = "fsl,anatop-regulator";
> > Â					regulator-name = "vdd1p1";
> > -					regulator-min-microvolt = <800000>;
> > -					regulator-max-microvolt = <1375000>;
> > +					regulator-min-microvolt = <1000000>;
> > +					regulator-max-microvolt = <1200000>;
> > Â					regulator-always-on;
> > Â					anatop-reg-offset = <0x110>;
> > Â					anatop-vol-bit-shift = <8>;
> @@ -554,8 +554,8 @@
> > Â				regulator-2p5 {
> > Â					compatible = "fsl,anatop-regulator";
> > Â					regulator-name = "vdd2p5";
> > -					regulator-min-microvolt = <2100000>;
> > -					regulator-max-microvolt = <2850000>;
> > +					regulator-min-microvolt = <2250000>;
> > +					regulator-max-microvolt = <2750000>;
> > Â					regulator-always-on;
> > Â					anatop-reg-offset = <0x130>;
> > Â					anatop-vol-bit-shift = <8>;
> diff --git a/arch/arm/boot/dts/imx6sx.dtsi b/arch/arm/boot/dts/imx6sx.dtsi
> index 7130ab8..596763c 100644
> --- a/arch/arm/boot/dts/imx6sx.dtsi
> +++ b/arch/arm/boot/dts/imx6sx.dtsi
> @@ -592,8 +592,8 @@
> > Â				regulator-1p1 {
> > Â					compatible = "fsl,anatop-regulator";
> > Â					regulator-name = "vdd1p1";
> > -					regulator-min-microvolt = <800000>;
> > -					regulator-max-microvolt = <1375000>;
> > +					regulator-min-microvolt = <1000000>;
> > +					regulator-max-microvolt = <1200000>;
> > Â					regulator-always-on;
> > Â					anatop-reg-offset = <0x110>;
> > Â					anatop-vol-bit-shift = <8>;
> @@ -622,8 +622,8 @@
> > Â				regulator-2p5 {
> > Â					compatible = "fsl,anatop-regulator";
> > Â					regulator-name = "vdd2p5";
> > -					regulator-min-microvolt = <2100000>;
> > -					regulator-max-microvolt = <2875000>;
> > +					regulator-min-microvolt = <2250000>;
> > +					regulator-max-microvolt = <2750000>;
> > Â					regulator-always-on;
> > Â					anatop-reg-offset = <0x130>;
> > Â					anatop-vol-bit-shift = <8>;