diff mbox

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

Message ID 20170628174518.5950-1-mindentropy@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

mindentropy June 28, 2017, 5:45 p.m. UTC 
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(-)

Comments

Fabio Estevam June 28, 2017, 8:19 p.m. UTC | #1 
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
diff mbox

Patch

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;
 			};
 		};