* [PATCH v3 1/6] arm64: dts: marvell: armada-3720-db: Sort the dts node alphabetically
2017-06-13 13:57 [PATCH v3 0/6] Update device tree for Armada 3720 DB Gregory CLEMENT
@ 2017-06-13 13:57 ` Gregory CLEMENT
2017-06-13 13:57 ` [PATCH v3 2/6] arm64: dts: marvell: armada-3720-db: Add information about the V2 board Gregory CLEMENT
` (5 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: Gregory CLEMENT @ 2017-06-13 13:57 UTC (permalink / raw)
To: linux-arm-kernel
Sort the reference nodes in alphabetical order to ease the merge of
future nodes.
Reviewed-by: Andreas F?rber <afaerber@suse.de>
Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
---
arch/arm64/boot/dts/marvell/armada-3720-db.dts | 83 +++++++++++++-------------
1 file changed, 41 insertions(+), 42 deletions(-)
diff --git a/arch/arm64/boot/dts/marvell/armada-3720-db.dts b/arch/arm64/boot/dts/marvell/armada-3720-db.dts
index a89855f57091..d6f12fafac6c 100644
--- a/arch/arm64/boot/dts/marvell/armada-3720-db.dts
+++ b/arch/arm64/boot/dts/marvell/armada-3720-db.dts
@@ -78,6 +78,20 @@
};
};
+ð0 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&rgmii_pins>;
+ phy-mode = "rgmii-id";
+ phy = <&phy0>;
+ status = "okay";
+};
+
+ð1 {
+ phy-mode = "sgmii";
+ phy = <&phy1>;
+ status = "okay";
+};
+
&i2c0 {
pinctrl-names = "default";
pinctrl-0 = <&i2c1_pins>;
@@ -108,11 +122,36 @@
};
};
+&mdio {
+ status = "okay";
+ phy0: ethernet-phy at 0 {
+ reg = <0>;
+ };
+
+ phy1: ethernet-phy at 1 {
+ reg = <1>;
+ };
+};
+
+/* CON17 (PCIe) / CON12 (mini-PCIe) */
+&pcie0 {
+ status = "okay";
+};
+
/* CON3 */
&sata {
status = "okay";
};
+&sdhci0 {
+ non-removable;
+ bus-width = <8>;
+ mmc-ddr-1_8v;
+ mmc-hs400-1_8v;
+ marvell,pad-type = "fixed-1-8v";
+ status = "okay";
+};
+
&spi0 {
status = "okay";
pinctrl-names = "default";
@@ -152,12 +191,8 @@
status = "okay";
};
-&sdhci0 {
- non-removable;
- bus-width = <8>;
- mmc-ddr-1_8v;
- mmc-hs400-1_8v;
- marvell,pad-type = "fixed-1-8v";
+/* CON27 */
+&usb2 {
status = "okay";
};
@@ -166,39 +201,3 @@
status = "okay";
usb-phy = <&usb3_phy>;
};
-
-/* CON17 (PCIe) / CON12 (mini-PCIe) */
-&pcie0 {
- status = "okay";
-};
-
-/* CON27 */
-&usb2 {
- status = "okay";
-};
-
-
-&mdio {
- status = "okay";
- phy0: ethernet-phy at 0 {
- reg = <0>;
- };
-
- phy1: ethernet-phy at 1 {
- reg = <1>;
- };
-};
-
-ð0 {
- pinctrl-names = "default";
- pinctrl-0 = <&rgmii_pins>;
- phy-mode = "rgmii-id";
- phy = <&phy0>;
- status = "okay";
-};
-
-ð1 {
- phy-mode = "sgmii";
- phy = <&phy1>;
- status = "okay";
-};
--
2.11.0
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH v3 2/6] arm64: dts: marvell: armada-3720-db: Add information about the V2 board
2017-06-13 13:57 [PATCH v3 0/6] Update device tree for Armada 3720 DB Gregory CLEMENT
2017-06-13 13:57 ` [PATCH v3 1/6] arm64: dts: marvell: armada-3720-db: Sort the dts node alphabetically Gregory CLEMENT
@ 2017-06-13 13:57 ` Gregory CLEMENT
2017-06-13 13:57 ` [PATCH v3 3/6] arm64: dts: marvell: armada-37xx: Align the compatible string Gregory CLEMENT
` (4 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: Gregory CLEMENT @ 2017-06-13 13:57 UTC (permalink / raw)
To: linux-arm-kernel
The initial device tree file was for the board V1.4. Now the V2.0 board
is also available. The same dtb will work for both, but the CON number
have changed, so update the comment in the dts to reflect this.
Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
---
arch/arm64/boot/dts/marvell/armada-3720-db.dts | 17 +++++++++++++----
1 file changed, 13 insertions(+), 4 deletions(-)
diff --git a/arch/arm64/boot/dts/marvell/armada-3720-db.dts b/arch/arm64/boot/dts/marvell/armada-3720-db.dts
index d6f12fafac6c..3ee920ac0015 100644
--- a/arch/arm64/boot/dts/marvell/armada-3720-db.dts
+++ b/arch/arm64/boot/dts/marvell/armada-3720-db.dts
@@ -42,6 +42,10 @@
* WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
* FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
* OTHER DEALINGS IN THE SOFTWARE.
+ *
+ * This file is compatible with the version 1.4 and the version 2.0 of
+ * the board, however the CON numbers are different between the 2
+ * version
*/
/dts-v1/;
@@ -78,6 +82,7 @@
};
};
+/* Gigabit module on CON19(V2.0)/CON21(V1.4) */
ð0 {
pinctrl-names = "default";
pinctrl-0 = <&rgmii_pins>;
@@ -86,6 +91,7 @@
status = "okay";
};
+/* Gigabit module on CON18(V2.0)/CON20(V1.4) */
ð1 {
phy-mode = "sgmii";
phy = <&phy1>;
@@ -133,7 +139,7 @@
};
};
-/* CON17 (PCIe) / CON12 (mini-PCIe) */
+/* CON15(V2.0)/CON17(V1.4) : PCIe / CON15(V2.0)/CON12(V1.4) :mini-PCIe */
&pcie0 {
status = "okay";
};
@@ -184,19 +190,22 @@
};
};
-/* Exported on the micro USB connector CON32 through an FTDI */
+/*
+ * Exported on the micro USB connector CON30(V2.0)/CON32(V1.4) through
+ * an FTDI
+ */
&uart0 {
pinctrl-names = "default";
pinctrl-0 = <&uart1_pins>;
status = "okay";
};
-/* CON27 */
+/* CON27(V2.0)/CON29(V1.4) */
&usb2 {
status = "okay";
};
-/* CON31 */
+/* CON29(V2.0)/CON31(V1.4) */
&usb3 {
status = "okay";
usb-phy = <&usb3_phy>;
--
2.11.0
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH v3 3/6] arm64: dts: marvell: armada-37xx: Align the compatible string
2017-06-13 13:57 [PATCH v3 0/6] Update device tree for Armada 3720 DB Gregory CLEMENT
2017-06-13 13:57 ` [PATCH v3 1/6] arm64: dts: marvell: armada-3720-db: Sort the dts node alphabetically Gregory CLEMENT
2017-06-13 13:57 ` [PATCH v3 2/6] arm64: dts: marvell: armada-3720-db: Add information about the V2 board Gregory CLEMENT
@ 2017-06-13 13:57 ` Gregory CLEMENT
2017-06-13 13:57 ` [PATCH v3 4/6] arm64: dts: marvell: armada-37xx: Use angle bracket for each register set Gregory CLEMENT
` (3 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: Gregory CLEMENT @ 2017-06-13 13:57 UTC (permalink / raw)
To: linux-arm-kernel
This cosmetic patch aligns the compatible string when there are on
several lines.
Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
---
arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
index 4d495ec39202..c1076fd0bd9e 100644
--- a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
@@ -163,7 +163,7 @@
pinctrl_nb: pinctrl at 13800 {
compatible = "marvell,armada3710-nb-pinctrl",
- "syscon", "simple-mfd";
+ "syscon", "simple-mfd";
reg = <0x13800 0x100>, <0x13C00 0x20>;
gpionb: gpio {
#gpio-cells = <2>;
@@ -219,7 +219,7 @@
pinctrl_sb: pinctrl at 18800 {
compatible = "marvell,armada3710-sb-pinctrl",
- "syscon", "simple-mfd";
+ "syscon", "simple-mfd";
reg = <0x18800 0x100>, <0x18C00 0x20>;
gpiosb: gpio {
#gpio-cells = <2>;
@@ -294,7 +294,7 @@
sdhci0: sdhci at d8000 {
compatible = "marvell,armada-3700-sdhci",
- "marvell,sdhci-xenon";
+ "marvell,sdhci-xenon";
reg = <0xd8000 0x300
0x17808 0x4>;
interrupts = <GIC_SPI 26 IRQ_TYPE_LEVEL_HIGH>;
--
2.11.0
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH v3 4/6] arm64: dts: marvell: armada-37xx: Use angle bracket for each register set
2017-06-13 13:57 [PATCH v3 0/6] Update device tree for Armada 3720 DB Gregory CLEMENT
` (2 preceding siblings ...)
2017-06-13 13:57 ` [PATCH v3 3/6] arm64: dts: marvell: armada-37xx: Align the compatible string Gregory CLEMENT
@ 2017-06-13 13:57 ` Gregory CLEMENT
2017-06-13 13:57 ` [PATCH v3 5/6] arm64: dts: marvell: Enable second SDHCI controller in Armada 37xx Gregory CLEMENT
` (2 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: Gregory CLEMENT @ 2017-06-13 13:57 UTC (permalink / raw)
To: linux-arm-kernel
When several groups of register address and size are used with reg, then
surround each one by angle bracket.
Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
---
arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
index c1076fd0bd9e..65562319770b 100644
--- a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
@@ -281,8 +281,8 @@
xor at 60900 {
compatible = "marvell,armada-3700-xor";
- reg = <0x60900 0x100
- 0x60b00 0x100>;
+ reg = <0x60900 0x100>,
+ <0x60b00 0x100>;
xor10 {
interrupts = <GIC_SPI 47 IRQ_TYPE_LEVEL_HIGH>;
@@ -295,8 +295,8 @@
sdhci0: sdhci at d8000 {
compatible = "marvell,armada-3700-sdhci",
"marvell,sdhci-xenon";
- reg = <0xd8000 0x300
- 0x17808 0x4>;
+ reg = <0xd8000 0x300>,
+ <0x17808 0x4>;
interrupts = <GIC_SPI 26 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&nb_periph_clk 0>;
clock-names = "core";
--
2.11.0
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH v3 5/6] arm64: dts: marvell: Enable second SDHCI controller in Armada 37xx
2017-06-13 13:57 [PATCH v3 0/6] Update device tree for Armada 3720 DB Gregory CLEMENT
` (3 preceding siblings ...)
2017-06-13 13:57 ` [PATCH v3 4/6] arm64: dts: marvell: armada-37xx: Use angle bracket for each register set Gregory CLEMENT
@ 2017-06-13 13:57 ` Gregory CLEMENT
2017-06-13 13:57 ` [PATCH v3 6/6] arm64: dts: marvell: armada-3720-db: Add vqmmc regulator for SD slot Gregory CLEMENT
2017-06-15 16:11 ` [PATCH v3 0/6] Update device tree for Armada 3720 DB Gregory CLEMENT
6 siblings, 0 replies; 9+ messages in thread
From: Gregory CLEMENT @ 2017-06-13 13:57 UTC (permalink / raw)
To: linux-arm-kernel
From: Konstantin Porotchkin <kostap@marvell.com>
The Armada 37xx SoCs has 2 SDHCI interfaces. This patch adds the second
one.
Moreover, the Armada 37xx DB v2 board populates the 2 SDHCI interfaces.
The second interface is using pluggable module that can either
have an SD connector or eMMC on it.
This patch adds support for SD module in the device DT.
[ gregory.clement at free-electrons.com:
- Add more detail in commit log
- Sort the dt node in address order
- Document the SD slot in the dts ]
Signed-off-by: Konstantin Porotchkin <kostap@marvell.com>
Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
---
arch/arm64/boot/dts/marvell/armada-3720-db.dts | 9 +++++++++
arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 11 +++++++++++
2 files changed, 20 insertions(+)
diff --git a/arch/arm64/boot/dts/marvell/armada-3720-db.dts b/arch/arm64/boot/dts/marvell/armada-3720-db.dts
index 3ee920ac0015..eab3bc7e6382 100644
--- a/arch/arm64/boot/dts/marvell/armada-3720-db.dts
+++ b/arch/arm64/boot/dts/marvell/armada-3720-db.dts
@@ -158,6 +158,15 @@
status = "okay";
};
+/* SD slot module on CON14(V2.0)/CON15(V1.4) */
+&sdhci1 {
+ wp-inverted;
+ cd-gpios = <&gpiosb 2 GPIO_ACTIVE_LOW>;
+ bus-width = <4>;
+ marvell,pad-type = "sd";
+ status = "okay";
+};
+
&spi0 {
status = "okay";
pinctrl-names = "default";
diff --git a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
index 65562319770b..b4d27857c61b 100644
--- a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
@@ -292,6 +292,17 @@
};
};
+ sdhci1: sdhci at d0000 {
+ compatible = "marvell,armada-3700-sdhci",
+ "marvell,sdhci-xenon";
+ reg = <0xd0000 0x300>,
+ <0x1e808 0x4>;
+ interrupts = <GIC_SPI 25 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&nb_periph_clk 0>;
+ clock-names = "core";
+ status = "disabled";
+ };
+
sdhci0: sdhci at d8000 {
compatible = "marvell,armada-3700-sdhci",
"marvell,sdhci-xenon";
--
2.11.0
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH v3 6/6] arm64: dts: marvell: armada-3720-db: Add vqmmc regulator for SD slot
2017-06-13 13:57 [PATCH v3 0/6] Update device tree for Armada 3720 DB Gregory CLEMENT
` (4 preceding siblings ...)
2017-06-13 13:57 ` [PATCH v3 5/6] arm64: dts: marvell: Enable second SDHCI controller in Armada 37xx Gregory CLEMENT
@ 2017-06-13 13:57 ` Gregory CLEMENT
2017-06-15 16:11 ` [PATCH v3 0/6] Update device tree for Armada 3720 DB Gregory CLEMENT
6 siblings, 0 replies; 9+ messages in thread
From: Gregory CLEMENT @ 2017-06-13 13:57 UTC (permalink / raw)
To: linux-arm-kernel
By adding this regulator, the SD cards are usable at higher speed
protocols such as SDR104.
This patch was tested with an SD HC card compatible with UHS-I.
Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
---
arch/arm64/boot/dts/marvell/armada-3720-db.dts | 15 +++++++++++++++
1 file changed, 15 insertions(+)
diff --git a/arch/arm64/boot/dts/marvell/armada-3720-db.dts b/arch/arm64/boot/dts/marvell/armada-3720-db.dts
index eab3bc7e6382..9df0f06ce607 100644
--- a/arch/arm64/boot/dts/marvell/armada-3720-db.dts
+++ b/arch/arm64/boot/dts/marvell/armada-3720-db.dts
@@ -80,6 +80,20 @@
compatible = "usb-nop-xceiv";
vcc-supply = <&exp_usb3_vbus>;
};
+
+ vcc_sd_reg1: regulator {
+ compatible = "regulator-gpio";
+ regulator-name = "vcc_sd1";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-boot-on;
+
+ gpios = <&gpiosb 23 GPIO_ACTIVE_HIGH>;
+ gpios-states = <0>;
+ states = <1800000 0x1
+ 3300000 0x0>;
+ enable-active-high;
+ };
};
/* Gigabit module on CON19(V2.0)/CON21(V1.4) */
@@ -164,6 +178,7 @@
cd-gpios = <&gpiosb 2 GPIO_ACTIVE_LOW>;
bus-width = <4>;
marvell,pad-type = "sd";
+ vqmmc-supply = <&vcc_sd_reg1>;
status = "okay";
};
--
2.11.0
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH v3 0/6] Update device tree for Armada 3720 DB
2017-06-13 13:57 [PATCH v3 0/6] Update device tree for Armada 3720 DB Gregory CLEMENT
` (5 preceding siblings ...)
2017-06-13 13:57 ` [PATCH v3 6/6] arm64: dts: marvell: armada-3720-db: Add vqmmc regulator for SD slot Gregory CLEMENT
@ 2017-06-15 16:11 ` Gregory CLEMENT
2017-06-15 16:24 ` Gregory CLEMENT
6 siblings, 1 reply; 9+ messages in thread
From: Gregory CLEMENT @ 2017-06-15 16:11 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On mar., juin 13 2017, Gregory CLEMENT <gregory.clement@free-electrons.com> wrote:
> Hi,
>
> the main purpose of this short series is to add the support of the
> second SDHCI controller on the Armada 3720 DB board (and on the Armada
> 37xx SoC).
>
> This series also adds some improvement to the device tree of the the
> Armada 3720 DB board and the last patch allow using higher speed on
> the SD card by adding the vqmmc regulator.
>
> In the v3, I use the angle bracket for each set of register. And I
> added the reviewed by flag from Andreas F?rber.
>
> In the v2, I added a patch aligning the compatible string and fix the
> patch " arm64: dts: marvell: Enable second SDHCI controller in Armada
> 37xx" too.
>
> Gregory
>
>
> Gregory CLEMENT (5):
> arm64: dts: marvell: armada-3720-db: Sort the dts node alphabetically
> arm64: dts: marvell: armada-3720-db: Add information about the V2
> board
> arm64: dts: marvell: armada-37xx: Align the compatible string
> arm64: dts: marvell: armada-37xx: Use angle bracket for each register
> set
> arm64: dts: marvell: armada-3720-db: Add vqmmc regulator for SD slot
>
> Konstantin Porotchkin (1):
> arm64: dts: marvell: Enable second SDHCI controller in Armada 37xx
All series applided on mvebu/dt
Gregory
>
> arch/arm64/boot/dts/marvell/armada-3720-db.dts | 120 ++++++++++++++++---------
> arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 25 ++++--
> 2 files changed, 94 insertions(+), 51 deletions(-)
>
> --
> 2.11.0
>
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 9+ messages in thread* [PATCH v3 0/6] Update device tree for Armada 3720 DB
2017-06-15 16:11 ` [PATCH v3 0/6] Update device tree for Armada 3720 DB Gregory CLEMENT
@ 2017-06-15 16:24 ` Gregory CLEMENT
0 siblings, 0 replies; 9+ messages in thread
From: Gregory CLEMENT @ 2017-06-15 16:24 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On jeu., juin 15 2017, Gregory CLEMENT <gregory.clement@free-electrons.com> wrote:
> Hi,
>
> On mar., juin 13 2017, Gregory CLEMENT <gregory.clement@free-electrons.com> wrote:
>
>> Hi,
>>
>> the main purpose of this short series is to add the support of the
>> second SDHCI controller on the Armada 3720 DB board (and on the Armada
>> 37xx SoC).
>>
>> This series also adds some improvement to the device tree of the the
>> Armada 3720 DB board and the last patch allow using higher speed on
>> the SD card by adding the vqmmc regulator.
>>
>> In the v3, I use the angle bracket for each set of register. And I
>> added the reviewed by flag from Andreas F?rber.
>>
>> In the v2, I added a patch aligning the compatible string and fix the
>> patch " arm64: dts: marvell: Enable second SDHCI controller in Armada
>> 37xx" too.
>>
>> Gregory
>>
>>
>> Gregory CLEMENT (5):
>> arm64: dts: marvell: armada-3720-db: Sort the dts node alphabetically
>> arm64: dts: marvell: armada-3720-db: Add information about the V2
>> board
>> arm64: dts: marvell: armada-37xx: Align the compatible string
>> arm64: dts: marvell: armada-37xx: Use angle bracket for each register
>> set
>> arm64: dts: marvell: armada-3720-db: Add vqmmc regulator for SD slot
>>
>> Konstantin Porotchkin (1):
>> arm64: dts: marvell: Enable second SDHCI controller in Armada 37xx
>
> All series applided on mvebu/dt
And now applied on the correct tree: mvebu/dt64
Gregory
>
> Gregory
>
>>
>> arch/arm64/boot/dts/marvell/armada-3720-db.dts | 120 ++++++++++++++++---------
>> arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 25 ++++--
>> 2 files changed, 94 insertions(+), 51 deletions(-)
>>
>> --
>> 2.11.0
>>
>
> --
> Gregory Clement, Free Electrons
> Kernel, drivers, real-time and embedded Linux
> development, consulting, training and support.
> http://free-electrons.com
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 9+ messages in thread