[PATCH 1/2] ARM: imx6sx-sdb: change default board as reva board

Best regards!
Anson Huang

-----Original Message-----
From: Peter Chen [mailto:peter.chen at freescale.com]
Sent: 2015-01-20 2:30 PM
To: Shawn Guo; Huang Yongcai-B20788
Cc: kernel at pengutronix.de; robh+dt at kernel.org; pawel.moll at arm.com;
mark.rutland at arm.com; linux-arm-kernel at lists.infradead.org;
devicetree at vger.kernel.org
Subject: Re: [PATCH 1/2] ARM: imx6sx-sdb: change default board as reva
board

On Tue, Jan 20, 2015 at 03:35:29PM +0800, Shawn Guo wrote:

quoted

On Wed, Jan 14, 2015 at 03:36:37PM +0800, Peter Chen wrote:

quoted

The imx6sx sdb board has two revisions, the current mainline one is
reva which is mainly for internal use. In this commit, we rename
imx6sx-sdb.dts to imx6sx-sdb.dtsi, and move the reva dedicated
contents to imx6sx-sdb-reva.dts.

Signed-off-by: Peter Chen <redacted>
---
 arch/arm/boot/dts/imx6sx-sdb-reva.dts              | 158

+++++++++++++++++++++

quoted

quoted

 .../boot/dts/{imx6sx-sdb.dts => imx6sx-sdb.dtsi}   | 131 -----------

------

quoted

arch/arm/boot/dts/Makefile needs an update.

Thanks, I will.

quoted

quoted

 2 files changed, 158 insertions(+), 131 deletions(-)  create mode
100644 arch/arm/boot/dts/imx6sx-sdb-reva.dts
 rename arch/arm/boot/dts/{imx6sx-sdb.dts => imx6sx-sdb.dtsi} (83%)

diff --git a/arch/arm/boot/dts/imx6sx-sdb-reva.dts

b/arch/arm/boot/dts/imx6sx-sdb-reva.dts
new file mode 100644
index 0000000..e7a8c34

--- /dev/null
+++ b/arch/arm/boot/dts/imx6sx-sdb-reva.dts

@@ -0,0 +1,158 @@
+/*
+ * Copyright (C) 2014-2015 Freescale Semiconductor, Inc.
+ *
+ * This program is free software; you can redistribute it and/or
+modify
+ * it under the terms of the GNU General Public License version 2
+as
+ * published by the Free Software Foundation.
+ */
+
+#include "imx6sx-sdb.dtsi"
+
+/ {
+	model = "Freescale i.MX6 SoloX SDB RevA Board"; };
+
+&cpu0 {
+	operating-points = <
+		/* kHz    uV */
+		996000  1250000
+		792000  1175000
+		396000  1075000
+	>;
+	fsl,soc-operating-points = <
+		/* ARM kHz  SOC uV */
+		996000      1175000
+		792000      1175000
+		396000      1175000
+	>;

I'm wondering why different revision of the board require a different
OPP settings.

Anson, do you know the reason?

As i.MX6SX SDB Rev-B board uses PFuse 200 which has only one power supply for both
VDDARM_IN and VDDSOC_IN, so it needs VDDARM_IN equal VDDSOC_IN, currently
we set Rev-B board as default in imx6sx.dtsi, which uses same voltage for both
VDDARM_IN and VDDSOC_IN.The Rev-A board uses PFuze 100 which has separate power
supply for VDDARM_IN and VDDSOC_IN, so it can use different voltage on VDDARM_IN
and VDDSOC_IN according to datasheet.

Anson

Peter


--

Best Regards,
Peter Chen