* [PATCH V2] ARM: dts: imx6: correct anatop regulators range
@ 2018-07-02 2:04 Anson Huang
2018-07-02 8:18 ` Lucas Stach
2018-07-03 6:39 ` Shawn Guo
0 siblings, 2 replies; 3+ messages in thread
From: Anson Huang @ 2018-07-02 2:04 UTC (permalink / raw)
To: shawnguo, s.hauer, kernel, fabio.estevam, robh+dt, mark.rutland,
linux-arm-kernel, devicetree, linux-kernel, l.stach
Cc: Linux-imx
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@nxp.com>
---
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>;
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH V2] ARM: dts: imx6: correct anatop regulators range
2018-07-02 2:04 [PATCH V2] ARM: dts: imx6: correct anatop regulators range Anson Huang
@ 2018-07-02 8:18 ` Lucas Stach
2018-07-03 6:39 ` Shawn Guo
1 sibling, 0 replies; 3+ messages in thread
From: Lucas Stach @ 2018-07-02 8:18 UTC (permalink / raw)
To: Anson Huang, shawnguo, s.hauer, kernel, fabio.estevam, robh+dt,
mark.rutland, linux-arm-kernel, devicetree, linux-kernel
Cc: Linux-imx
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@nxp.com>
Reviewed-by: Lucas Stach <l.stach@pengutronix.de>
> ---
> 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>;
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH V2] ARM: dts: imx6: correct anatop regulators range
2018-07-02 2:04 [PATCH V2] ARM: dts: imx6: correct anatop regulators range Anson Huang
2018-07-02 8:18 ` Lucas Stach
@ 2018-07-03 6:39 ` Shawn Guo
1 sibling, 0 replies; 3+ messages in thread
From: Shawn Guo @ 2018-07-03 6:39 UTC (permalink / raw)
To: Anson Huang
Cc: s.hauer, kernel, fabio.estevam, robh+dt, mark.rutland,
linux-arm-kernel, devicetree, linux-kernel, l.stach, Linux-imx
On Mon, Jul 02, 2018 at 10:04:36AM +0800, Anson Huang wrote:
> 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@nxp.com>
Applied, thanks.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-07-03 6:39 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-07-02 2:04 [PATCH V2] ARM: dts: imx6: correct anatop regulators range Anson Huang
2018-07-02 8:18 ` Lucas Stach
2018-07-03 6:39 ` Shawn Guo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).