* [PATCH 0/2] ARM: imx6sx-sdb: add revb board
@ 2015-01-14 7:36 ` Peter Chen
0 siblings, 0 replies; 14+ messages in thread
From: Peter Chen @ 2015-01-14 7:36 UTC (permalink / raw)
To: shawn.guo, kernel, robh+dt, pawel.moll, mark.rutland
Cc: devicetree, Peter Chen, linux-arm-kernel
Hi Shawn,
The imx6sx-sdb board has two revisions, the current mainline is the
reva which is used by internal user mainly, and the revb is used widely
for internal and external users.
The main changes between reva and revb board:
- cpu operating point and related voltage
- pfuse
- qspi
- can transceiver (not mainlined)
Peter Chen (2):
ARM: imx6sx-sdb: change default board as reva board
ARM: imx6sx-sdb: add revb board
arch/arm/boot/dts/imx6sx-sdb-reva.dts | 158 +++++++++++++++++++++
arch/arm/boot/dts/imx6sx-sdb-revb.dts | 145 +++++++++++++++++++
.../boot/dts/{imx6sx-sdb.dts => imx6sx-sdb.dtsi} | 131 -----------------
3 files changed, 303 insertions(+), 131 deletions(-)
create mode 100644 arch/arm/boot/dts/imx6sx-sdb-reva.dts
create mode 100644 arch/arm/boot/dts/imx6sx-sdb-revb.dts
rename arch/arm/boot/dts/{imx6sx-sdb.dts => imx6sx-sdb.dtsi} (83%)
--
1.9.1
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 0/2] ARM: imx6sx-sdb: add revb board
@ 2015-01-14 7:36 ` Peter Chen
0 siblings, 0 replies; 14+ messages in thread
From: Peter Chen @ 2015-01-14 7:36 UTC (permalink / raw)
To: linux-arm-kernel
Hi Shawn,
The imx6sx-sdb board has two revisions, the current mainline is the
reva which is used by internal user mainly, and the revb is used widely
for internal and external users.
The main changes between reva and revb board:
- cpu operating point and related voltage
- pfuse
- qspi
- can transceiver (not mainlined)
Peter Chen (2):
ARM: imx6sx-sdb: change default board as reva board
ARM: imx6sx-sdb: add revb board
arch/arm/boot/dts/imx6sx-sdb-reva.dts | 158 +++++++++++++++++++++
arch/arm/boot/dts/imx6sx-sdb-revb.dts | 145 +++++++++++++++++++
.../boot/dts/{imx6sx-sdb.dts => imx6sx-sdb.dtsi} | 131 -----------------
3 files changed, 303 insertions(+), 131 deletions(-)
create mode 100644 arch/arm/boot/dts/imx6sx-sdb-reva.dts
create mode 100644 arch/arm/boot/dts/imx6sx-sdb-revb.dts
rename arch/arm/boot/dts/{imx6sx-sdb.dts => imx6sx-sdb.dtsi} (83%)
--
1.9.1
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 1/2] ARM: imx6sx-sdb: change default board as reva board
2015-01-14 7:36 ` Peter Chen
@ 2015-01-14 7:36 ` Peter Chen
-1 siblings, 0 replies; 14+ messages in thread
From: Peter Chen @ 2015-01-14 7:36 UTC (permalink / raw)
To: shawn.guo-QSEj5FYQhm4dnm+yROfE0A, kernel-bIcnvbaLZ9MEGnE8C9+IrQ,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
mark.rutland-5wv7dgnIgG8
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
devicetree-u79uwXL29TY76Z2rM5mHXA, Peter Chen
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 <peter.chen-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
---
arch/arm/boot/dts/imx6sx-sdb-reva.dts | 158 +++++++++++++++++++++
.../boot/dts/{imx6sx-sdb.dts => imx6sx-sdb.dtsi} | 131 -----------------
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
+ >;
+};
+
+&i2c1 {
+ clock-frequency = <100000>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_i2c1>;
+ status = "okay";
+
+ pmic: pfuze100@08 {
+ compatible = "fsl,pfuze100";
+ reg = <0x08>;
+
+ regulators {
+ sw1a_reg: sw1ab {
+ regulator-min-microvolt = <300000>;
+ regulator-max-microvolt = <1875000>;
+ regulator-boot-on;
+ regulator-always-on;
+ regulator-ramp-delay = <6250>;
+ };
+
+ sw1c_reg: sw1c {
+ regulator-min-microvolt = <300000>;
+ regulator-max-microvolt = <1875000>;
+ regulator-boot-on;
+ regulator-always-on;
+ regulator-ramp-delay = <6250>;
+ };
+
+ sw2_reg: sw2 {
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ sw3a_reg: sw3a {
+ regulator-min-microvolt = <400000>;
+ regulator-max-microvolt = <1975000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ sw3b_reg: sw3b {
+ regulator-min-microvolt = <400000>;
+ regulator-max-microvolt = <1975000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ sw4_reg: sw4 {
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <3300000>;
+ };
+
+ swbst_reg: swbst {
+ regulator-min-microvolt = <5000000>;
+ regulator-max-microvolt = <5150000>;
+ };
+
+ snvs_reg: vsnvs {
+ regulator-min-microvolt = <1000000>;
+ regulator-max-microvolt = <3000000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ vref_reg: vrefddr {
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ vgen1_reg: vgen1 {
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <1550000>;
+ regulator-always-on;
+ };
+
+ vgen2_reg: vgen2 {
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <1550000>;
+ };
+
+ vgen3_reg: vgen3 {
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ vgen4_reg: vgen4 {
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ vgen5_reg: vgen5 {
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ vgen6_reg: vgen6 {
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+ };
+ };
+};
+
+&qspi2 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_qspi2>;
+ status = "okay";
+
+ flash0: s25fl128s@0 {
+ reg = <0>;
+ #address-cells = <1>;
+ #size-cells = <1>;
+ compatible = "spansion,s25fl128s";
+ spi-max-frequency = <66000000>;
+ };
+
+ flash1: s25fl128s@1 {
+ reg = <1>;
+ #address-cells = <1>;
+ #size-cells = <1>;
+ compatible = "spansion,s25fl128s";
+ spi-max-frequency = <66000000>;
+ };
+};
diff --git a/arch/arm/boot/dts/imx6sx-sdb.dts b/arch/arm/boot/dts/imx6sx-sdb.dtsi
similarity index 83%
rename from arch/arm/boot/dts/imx6sx-sdb.dts
rename to arch/arm/boot/dts/imx6sx-sdb.dtsi
index cdffe84..7b02aa7 100644
--- a/arch/arm/boot/dts/imx6sx-sdb.dts
+++ b/arch/arm/boot/dts/imx6sx-sdb.dtsi
@@ -13,7 +13,6 @@
#include "imx6sx.dtsi"
/ {
- model = "Freescale i.MX6 SoloX SDB Board";
compatible = "fsl,imx6sx-sdb", "fsl,imx6sx";
chosen {
@@ -169,114 +168,6 @@
status = "okay";
};
-&i2c1 {
- clock-frequency = <100000>;
- pinctrl-names = "default";
- pinctrl-0 = <&pinctrl_i2c1>;
- status = "okay";
-
- pmic: pfuze100@08 {
- compatible = "fsl,pfuze100";
- reg = <0x08>;
-
- regulators {
- sw1a_reg: sw1ab {
- regulator-min-microvolt = <300000>;
- regulator-max-microvolt = <1875000>;
- regulator-boot-on;
- regulator-always-on;
- regulator-ramp-delay = <6250>;
- };
-
- sw1c_reg: sw1c {
- regulator-min-microvolt = <300000>;
- regulator-max-microvolt = <1875000>;
- regulator-boot-on;
- regulator-always-on;
- regulator-ramp-delay = <6250>;
- };
-
- sw2_reg: sw2 {
- regulator-min-microvolt = <800000>;
- regulator-max-microvolt = <3300000>;
- regulator-boot-on;
- regulator-always-on;
- };
-
- sw3a_reg: sw3a {
- regulator-min-microvolt = <400000>;
- regulator-max-microvolt = <1975000>;
- regulator-boot-on;
- regulator-always-on;
- };
-
- sw3b_reg: sw3b {
- regulator-min-microvolt = <400000>;
- regulator-max-microvolt = <1975000>;
- regulator-boot-on;
- regulator-always-on;
- };
-
- sw4_reg: sw4 {
- regulator-min-microvolt = <800000>;
- regulator-max-microvolt = <3300000>;
- };
-
- swbst_reg: swbst {
- regulator-min-microvolt = <5000000>;
- regulator-max-microvolt = <5150000>;
- };
-
- snvs_reg: vsnvs {
- regulator-min-microvolt = <1000000>;
- regulator-max-microvolt = <3000000>;
- regulator-boot-on;
- regulator-always-on;
- };
-
- vref_reg: vrefddr {
- regulator-boot-on;
- regulator-always-on;
- };
-
- vgen1_reg: vgen1 {
- regulator-min-microvolt = <800000>;
- regulator-max-microvolt = <1550000>;
- regulator-always-on;
- };
-
- vgen2_reg: vgen2 {
- regulator-min-microvolt = <800000>;
- regulator-max-microvolt = <1550000>;
- };
-
- vgen3_reg: vgen3 {
- regulator-min-microvolt = <1800000>;
- regulator-max-microvolt = <3300000>;
- regulator-always-on;
- };
-
- vgen4_reg: vgen4 {
- regulator-min-microvolt = <1800000>;
- regulator-max-microvolt = <3300000>;
- regulator-always-on;
- };
-
- vgen5_reg: vgen5 {
- regulator-min-microvolt = <1800000>;
- regulator-max-microvolt = <3300000>;
- regulator-always-on;
- };
-
- vgen6_reg: vgen6 {
- regulator-min-microvolt = <1800000>;
- regulator-max-microvolt = <3300000>;
- regulator-always-on;
- };
- };
- };
-};
-
&i2c4 {
clock-frequency = <100000>;
pinctrl-names = "default";
@@ -340,28 +231,6 @@
status = "okay";
};
-&qspi2 {
- pinctrl-names = "default";
- pinctrl-0 = <&pinctrl_qspi2>;
- status = "okay";
-
- flash0: s25fl128s@0 {
- reg = <0>;
- #address-cells = <1>;
- #size-cells = <1>;
- compatible = "spansion,s25fl128s";
- spi-max-frequency = <66000000>;
- };
-
- flash1: s25fl128s@1 {
- reg = <1>;
- #address-cells = <1>;
- #size-cells = <1>;
- compatible = "spansion,s25fl128s";
- spi-max-frequency = <66000000>;
- };
-};
-
&ssi2 {
status = "okay";
};
--
1.9.1
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 1/2] ARM: imx6sx-sdb: change default board as reva board
@ 2015-01-14 7:36 ` Peter Chen
0 siblings, 0 replies; 14+ messages in thread
From: Peter Chen @ 2015-01-14 7:36 UTC (permalink / raw)
To: linux-arm-kernel
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 <peter.chen@freescale.com>
---
arch/arm/boot/dts/imx6sx-sdb-reva.dts | 158 +++++++++++++++++++++
.../boot/dts/{imx6sx-sdb.dts => imx6sx-sdb.dtsi} | 131 -----------------
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
+ >;
+};
+
+&i2c1 {
+ clock-frequency = <100000>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_i2c1>;
+ status = "okay";
+
+ pmic: pfuze100 at 08 {
+ compatible = "fsl,pfuze100";
+ reg = <0x08>;
+
+ regulators {
+ sw1a_reg: sw1ab {
+ regulator-min-microvolt = <300000>;
+ regulator-max-microvolt = <1875000>;
+ regulator-boot-on;
+ regulator-always-on;
+ regulator-ramp-delay = <6250>;
+ };
+
+ sw1c_reg: sw1c {
+ regulator-min-microvolt = <300000>;
+ regulator-max-microvolt = <1875000>;
+ regulator-boot-on;
+ regulator-always-on;
+ regulator-ramp-delay = <6250>;
+ };
+
+ sw2_reg: sw2 {
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ sw3a_reg: sw3a {
+ regulator-min-microvolt = <400000>;
+ regulator-max-microvolt = <1975000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ sw3b_reg: sw3b {
+ regulator-min-microvolt = <400000>;
+ regulator-max-microvolt = <1975000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ sw4_reg: sw4 {
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <3300000>;
+ };
+
+ swbst_reg: swbst {
+ regulator-min-microvolt = <5000000>;
+ regulator-max-microvolt = <5150000>;
+ };
+
+ snvs_reg: vsnvs {
+ regulator-min-microvolt = <1000000>;
+ regulator-max-microvolt = <3000000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ vref_reg: vrefddr {
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ vgen1_reg: vgen1 {
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <1550000>;
+ regulator-always-on;
+ };
+
+ vgen2_reg: vgen2 {
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <1550000>;
+ };
+
+ vgen3_reg: vgen3 {
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ vgen4_reg: vgen4 {
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ vgen5_reg: vgen5 {
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ vgen6_reg: vgen6 {
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+ };
+ };
+};
+
+&qspi2 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_qspi2>;
+ status = "okay";
+
+ flash0: s25fl128s at 0 {
+ reg = <0>;
+ #address-cells = <1>;
+ #size-cells = <1>;
+ compatible = "spansion,s25fl128s";
+ spi-max-frequency = <66000000>;
+ };
+
+ flash1: s25fl128s at 1 {
+ reg = <1>;
+ #address-cells = <1>;
+ #size-cells = <1>;
+ compatible = "spansion,s25fl128s";
+ spi-max-frequency = <66000000>;
+ };
+};
diff --git a/arch/arm/boot/dts/imx6sx-sdb.dts b/arch/arm/boot/dts/imx6sx-sdb.dtsi
similarity index 83%
rename from arch/arm/boot/dts/imx6sx-sdb.dts
rename to arch/arm/boot/dts/imx6sx-sdb.dtsi
index cdffe84..7b02aa7 100644
--- a/arch/arm/boot/dts/imx6sx-sdb.dts
+++ b/arch/arm/boot/dts/imx6sx-sdb.dtsi
@@ -13,7 +13,6 @@
#include "imx6sx.dtsi"
/ {
- model = "Freescale i.MX6 SoloX SDB Board";
compatible = "fsl,imx6sx-sdb", "fsl,imx6sx";
chosen {
@@ -169,114 +168,6 @@
status = "okay";
};
-&i2c1 {
- clock-frequency = <100000>;
- pinctrl-names = "default";
- pinctrl-0 = <&pinctrl_i2c1>;
- status = "okay";
-
- pmic: pfuze100 at 08 {
- compatible = "fsl,pfuze100";
- reg = <0x08>;
-
- regulators {
- sw1a_reg: sw1ab {
- regulator-min-microvolt = <300000>;
- regulator-max-microvolt = <1875000>;
- regulator-boot-on;
- regulator-always-on;
- regulator-ramp-delay = <6250>;
- };
-
- sw1c_reg: sw1c {
- regulator-min-microvolt = <300000>;
- regulator-max-microvolt = <1875000>;
- regulator-boot-on;
- regulator-always-on;
- regulator-ramp-delay = <6250>;
- };
-
- sw2_reg: sw2 {
- regulator-min-microvolt = <800000>;
- regulator-max-microvolt = <3300000>;
- regulator-boot-on;
- regulator-always-on;
- };
-
- sw3a_reg: sw3a {
- regulator-min-microvolt = <400000>;
- regulator-max-microvolt = <1975000>;
- regulator-boot-on;
- regulator-always-on;
- };
-
- sw3b_reg: sw3b {
- regulator-min-microvolt = <400000>;
- regulator-max-microvolt = <1975000>;
- regulator-boot-on;
- regulator-always-on;
- };
-
- sw4_reg: sw4 {
- regulator-min-microvolt = <800000>;
- regulator-max-microvolt = <3300000>;
- };
-
- swbst_reg: swbst {
- regulator-min-microvolt = <5000000>;
- regulator-max-microvolt = <5150000>;
- };
-
- snvs_reg: vsnvs {
- regulator-min-microvolt = <1000000>;
- regulator-max-microvolt = <3000000>;
- regulator-boot-on;
- regulator-always-on;
- };
-
- vref_reg: vrefddr {
- regulator-boot-on;
- regulator-always-on;
- };
-
- vgen1_reg: vgen1 {
- regulator-min-microvolt = <800000>;
- regulator-max-microvolt = <1550000>;
- regulator-always-on;
- };
-
- vgen2_reg: vgen2 {
- regulator-min-microvolt = <800000>;
- regulator-max-microvolt = <1550000>;
- };
-
- vgen3_reg: vgen3 {
- regulator-min-microvolt = <1800000>;
- regulator-max-microvolt = <3300000>;
- regulator-always-on;
- };
-
- vgen4_reg: vgen4 {
- regulator-min-microvolt = <1800000>;
- regulator-max-microvolt = <3300000>;
- regulator-always-on;
- };
-
- vgen5_reg: vgen5 {
- regulator-min-microvolt = <1800000>;
- regulator-max-microvolt = <3300000>;
- regulator-always-on;
- };
-
- vgen6_reg: vgen6 {
- regulator-min-microvolt = <1800000>;
- regulator-max-microvolt = <3300000>;
- regulator-always-on;
- };
- };
- };
-};
-
&i2c4 {
clock-frequency = <100000>;
pinctrl-names = "default";
@@ -340,28 +231,6 @@
status = "okay";
};
-&qspi2 {
- pinctrl-names = "default";
- pinctrl-0 = <&pinctrl_qspi2>;
- status = "okay";
-
- flash0: s25fl128s at 0 {
- reg = <0>;
- #address-cells = <1>;
- #size-cells = <1>;
- compatible = "spansion,s25fl128s";
- spi-max-frequency = <66000000>;
- };
-
- flash1: s25fl128s at 1 {
- reg = <1>;
- #address-cells = <1>;
- #size-cells = <1>;
- compatible = "spansion,s25fl128s";
- spi-max-frequency = <66000000>;
- };
-};
-
&ssi2 {
status = "okay";
};
--
1.9.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 2/2] ARM: imx6sx-sdb: add revb board
2015-01-14 7:36 ` Peter Chen
@ 2015-01-14 7:36 ` Peter Chen
-1 siblings, 0 replies; 14+ messages in thread
From: Peter Chen @ 2015-01-14 7:36 UTC (permalink / raw)
To: shawn.guo, kernel, robh+dt, pawel.moll, mark.rutland
Cc: devicetree, Peter Chen, linux-arm-kernel
Add imx6sx-sdb revb board
Signed-off-by: Peter Chen <peter.chen@freescale.com>
---
arch/arm/boot/dts/imx6sx-sdb-revb.dts | 145 ++++++++++++++++++++++++++++++++++
1 file changed, 145 insertions(+)
create mode 100644 arch/arm/boot/dts/imx6sx-sdb-revb.dts
diff --git a/arch/arm/boot/dts/imx6sx-sdb-revb.dts b/arch/arm/boot/dts/imx6sx-sdb-revb.dts
new file mode 100644
index 0000000..d68dfd4
--- /dev/null
+++ b/arch/arm/boot/dts/imx6sx-sdb-revb.dts
@@ -0,0 +1,145 @@
+/*
+ * 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 RevB Board";
+};
+
+&cpu0 {
+ operating-points = <
+ /* kHz uV */
+ 996000 1250000
+ 792000 1175000
+ 396000 1175000
+ >;
+ fsl,soc-operating-points = <
+ /* ARM kHz SOC uV */
+ 996000 1250000
+ 792000 1175000
+ 396000 1175000
+ >;
+};
+
+&i2c1 {
+ clock-frequency = <100000>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_i2c1>;
+ status = "okay";
+
+ pmic: pfuze100@08 {
+ compatible = "fsl,pfuze200";
+ reg = <0x08>;
+
+ regulators {
+ sw1a_reg: sw1ab {
+ regulator-min-microvolt = <300000>;
+ regulator-max-microvolt = <1875000>;
+ regulator-boot-on;
+ regulator-always-on;
+ regulator-ramp-delay = <6250>;
+ };
+
+ sw2_reg: sw2 {
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ sw3a_reg: sw3a {
+ regulator-min-microvolt = <400000>;
+ regulator-max-microvolt = <1975000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ sw3b_reg: sw3b {
+ regulator-min-microvolt = <400000>;
+ regulator-max-microvolt = <1975000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ swbst_reg: swbst {
+ regulator-min-microvolt = <5000000>;
+ regulator-max-microvolt = <5150000>;
+ };
+
+ snvs_reg: vsnvs {
+ regulator-min-microvolt = <1000000>;
+ regulator-max-microvolt = <3000000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ vref_reg: vrefddr {
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ vgen1_reg: vgen1 {
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <1550000>;
+ regulator-always-on;
+ };
+
+ vgen2_reg: vgen2 {
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <1550000>;
+ };
+
+ vgen3_reg: vgen3 {
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ vgen4_reg: vgen4 {
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ vgen5_reg: vgen5 {
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ vgen6_reg: vgen6 {
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+ };
+ };
+};
+
+&qspi2 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_qspi2>;
+ status = "okay";
+
+ flash0: n25q256a@0 {
+ #address-cells = <1>;
+ #size-cells = <1>;
+ compatible = "micron,n25q256a";
+ spi-max-frequency = <29000000>;
+ reg = <0>;
+ };
+
+ flash1: n25q256a@1 {
+ #address-cells = <1>;
+ #size-cells = <1>;
+ compatible = "micron,n25q256a";
+ spi-max-frequency = <29000000>;
+ reg = <1>;
+ };
+};
--
1.9.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 2/2] ARM: imx6sx-sdb: add revb board
@ 2015-01-14 7:36 ` Peter Chen
0 siblings, 0 replies; 14+ messages in thread
From: Peter Chen @ 2015-01-14 7:36 UTC (permalink / raw)
To: linux-arm-kernel
Add imx6sx-sdb revb board
Signed-off-by: Peter Chen <peter.chen@freescale.com>
---
arch/arm/boot/dts/imx6sx-sdb-revb.dts | 145 ++++++++++++++++++++++++++++++++++
1 file changed, 145 insertions(+)
create mode 100644 arch/arm/boot/dts/imx6sx-sdb-revb.dts
diff --git a/arch/arm/boot/dts/imx6sx-sdb-revb.dts b/arch/arm/boot/dts/imx6sx-sdb-revb.dts
new file mode 100644
index 0000000..d68dfd4
--- /dev/null
+++ b/arch/arm/boot/dts/imx6sx-sdb-revb.dts
@@ -0,0 +1,145 @@
+/*
+ * 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 RevB Board";
+};
+
+&cpu0 {
+ operating-points = <
+ /* kHz uV */
+ 996000 1250000
+ 792000 1175000
+ 396000 1175000
+ >;
+ fsl,soc-operating-points = <
+ /* ARM kHz SOC uV */
+ 996000 1250000
+ 792000 1175000
+ 396000 1175000
+ >;
+};
+
+&i2c1 {
+ clock-frequency = <100000>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_i2c1>;
+ status = "okay";
+
+ pmic: pfuze100 at 08 {
+ compatible = "fsl,pfuze200";
+ reg = <0x08>;
+
+ regulators {
+ sw1a_reg: sw1ab {
+ regulator-min-microvolt = <300000>;
+ regulator-max-microvolt = <1875000>;
+ regulator-boot-on;
+ regulator-always-on;
+ regulator-ramp-delay = <6250>;
+ };
+
+ sw2_reg: sw2 {
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ sw3a_reg: sw3a {
+ regulator-min-microvolt = <400000>;
+ regulator-max-microvolt = <1975000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ sw3b_reg: sw3b {
+ regulator-min-microvolt = <400000>;
+ regulator-max-microvolt = <1975000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ swbst_reg: swbst {
+ regulator-min-microvolt = <5000000>;
+ regulator-max-microvolt = <5150000>;
+ };
+
+ snvs_reg: vsnvs {
+ regulator-min-microvolt = <1000000>;
+ regulator-max-microvolt = <3000000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ vref_reg: vrefddr {
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
+ vgen1_reg: vgen1 {
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <1550000>;
+ regulator-always-on;
+ };
+
+ vgen2_reg: vgen2 {
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <1550000>;
+ };
+
+ vgen3_reg: vgen3 {
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ vgen4_reg: vgen4 {
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ vgen5_reg: vgen5 {
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+
+ vgen6_reg: vgen6 {
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-always-on;
+ };
+ };
+ };
+};
+
+&qspi2 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_qspi2>;
+ status = "okay";
+
+ flash0: n25q256a at 0 {
+ #address-cells = <1>;
+ #size-cells = <1>;
+ compatible = "micron,n25q256a";
+ spi-max-frequency = <29000000>;
+ reg = <0>;
+ };
+
+ flash1: n25q256a at 1 {
+ #address-cells = <1>;
+ #size-cells = <1>;
+ compatible = "micron,n25q256a";
+ spi-max-frequency = <29000000>;
+ reg = <1>;
+ };
+};
--
1.9.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] ARM: imx6sx-sdb: change default board as reva board
2015-01-20 7:35 ` Shawn Guo
@ 2015-01-20 6:30 ` Peter Chen
-1 siblings, 0 replies; 14+ messages in thread
From: Peter Chen @ 2015-01-20 6:30 UTC (permalink / raw)
To: Shawn Guo, b20788
Cc: mark.rutland, devicetree, pawel.moll, robh+dt, kernel, linux-arm-kernel
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:
> > 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 <peter.chen@freescale.com>
> > ---
> > 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.
> > 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
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 1/2] ARM: imx6sx-sdb: change default board as reva board
@ 2015-01-20 6:30 ` Peter Chen
0 siblings, 0 replies; 14+ messages in thread
From: Peter Chen @ 2015-01-20 6:30 UTC (permalink / raw)
To: linux-arm-kernel
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:
> > 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 <peter.chen@freescale.com>
> > ---
> > 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.
> > 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
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] ARM: imx6sx-sdb: change default board as reva board
2015-01-14 7:36 ` Peter Chen
@ 2015-01-20 7:35 ` Shawn Guo
-1 siblings, 0 replies; 14+ messages in thread
From: Shawn Guo @ 2015-01-20 7:35 UTC (permalink / raw)
To: Peter Chen
Cc: kernel-bIcnvbaLZ9MEGnE8C9+IrQ, robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
pawel.moll-5wv7dgnIgG8, mark.rutland-5wv7dgnIgG8,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
devicetree-u79uwXL29TY76Z2rM5mHXA
On Wed, Jan 14, 2015 at 03:36:37PM +0800, Peter Chen wrote:
> 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 <peter.chen-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
> ---
> 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.
> 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.
Shawn
> +};
> +
> +&i2c1 {
> + clock-frequency = <100000>;
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_i2c1>;
> + status = "okay";
> +
> + pmic: pfuze100@08 {
> + compatible = "fsl,pfuze100";
> + reg = <0x08>;
> +
> + regulators {
> + sw1a_reg: sw1ab {
> + regulator-min-microvolt = <300000>;
> + regulator-max-microvolt = <1875000>;
> + regulator-boot-on;
> + regulator-always-on;
> + regulator-ramp-delay = <6250>;
> + };
> +
> + sw1c_reg: sw1c {
> + regulator-min-microvolt = <300000>;
> + regulator-max-microvolt = <1875000>;
> + regulator-boot-on;
> + regulator-always-on;
> + regulator-ramp-delay = <6250>;
> + };
> +
> + sw2_reg: sw2 {
> + regulator-min-microvolt = <800000>;
> + regulator-max-microvolt = <3300000>;
> + regulator-boot-on;
> + regulator-always-on;
> + };
> +
> + sw3a_reg: sw3a {
> + regulator-min-microvolt = <400000>;
> + regulator-max-microvolt = <1975000>;
> + regulator-boot-on;
> + regulator-always-on;
> + };
> +
> + sw3b_reg: sw3b {
> + regulator-min-microvolt = <400000>;
> + regulator-max-microvolt = <1975000>;
> + regulator-boot-on;
> + regulator-always-on;
> + };
> +
> + sw4_reg: sw4 {
> + regulator-min-microvolt = <800000>;
> + regulator-max-microvolt = <3300000>;
> + };
> +
> + swbst_reg: swbst {
> + regulator-min-microvolt = <5000000>;
> + regulator-max-microvolt = <5150000>;
> + };
> +
> + snvs_reg: vsnvs {
> + regulator-min-microvolt = <1000000>;
> + regulator-max-microvolt = <3000000>;
> + regulator-boot-on;
> + regulator-always-on;
> + };
> +
> + vref_reg: vrefddr {
> + regulator-boot-on;
> + regulator-always-on;
> + };
> +
> + vgen1_reg: vgen1 {
> + regulator-min-microvolt = <800000>;
> + regulator-max-microvolt = <1550000>;
> + regulator-always-on;
> + };
> +
> + vgen2_reg: vgen2 {
> + regulator-min-microvolt = <800000>;
> + regulator-max-microvolt = <1550000>;
> + };
> +
> + vgen3_reg: vgen3 {
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <3300000>;
> + regulator-always-on;
> + };
> +
> + vgen4_reg: vgen4 {
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <3300000>;
> + regulator-always-on;
> + };
> +
> + vgen5_reg: vgen5 {
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <3300000>;
> + regulator-always-on;
> + };
> +
> + vgen6_reg: vgen6 {
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <3300000>;
> + regulator-always-on;
> + };
> + };
> + };
> +};
> +
> +&qspi2 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_qspi2>;
> + status = "okay";
> +
> + flash0: s25fl128s@0 {
> + reg = <0>;
> + #address-cells = <1>;
> + #size-cells = <1>;
> + compatible = "spansion,s25fl128s";
> + spi-max-frequency = <66000000>;
> + };
> +
> + flash1: s25fl128s@1 {
> + reg = <1>;
> + #address-cells = <1>;
> + #size-cells = <1>;
> + compatible = "spansion,s25fl128s";
> + spi-max-frequency = <66000000>;
> + };
> +};
> diff --git a/arch/arm/boot/dts/imx6sx-sdb.dts b/arch/arm/boot/dts/imx6sx-sdb.dtsi
> similarity index 83%
> rename from arch/arm/boot/dts/imx6sx-sdb.dts
> rename to arch/arm/boot/dts/imx6sx-sdb.dtsi
> index cdffe84..7b02aa7 100644
> --- a/arch/arm/boot/dts/imx6sx-sdb.dts
> +++ b/arch/arm/boot/dts/imx6sx-sdb.dtsi
> @@ -13,7 +13,6 @@
> #include "imx6sx.dtsi"
>
> / {
> - model = "Freescale i.MX6 SoloX SDB Board";
> compatible = "fsl,imx6sx-sdb", "fsl,imx6sx";
>
> chosen {
> @@ -169,114 +168,6 @@
> status = "okay";
> };
>
> -&i2c1 {
> - clock-frequency = <100000>;
> - pinctrl-names = "default";
> - pinctrl-0 = <&pinctrl_i2c1>;
> - status = "okay";
> -
> - pmic: pfuze100@08 {
> - compatible = "fsl,pfuze100";
> - reg = <0x08>;
> -
> - regulators {
> - sw1a_reg: sw1ab {
> - regulator-min-microvolt = <300000>;
> - regulator-max-microvolt = <1875000>;
> - regulator-boot-on;
> - regulator-always-on;
> - regulator-ramp-delay = <6250>;
> - };
> -
> - sw1c_reg: sw1c {
> - regulator-min-microvolt = <300000>;
> - regulator-max-microvolt = <1875000>;
> - regulator-boot-on;
> - regulator-always-on;
> - regulator-ramp-delay = <6250>;
> - };
> -
> - sw2_reg: sw2 {
> - regulator-min-microvolt = <800000>;
> - regulator-max-microvolt = <3300000>;
> - regulator-boot-on;
> - regulator-always-on;
> - };
> -
> - sw3a_reg: sw3a {
> - regulator-min-microvolt = <400000>;
> - regulator-max-microvolt = <1975000>;
> - regulator-boot-on;
> - regulator-always-on;
> - };
> -
> - sw3b_reg: sw3b {
> - regulator-min-microvolt = <400000>;
> - regulator-max-microvolt = <1975000>;
> - regulator-boot-on;
> - regulator-always-on;
> - };
> -
> - sw4_reg: sw4 {
> - regulator-min-microvolt = <800000>;
> - regulator-max-microvolt = <3300000>;
> - };
> -
> - swbst_reg: swbst {
> - regulator-min-microvolt = <5000000>;
> - regulator-max-microvolt = <5150000>;
> - };
> -
> - snvs_reg: vsnvs {
> - regulator-min-microvolt = <1000000>;
> - regulator-max-microvolt = <3000000>;
> - regulator-boot-on;
> - regulator-always-on;
> - };
> -
> - vref_reg: vrefddr {
> - regulator-boot-on;
> - regulator-always-on;
> - };
> -
> - vgen1_reg: vgen1 {
> - regulator-min-microvolt = <800000>;
> - regulator-max-microvolt = <1550000>;
> - regulator-always-on;
> - };
> -
> - vgen2_reg: vgen2 {
> - regulator-min-microvolt = <800000>;
> - regulator-max-microvolt = <1550000>;
> - };
> -
> - vgen3_reg: vgen3 {
> - regulator-min-microvolt = <1800000>;
> - regulator-max-microvolt = <3300000>;
> - regulator-always-on;
> - };
> -
> - vgen4_reg: vgen4 {
> - regulator-min-microvolt = <1800000>;
> - regulator-max-microvolt = <3300000>;
> - regulator-always-on;
> - };
> -
> - vgen5_reg: vgen5 {
> - regulator-min-microvolt = <1800000>;
> - regulator-max-microvolt = <3300000>;
> - regulator-always-on;
> - };
> -
> - vgen6_reg: vgen6 {
> - regulator-min-microvolt = <1800000>;
> - regulator-max-microvolt = <3300000>;
> - regulator-always-on;
> - };
> - };
> - };
> -};
> -
> &i2c4 {
> clock-frequency = <100000>;
> pinctrl-names = "default";
> @@ -340,28 +231,6 @@
> status = "okay";
> };
>
> -&qspi2 {
> - pinctrl-names = "default";
> - pinctrl-0 = <&pinctrl_qspi2>;
> - status = "okay";
> -
> - flash0: s25fl128s@0 {
> - reg = <0>;
> - #address-cells = <1>;
> - #size-cells = <1>;
> - compatible = "spansion,s25fl128s";
> - spi-max-frequency = <66000000>;
> - };
> -
> - flash1: s25fl128s@1 {
> - reg = <1>;
> - #address-cells = <1>;
> - #size-cells = <1>;
> - compatible = "spansion,s25fl128s";
> - spi-max-frequency = <66000000>;
> - };
> -};
> -
> &ssi2 {
> status = "okay";
> };
> --
> 1.9.1
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 1/2] ARM: imx6sx-sdb: change default board as reva board
@ 2015-01-20 7:35 ` Shawn Guo
0 siblings, 0 replies; 14+ messages in thread
From: Shawn Guo @ 2015-01-20 7:35 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Jan 14, 2015 at 03:36:37PM +0800, Peter Chen wrote:
> 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 <peter.chen@freescale.com>
> ---
> 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.
> 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.
Shawn
> +};
> +
> +&i2c1 {
> + clock-frequency = <100000>;
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_i2c1>;
> + status = "okay";
> +
> + pmic: pfuze100 at 08 {
> + compatible = "fsl,pfuze100";
> + reg = <0x08>;
> +
> + regulators {
> + sw1a_reg: sw1ab {
> + regulator-min-microvolt = <300000>;
> + regulator-max-microvolt = <1875000>;
> + regulator-boot-on;
> + regulator-always-on;
> + regulator-ramp-delay = <6250>;
> + };
> +
> + sw1c_reg: sw1c {
> + regulator-min-microvolt = <300000>;
> + regulator-max-microvolt = <1875000>;
> + regulator-boot-on;
> + regulator-always-on;
> + regulator-ramp-delay = <6250>;
> + };
> +
> + sw2_reg: sw2 {
> + regulator-min-microvolt = <800000>;
> + regulator-max-microvolt = <3300000>;
> + regulator-boot-on;
> + regulator-always-on;
> + };
> +
> + sw3a_reg: sw3a {
> + regulator-min-microvolt = <400000>;
> + regulator-max-microvolt = <1975000>;
> + regulator-boot-on;
> + regulator-always-on;
> + };
> +
> + sw3b_reg: sw3b {
> + regulator-min-microvolt = <400000>;
> + regulator-max-microvolt = <1975000>;
> + regulator-boot-on;
> + regulator-always-on;
> + };
> +
> + sw4_reg: sw4 {
> + regulator-min-microvolt = <800000>;
> + regulator-max-microvolt = <3300000>;
> + };
> +
> + swbst_reg: swbst {
> + regulator-min-microvolt = <5000000>;
> + regulator-max-microvolt = <5150000>;
> + };
> +
> + snvs_reg: vsnvs {
> + regulator-min-microvolt = <1000000>;
> + regulator-max-microvolt = <3000000>;
> + regulator-boot-on;
> + regulator-always-on;
> + };
> +
> + vref_reg: vrefddr {
> + regulator-boot-on;
> + regulator-always-on;
> + };
> +
> + vgen1_reg: vgen1 {
> + regulator-min-microvolt = <800000>;
> + regulator-max-microvolt = <1550000>;
> + regulator-always-on;
> + };
> +
> + vgen2_reg: vgen2 {
> + regulator-min-microvolt = <800000>;
> + regulator-max-microvolt = <1550000>;
> + };
> +
> + vgen3_reg: vgen3 {
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <3300000>;
> + regulator-always-on;
> + };
> +
> + vgen4_reg: vgen4 {
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <3300000>;
> + regulator-always-on;
> + };
> +
> + vgen5_reg: vgen5 {
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <3300000>;
> + regulator-always-on;
> + };
> +
> + vgen6_reg: vgen6 {
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <3300000>;
> + regulator-always-on;
> + };
> + };
> + };
> +};
> +
> +&qspi2 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_qspi2>;
> + status = "okay";
> +
> + flash0: s25fl128s at 0 {
> + reg = <0>;
> + #address-cells = <1>;
> + #size-cells = <1>;
> + compatible = "spansion,s25fl128s";
> + spi-max-frequency = <66000000>;
> + };
> +
> + flash1: s25fl128s at 1 {
> + reg = <1>;
> + #address-cells = <1>;
> + #size-cells = <1>;
> + compatible = "spansion,s25fl128s";
> + spi-max-frequency = <66000000>;
> + };
> +};
> diff --git a/arch/arm/boot/dts/imx6sx-sdb.dts b/arch/arm/boot/dts/imx6sx-sdb.dtsi
> similarity index 83%
> rename from arch/arm/boot/dts/imx6sx-sdb.dts
> rename to arch/arm/boot/dts/imx6sx-sdb.dtsi
> index cdffe84..7b02aa7 100644
> --- a/arch/arm/boot/dts/imx6sx-sdb.dts
> +++ b/arch/arm/boot/dts/imx6sx-sdb.dtsi
> @@ -13,7 +13,6 @@
> #include "imx6sx.dtsi"
>
> / {
> - model = "Freescale i.MX6 SoloX SDB Board";
> compatible = "fsl,imx6sx-sdb", "fsl,imx6sx";
>
> chosen {
> @@ -169,114 +168,6 @@
> status = "okay";
> };
>
> -&i2c1 {
> - clock-frequency = <100000>;
> - pinctrl-names = "default";
> - pinctrl-0 = <&pinctrl_i2c1>;
> - status = "okay";
> -
> - pmic: pfuze100 at 08 {
> - compatible = "fsl,pfuze100";
> - reg = <0x08>;
> -
> - regulators {
> - sw1a_reg: sw1ab {
> - regulator-min-microvolt = <300000>;
> - regulator-max-microvolt = <1875000>;
> - regulator-boot-on;
> - regulator-always-on;
> - regulator-ramp-delay = <6250>;
> - };
> -
> - sw1c_reg: sw1c {
> - regulator-min-microvolt = <300000>;
> - regulator-max-microvolt = <1875000>;
> - regulator-boot-on;
> - regulator-always-on;
> - regulator-ramp-delay = <6250>;
> - };
> -
> - sw2_reg: sw2 {
> - regulator-min-microvolt = <800000>;
> - regulator-max-microvolt = <3300000>;
> - regulator-boot-on;
> - regulator-always-on;
> - };
> -
> - sw3a_reg: sw3a {
> - regulator-min-microvolt = <400000>;
> - regulator-max-microvolt = <1975000>;
> - regulator-boot-on;
> - regulator-always-on;
> - };
> -
> - sw3b_reg: sw3b {
> - regulator-min-microvolt = <400000>;
> - regulator-max-microvolt = <1975000>;
> - regulator-boot-on;
> - regulator-always-on;
> - };
> -
> - sw4_reg: sw4 {
> - regulator-min-microvolt = <800000>;
> - regulator-max-microvolt = <3300000>;
> - };
> -
> - swbst_reg: swbst {
> - regulator-min-microvolt = <5000000>;
> - regulator-max-microvolt = <5150000>;
> - };
> -
> - snvs_reg: vsnvs {
> - regulator-min-microvolt = <1000000>;
> - regulator-max-microvolt = <3000000>;
> - regulator-boot-on;
> - regulator-always-on;
> - };
> -
> - vref_reg: vrefddr {
> - regulator-boot-on;
> - regulator-always-on;
> - };
> -
> - vgen1_reg: vgen1 {
> - regulator-min-microvolt = <800000>;
> - regulator-max-microvolt = <1550000>;
> - regulator-always-on;
> - };
> -
> - vgen2_reg: vgen2 {
> - regulator-min-microvolt = <800000>;
> - regulator-max-microvolt = <1550000>;
> - };
> -
> - vgen3_reg: vgen3 {
> - regulator-min-microvolt = <1800000>;
> - regulator-max-microvolt = <3300000>;
> - regulator-always-on;
> - };
> -
> - vgen4_reg: vgen4 {
> - regulator-min-microvolt = <1800000>;
> - regulator-max-microvolt = <3300000>;
> - regulator-always-on;
> - };
> -
> - vgen5_reg: vgen5 {
> - regulator-min-microvolt = <1800000>;
> - regulator-max-microvolt = <3300000>;
> - regulator-always-on;
> - };
> -
> - vgen6_reg: vgen6 {
> - regulator-min-microvolt = <1800000>;
> - regulator-max-microvolt = <3300000>;
> - regulator-always-on;
> - };
> - };
> - };
> -};
> -
> &i2c4 {
> clock-frequency = <100000>;
> pinctrl-names = "default";
> @@ -340,28 +231,6 @@
> status = "okay";
> };
>
> -&qspi2 {
> - pinctrl-names = "default";
> - pinctrl-0 = <&pinctrl_qspi2>;
> - status = "okay";
> -
> - flash0: s25fl128s at 0 {
> - reg = <0>;
> - #address-cells = <1>;
> - #size-cells = <1>;
> - compatible = "spansion,s25fl128s";
> - spi-max-frequency = <66000000>;
> - };
> -
> - flash1: s25fl128s at 1 {
> - reg = <1>;
> - #address-cells = <1>;
> - #size-cells = <1>;
> - compatible = "spansion,s25fl128s";
> - spi-max-frequency = <66000000>;
> - };
> -};
> -
> &ssi2 {
> status = "okay";
> };
> --
> 1.9.1
>
^ permalink raw reply [flat|nested] 14+ messages in thread
* RE: [PATCH 1/2] ARM: imx6sx-sdb: change default board as reva board
2015-01-20 6:30 ` Peter Chen
@ 2015-01-20 7:51 ` Anson.Huang at freescale.com
-1 siblings, 0 replies; 14+ messages in thread
From: Anson.Huang-KZfg59tc24xl57MIdRCFDg @ 2015-01-20 7:51 UTC (permalink / raw)
To: Peter Chen, Shawn Guo
Cc: kernel-bIcnvbaLZ9MEGnE8C9+IrQ, robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
pawel.moll-5wv7dgnIgG8, mark.rutland-5wv7dgnIgG8,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
devicetree-u79uwXL29TY76Z2rM5mHXA
Best regards!
Anson Huang
> -----Original Message-----
> From: Peter Chen [mailto:peter.chen-KZfg59tc24xl57MIdRCFDg@public.gmane.org]
> Sent: 2015-01-20 2:30 PM
> To: Shawn Guo; Huang Yongcai-B20788
> Cc: kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org; robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org; pawel.moll-5wv7dgnIgG8@public.gmane.org;
> mark.rutland-5wv7dgnIgG8@public.gmane.org; linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org;
> devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.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:
> > On Wed, Jan 14, 2015 at 03:36:37PM +0800, Peter Chen wrote:
> > > 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 <peter.chen-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
> > > ---
> > > 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.
>
>
> > > 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
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 1/2] ARM: imx6sx-sdb: change default board as reva board
@ 2015-01-20 7:51 ` Anson.Huang at freescale.com
0 siblings, 0 replies; 14+ messages in thread
From: Anson.Huang at freescale.com @ 2015-01-20 7:51 UTC (permalink / raw)
To: linux-arm-kernel
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:
> > On Wed, Jan 14, 2015 at 03:36:37PM +0800, Peter Chen wrote:
> > > 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 <peter.chen@freescale.com>
> > > ---
> > > 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.
>
>
> > > 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
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/2] ARM: imx6sx-sdb: change default board as reva board
2015-01-20 7:51 ` Anson.Huang at freescale.com
@ 2015-01-20 8:03 ` Shawn Guo
-1 siblings, 0 replies; 14+ messages in thread
From: Shawn Guo @ 2015-01-20 8:03 UTC (permalink / raw)
To: Anson.Huang-KZfg59tc24xl57MIdRCFDg
Cc: Peter Chen, kernel-bIcnvbaLZ9MEGnE8C9+IrQ,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
mark.rutland-5wv7dgnIgG8,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
devicetree-u79uwXL29TY76Z2rM5mHXA
On Tue, Jan 20, 2015 at 07:51:31AM +0000, Anson.Huang-KZfg59tc24xl57MIdRCFDg@public.gmane.org wrote:
> > > > +&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.
Thanks for the clarification, Anson.
Peter,
OPP is a CPU characteristic defined by data sheet, so we should define
OPP according to data sheet in imx6sx.dtsi, and only overwrite the
property in Rev-B dts per board/PMIC quirk.
Also, please put the rationale why we need OPP quick on Rev-B into
commit log.
Shawn
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 1/2] ARM: imx6sx-sdb: change default board as reva board
@ 2015-01-20 8:03 ` Shawn Guo
0 siblings, 0 replies; 14+ messages in thread
From: Shawn Guo @ 2015-01-20 8:03 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Jan 20, 2015 at 07:51:31AM +0000, Anson.Huang at freescale.com wrote:
> > > > +&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.
Thanks for the clarification, Anson.
Peter,
OPP is a CPU characteristic defined by data sheet, so we should define
OPP according to data sheet in imx6sx.dtsi, and only overwrite the
property in Rev-B dts per board/PMIC quirk.
Also, please put the rationale why we need OPP quick on Rev-B into
commit log.
Shawn
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2015-01-20 8:03 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-14 7:36 [PATCH 0/2] ARM: imx6sx-sdb: add revb board Peter Chen
2015-01-14 7:36 ` Peter Chen
[not found] ` <1421220998-3491-1-git-send-email-peter.chen-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
2015-01-14 7:36 ` [PATCH 1/2] ARM: imx6sx-sdb: change default board as reva board Peter Chen
2015-01-14 7:36 ` Peter Chen
[not found] ` <1421220998-3491-2-git-send-email-peter.chen-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
2015-01-20 7:35 ` Shawn Guo
2015-01-20 7:35 ` Shawn Guo
2015-01-20 6:30 ` Peter Chen
2015-01-20 6:30 ` Peter Chen
2015-01-20 7:51 ` Anson.Huang-KZfg59tc24xl57MIdRCFDg
2015-01-20 7:51 ` Anson.Huang at freescale.com
[not found] ` <BN1PR0301MB062873E37C225DD38B2BCE3CFF4B0-RQSpjbwlmjRcEY/EnGXkZZwN6zqB+hSMnBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org>
2015-01-20 8:03 ` Shawn Guo
2015-01-20 8:03 ` Shawn Guo
2015-01-14 7:36 ` [PATCH 2/2] ARM: imx6sx-sdb: add revb board Peter Chen
2015-01-14 7:36 ` Peter Chen
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.