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:

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 +++++++++++++++++++++
 .../boot/dts/{imx6sx-sdb.dts => imx6sx-sdb.dtsi}   | 131 -----------------

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

Thanks, I will.

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?

Peter


-- 

Best Regards,
Peter Chen