Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1676921 > unrolled thread

[PATCH] Modified imx7d-sdb.dts file so that VLDO4 provides 2.8V for MIPI CSI/DSI

Started bymindentropy <mindentropy@gmail.com>
First post2017-06-28 20:00 +0200
Last post2017-07-01 14:50 +0200
Articles 4 — 2 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH] Modified imx7d-sdb.dts file so that VLDO4 provides 2.8V for MIPI CSI/DSI mindentropy <mindentropy@gmail.com> - 2017-06-28 20:00 +0200
    Re: [PATCH] Modified imx7d-sdb.dts file so that VLDO4 provides 2.8V  for MIPI CSI/DSI Fabio Estevam <festevam@gmail.com> - 2017-06-28 22:30 +0200
      [PATCH v2] ARM: dts: imx7d-sdb: Set VLDO4 outpt to 2.8V for MIPI CSI/DSI mindentropy <mindentropy@gmail.com> - 2017-07-01 09:50 +0200
        Re: [PATCH v2] ARM: dts: imx7d-sdb: Set VLDO4 outpt to 2.8V for MIPI CSI/DSI Fabio Estevam <festevam@gmail.com> - 2017-07-01 14:50 +0200

#1676921 — [PATCH] Modified imx7d-sdb.dts file so that VLDO4 provides 2.8V for MIPI CSI/DSI

Frommindentropy <mindentropy@gmail.com>
Date2017-06-28 20:00 +0200
Subject[PATCH] Modified imx7d-sdb.dts file so that VLDO4 provides 2.8V for MIPI CSI/DSI
Message-ID<tXppX-t5-59@gated-at.bofh.it>
From: Gautam Bhat <mindentropy@gmail.com>

 * According to mx7sabresd schematics the MIPI CSI/DSI voltage rails
   should be 2.8V but the voltage provided is a maximum of 3.3V and
   minimum of 1.8V. Providing such a higher voltage might damage the
   peripherals and violates the schematic settings.

Signed-off-by: Gautam Bhat <mindentropy@gmail.com>
---
 arch/arm/boot/dts/imx7d-sdb.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/imx7d-sdb.dts b/arch/arm/boot/dts/imx7d-sdb.dts
index 5be01a1bf840..1da1f97ec342 100644
--- a/arch/arm/boot/dts/imx7d-sdb.dts
+++ b/arch/arm/boot/dts/imx7d-sdb.dts
@@ -262,8 +262,8 @@
 			};
 
 			vgen6_reg: vldo4 {
-				regulator-min-microvolt = <1800000>;
-				regulator-max-microvolt = <3300000>;
+				regulator-min-microvolt = <2800000>;
+				regulator-max-microvolt = <2800000>;
 				regulator-always-on;
 			};
 		};
-- 
2.11.0

[toc] | [next] | [standalone]

#1677141 — Re: [PATCH] Modified imx7d-sdb.dts file so that VLDO4 provides 2.8V for MIPI CSI/DSI

FromFabio Estevam <festevam@gmail.com>
Date2017-06-28 22:30 +0200
SubjectRe: [PATCH] Modified imx7d-sdb.dts file so that VLDO4 provides 2.8V for MIPI CSI/DSI
Message-ID<tXrL4-26y-19@gated-at.bofh.it>
In reply to#1676921
Hi Gautam,

On Wed, Jun 28, 2017 at 2:45 PM, mindentropy <mindentropy@gmail.com> wrote:
> From: Gautam Bhat <mindentropy@gmail.com>
>
>  * According to mx7sabresd schematics the MIPI CSI/DSI voltage rails
>    should be 2.8V but the voltage provided is a maximum of 3.3V and
>    minimum of 1.8V. Providing such a higher voltage might damage the
>    peripherals and violates the schematic settings.
>
> Signed-off-by: Gautam Bhat <mindentropy@gmail.com>

Patch looks good, but the Subject could be improved to something like:

ARM: dts: imx7d-sdb: Set LDO4 to 2.8V

[toc] | [prev] | [next] | [standalone]

#1679278 — [PATCH v2] ARM: dts: imx7d-sdb: Set VLDO4 outpt to 2.8V for MIPI CSI/DSI

Frommindentropy <mindentropy@gmail.com>
Date2017-07-01 09:50 +0200
Subject[PATCH v2] ARM: dts: imx7d-sdb: Set VLDO4 outpt to 2.8V for MIPI CSI/DSI
Message-ID<tYlkd-c6-3@gated-at.bofh.it>
In reply to#1677141
From: Gautam Bhat <mindentropy@gmail.com>

 * According to mx7sabresd schematics the MIPI CSI/DSI voltage rails
   should be 2.8V but the voltage provided is a maximum of 3.3V and
   minimum of 1.8V. Providing such a higher voltage might damage the
   peripherals and violates the schematic settings.

Signed-off-by: Gautam Bhat <mindentropy@gmail.com>
---
Changes in v2:
 - Added subject line according to convention

 arch/arm/boot/dts/imx7d-sdb.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/imx7d-sdb.dts b/arch/arm/boot/dts/imx7d-sdb.dts
index 5be01a1bf840..1da1f97ec342 100644
--- a/arch/arm/boot/dts/imx7d-sdb.dts
+++ b/arch/arm/boot/dts/imx7d-sdb.dts
@@ -262,8 +262,8 @@
 			};
 
 			vgen6_reg: vldo4 {
-				regulator-min-microvolt = <1800000>;
-				regulator-max-microvolt = <3300000>;
+				regulator-min-microvolt = <2800000>;
+				regulator-max-microvolt = <2800000>;
 				regulator-always-on;
 			};
 		};
-- 
2.11.0

[toc] | [prev] | [next] | [standalone]

#1679349 — Re: [PATCH v2] ARM: dts: imx7d-sdb: Set VLDO4 outpt to 2.8V for MIPI CSI/DSI

FromFabio Estevam <festevam@gmail.com>
Date2017-07-01 14:50 +0200
SubjectRe: [PATCH v2] ARM: dts: imx7d-sdb: Set VLDO4 outpt to 2.8V for MIPI CSI/DSI
Message-ID<tYq0x-3fA-1@gated-at.bofh.it>
In reply to#1679278
On Sat, Jul 1, 2017 at 4:39 AM, mindentropy <mindentropy@gmail.com> wrote:
> From: Gautam Bhat <mindentropy@gmail.com>
>
>  * According to mx7sabresd schematics the MIPI CSI/DSI voltage rails
>    should be 2.8V but the voltage provided is a maximum of 3.3V and
>    minimum of 1.8V. Providing such a higher voltage might damage the
>    peripherals and violates the schematic settings.
>
> Signed-off-by: Gautam Bhat <mindentropy@gmail.com>

Reviewed-by: Fabio Estevam <fabio.estevam@nxp.com>

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web