* [PATCH] arm: mvebu: add DTS file for Marvell RD-A370-A1 board
@ 2013-01-09 19:48 Florian Fainelli
2013-01-09 19:56 ` [PATCH v2] " Florian Fainelli
0 siblings, 1 reply; 18+ messages in thread
From: Florian Fainelli @ 2013-01-09 19:48 UTC (permalink / raw)
To: linux-arm-kernel
This patch adds the DTS file to support the Marvell RD-A370-A1
(Reference Design) board also known as RD-88F6710 board. It is almost
entirely similar to the DB-A370 board except that the first Ethernet PHY
is SGMII-wired and the second is a switch which is RGMII-wired.
Signed-off-by: Florian Fainelli <florian@openwrt.org>
---
Thomas, Cl?ment, as support for other peripherals gets merged, I will update
this DTS file to reflect this. Thanks!
arch/arm/boot/dts/armada-370-rd.dts | 61 +++++++++++++++++++++++++++++++++++
1 file changed, 61 insertions(+)
create mode 100644 arch/arm/boot/dts/armada-370-rd.dts
diff --git a/arch/arm/boot/dts/armada-370-rd.dts b/arch/arm/boot/dts/armada-370-rd.dts
new file mode 100644
index 0000000..d62dfac
--- /dev/null
+++ b/arch/arm/boot/dts/armada-370-rd.dts
@@ -0,0 +1,61 @@
+/*
+ * Device Tree file for Marvell Armada 370 Reference Design board
+ * (RD-88F6710-A1)
+ *
+ * Copied from arch/arm/boot/dts/armada-370-db.dts
+ *
+ * Copyright (C) 2013 Florian Fainelli <florian@openwrt.org>
+ *
+ * This file is licensed under the terms of the GNU General Public
+ * License version 2. This program is licensed "as is" without any
+ * warranty of any kind, whether express or implied.
+ */
+
+/dts-v1/;
+/include/ "armada-370.dtsi"
+
+/ {
+ model = "Marvell Armada 370 Reference Design";
+ compatible = "marvell,a370-rd", "marvell,armada370", "marvell,armada-370-xp";
+
+ chosen {
+ bootargs = "console=ttyS0,115200 earlyprintk";
+ };
+
+ memory {
+ device_type = "memory";
+ reg = <0x00000000 0x20000000>; /* 512 MB */
+ };
+
+ soc {
+ serial at d0012000 {
+ clock-frequency = <200000000>;
+ status = "okay";
+ };
+ sata at d00a0000 {
+ nr-ports = <2>;
+ status = "okay";
+ };
+
+ mdio {
+ phy0: ethernet-phy at 0 {
+ reg = <0>;
+ };
+
+ phy1: ethernet-phy at 1 {
+ reg = <1>;
+ };
+ };
+
+ ethernet at d0070000 {
+ status = "okay";
+ phy = <&phy0>;
+ phy-mode = "sgmii";
+ };
+ ethernet at d0074000 {
+ status = "okay";
+ phy = <&phy1>;
+ phy-mode = "rgmii-id";
+ };
+ };
+};
--
1.7.10.4
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v2] arm: mvebu: add DTS file for Marvell RD-A370-A1 board
2013-01-09 19:48 [PATCH] arm: mvebu: add DTS file for Marvell RD-A370-A1 board Florian Fainelli
@ 2013-01-09 19:56 ` Florian Fainelli
2013-01-10 9:38 ` Gregory CLEMENT
` (3 more replies)
0 siblings, 4 replies; 18+ messages in thread
From: Florian Fainelli @ 2013-01-09 19:56 UTC (permalink / raw)
To: linux-arm-kernel
This patch adds the DTS file to support the Marvell RD-A370-A1
(Reference Design board) also known as RD-88F6710 board. It is almost
entirely similar to the DB-A370 board except that the first Ethernet PHY
is SGMII-wired and the second is a switch which is RGMII-wired.
Signed-off-by: Florian Fainelli <florian@openwrt.org>
---
Thomas, Cl?ment, as support for other peripherals gets merged, I will update
this DTS file to reflect this. Thanks!
Changes since v1:
- added missing entry in arch/arm/boot/dts/Makefile to actually build the DTB
arch/arm/boot/dts/Makefile | 1 +
arch/arm/boot/dts/armada-370-rd.dts | 61 +++++++++++++++++++++++++++++++++++
2 files changed, 62 insertions(+)
create mode 100644 arch/arm/boot/dts/armada-370-rd.dts
diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index b7246d6..7a75171 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -78,6 +78,7 @@ dtb-$(CONFIG_ARCH_MSM) += msm8660-surf.dtb \
msm8960-cdp.dtb
dtb-$(CONFIG_ARCH_MVEBU) += armada-370-db.dtb \
armada-370-mirabox.dtb \
+ armada-370-rd.dtb \
armada-xp-db.dtb \
armada-xp-openblocks-ax3-4.dtb
dtb-$(CONFIG_ARCH_MXC) += imx51-babbage.dtb \
diff --git a/arch/arm/boot/dts/armada-370-rd.dts b/arch/arm/boot/dts/armada-370-rd.dts
new file mode 100644
index 0000000..d62dfac
--- /dev/null
+++ b/arch/arm/boot/dts/armada-370-rd.dts
@@ -0,0 +1,61 @@
+/*
+ * Device Tree file for Marvell Armada 370 Reference Design board
+ * (RD-88F6710-A1)
+ *
+ * Copied from arch/arm/boot/dts/armada-370-db.dts
+ *
+ * Copyright (C) 2013 Florian Fainelli <florian@openwrt.org>
+ *
+ * This file is licensed under the terms of the GNU General Public
+ * License version 2. This program is licensed "as is" without any
+ * warranty of any kind, whether express or implied.
+ */
+
+/dts-v1/;
+/include/ "armada-370.dtsi"
+
+/ {
+ model = "Marvell Armada 370 Reference Design";
+ compatible = "marvell,a370-rd", "marvell,armada370", "marvell,armada-370-xp";
+
+ chosen {
+ bootargs = "console=ttyS0,115200 earlyprintk";
+ };
+
+ memory {
+ device_type = "memory";
+ reg = <0x00000000 0x20000000>; /* 512 MB */
+ };
+
+ soc {
+ serial at d0012000 {
+ clock-frequency = <200000000>;
+ status = "okay";
+ };
+ sata at d00a0000 {
+ nr-ports = <2>;
+ status = "okay";
+ };
+
+ mdio {
+ phy0: ethernet-phy at 0 {
+ reg = <0>;
+ };
+
+ phy1: ethernet-phy at 1 {
+ reg = <1>;
+ };
+ };
+
+ ethernet at d0070000 {
+ status = "okay";
+ phy = <&phy0>;
+ phy-mode = "sgmii";
+ };
+ ethernet at d0074000 {
+ status = "okay";
+ phy = <&phy1>;
+ phy-mode = "rgmii-id";
+ };
+ };
+};
--
1.7.10.4
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v2] arm: mvebu: add DTS file for Marvell RD-A370-A1 board
2013-01-09 19:56 ` [PATCH v2] " Florian Fainelli
@ 2013-01-10 9:38 ` Gregory CLEMENT
2013-01-10 12:51 ` Gregory CLEMENT
` (2 subsequent siblings)
3 siblings, 0 replies; 18+ messages in thread
From: Gregory CLEMENT @ 2013-01-10 9:38 UTC (permalink / raw)
To: linux-arm-kernel
On 01/09/2013 08:56 PM, Florian Fainelli wrote:
> This patch adds the DTS file to support the Marvell RD-A370-A1
> (Reference Design board) also known as RD-88F6710 board. It is almost
> entirely similar to the DB-A370 board except that the first Ethernet PHY
> is SGMII-wired and the second is a switch which is RGMII-wired.
>
> Signed-off-by: Florian Fainelli <florian@openwrt.org>
Looks OK for me
Acked-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
> ---
> Thomas, Cl?ment, as support for other peripherals gets merged, I will update
> this DTS file to reflect this. Thanks!
>
> Changes since v1:
> - added missing entry in arch/arm/boot/dts/Makefile to actually build the DTB
>
> arch/arm/boot/dts/Makefile | 1 +
> arch/arm/boot/dts/armada-370-rd.dts | 61 +++++++++++++++++++++++++++++++++++
> 2 files changed, 62 insertions(+)
> create mode 100644 arch/arm/boot/dts/armada-370-rd.dts
>
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index b7246d6..7a75171 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -78,6 +78,7 @@ dtb-$(CONFIG_ARCH_MSM) += msm8660-surf.dtb \
> msm8960-cdp.dtb
> dtb-$(CONFIG_ARCH_MVEBU) += armada-370-db.dtb \
> armada-370-mirabox.dtb \
> + armada-370-rd.dtb \
> armada-xp-db.dtb \
> armada-xp-openblocks-ax3-4.dtb
> dtb-$(CONFIG_ARCH_MXC) += imx51-babbage.dtb \
> diff --git a/arch/arm/boot/dts/armada-370-rd.dts b/arch/arm/boot/dts/armada-370-rd.dts
> new file mode 100644
> index 0000000..d62dfac
> --- /dev/null
> +++ b/arch/arm/boot/dts/armada-370-rd.dts
> @@ -0,0 +1,61 @@
> +/*
> + * Device Tree file for Marvell Armada 370 Reference Design board
> + * (RD-88F6710-A1)
> + *
> + * Copied from arch/arm/boot/dts/armada-370-db.dts
> + *
> + * Copyright (C) 2013 Florian Fainelli <florian@openwrt.org>
> + *
> + * This file is licensed under the terms of the GNU General Public
> + * License version 2. This program is licensed "as is" without any
> + * warranty of any kind, whether express or implied.
> + */
> +
> +/dts-v1/;
> +/include/ "armada-370.dtsi"
> +
> +/ {
> + model = "Marvell Armada 370 Reference Design";
> + compatible = "marvell,a370-rd", "marvell,armada370", "marvell,armada-370-xp";
> +
> + chosen {
> + bootargs = "console=ttyS0,115200 earlyprintk";
> + };
> +
> + memory {
> + device_type = "memory";
> + reg = <0x00000000 0x20000000>; /* 512 MB */
> + };
> +
> + soc {
> + serial at d0012000 {
> + clock-frequency = <200000000>;
> + status = "okay";
> + };
> + sata at d00a0000 {
> + nr-ports = <2>;
> + status = "okay";
> + };
> +
> + mdio {
> + phy0: ethernet-phy at 0 {
> + reg = <0>;
> + };
> +
> + phy1: ethernet-phy at 1 {
> + reg = <1>;
> + };
> + };
> +
> + ethernet at d0070000 {
> + status = "okay";
> + phy = <&phy0>;
> + phy-mode = "sgmii";
> + };
> + ethernet at d0074000 {
> + status = "okay";
> + phy = <&phy1>;
> + phy-mode = "rgmii-id";
> + };
> + };
> +};
>
--
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] 18+ messages in thread
* [PATCH v2] arm: mvebu: add DTS file for Marvell RD-A370-A1 board
2013-01-09 19:56 ` [PATCH v2] " Florian Fainelli
2013-01-10 9:38 ` Gregory CLEMENT
@ 2013-01-10 12:51 ` Gregory CLEMENT
2013-01-10 12:57 ` Florian Fainelli
2013-01-11 14:45 ` Florian Fainelli
2013-01-14 14:47 ` [PATCH v3] " Florian Fainelli
3 siblings, 1 reply; 18+ messages in thread
From: Gregory CLEMENT @ 2013-01-10 12:51 UTC (permalink / raw)
To: linux-arm-kernel
On 01/09/2013 08:56 PM, Florian Fainelli wrote:
> This patch adds the DTS file to support the Marvell RD-A370-A1
> (Reference Design board) also known as RD-88F6710 board. It is almost
> entirely similar to the DB-A370 board except that the first Ethernet PHY
> is SGMII-wired and the second is a switch which is RGMII-wired.
>
> Signed-off-by: Florian Fainelli <florian@openwrt.org>
> ---
> Thomas, Cl?ment, as support for other peripherals gets merged, I will update
> this DTS file to reflect this. Thanks!
>
> Changes since v1:
> - added missing entry in arch/arm/boot/dts/Makefile to actually build the DTB
>
> arch/arm/boot/dts/Makefile | 1 +
> arch/arm/boot/dts/armada-370-rd.dts | 61 +++++++++++++++++++++++++++++++++++
> 2 files changed, 62 insertions(+)
> create mode 100644 arch/arm/boot/dts/armada-370-rd.dts
>
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index b7246d6..7a75171 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -78,6 +78,7 @@ dtb-$(CONFIG_ARCH_MSM) += msm8660-surf.dtb \
> msm8960-cdp.dtb
> dtb-$(CONFIG_ARCH_MVEBU) += armada-370-db.dtb \
> armada-370-mirabox.dtb \
> + armada-370-rd.dtb \
> armada-xp-db.dtb \
> armada-xp-openblocks-ax3-4.dtb
> dtb-$(CONFIG_ARCH_MXC) += imx51-babbage.dtb \
> diff --git a/arch/arm/boot/dts/armada-370-rd.dts b/arch/arm/boot/dts/armada-370-rd.dts
> new file mode 100644
> index 0000000..d62dfac
> --- /dev/null
> +++ b/arch/arm/boot/dts/armada-370-rd.dts
> @@ -0,0 +1,61 @@
> +/*
> + * Device Tree file for Marvell Armada 370 Reference Design board
> + * (RD-88F6710-A1)
> + *
> + * Copied from arch/arm/boot/dts/armada-370-db.dts
> + *
> + * Copyright (C) 2013 Florian Fainelli <florian@openwrt.org>
> + *
> + * This file is licensed under the terms of the GNU General Public
> + * License version 2. This program is licensed "as is" without any
> + * warranty of any kind, whether express or implied.
> + */
> +
> +/dts-v1/;
> +/include/ "armada-370.dtsi"
> +
> +/ {
> + model = "Marvell Armada 370 Reference Design";
> + compatible = "marvell,a370-rd", "marvell,armada370", "marvell,armada-370-xp";
> +
> + chosen {
> + bootargs = "console=ttyS0,115200 earlyprintk";
> + };
> +
> + memory {
> + device_type = "memory";
> + reg = <0x00000000 0x20000000>; /* 512 MB */
Hi Florian,
As you based your file on our (erroneous) dts, could you confirm that
the RD-A370-A1 board have "only" 512MB?
Thanks,
> + };
> +
> + soc {
> + serial at d0012000 {
> + clock-frequency = <200000000>;
> + status = "okay";
> + };
> + sata at d00a0000 {
> + nr-ports = <2>;
> + status = "okay";
> + };
> +
> + mdio {
> + phy0: ethernet-phy at 0 {
> + reg = <0>;
> + };
> +
> + phy1: ethernet-phy at 1 {
> + reg = <1>;
> + };
> + };
> +
> + ethernet at d0070000 {
> + status = "okay";
> + phy = <&phy0>;
> + phy-mode = "sgmii";
> + };
> + ethernet at d0074000 {
> + status = "okay";
> + phy = <&phy1>;
> + phy-mode = "rgmii-id";
> + };
> + };
> +};
>
--
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] 18+ messages in thread
* [PATCH v2] arm: mvebu: add DTS file for Marvell RD-A370-A1 board
2013-01-10 12:51 ` Gregory CLEMENT
@ 2013-01-10 12:57 ` Florian Fainelli
0 siblings, 0 replies; 18+ messages in thread
From: Florian Fainelli @ 2013-01-10 12:57 UTC (permalink / raw)
To: linux-arm-kernel
Hello Gregory,
Le 01/10/13 13:51, Gregory CLEMENT a ?crit :
> +
> + memory {
> + device_type = "memory";
> + reg = <0x00000000 0x20000000>; /* 512 MB */
> Hi Florian,
>
> As you based your file on our (erroneous) dts, could you confirm that
> the RD-A370-A1 board have "only" 512MB?
Yes it only has 512MB of RAM, this is not a mistake.
--
Florian
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v2] arm: mvebu: add DTS file for Marvell RD-A370-A1 board
2013-01-09 19:56 ` [PATCH v2] " Florian Fainelli
2013-01-10 9:38 ` Gregory CLEMENT
2013-01-10 12:51 ` Gregory CLEMENT
@ 2013-01-11 14:45 ` Florian Fainelli
2013-01-11 14:53 ` Gregory CLEMENT
2013-01-11 14:56 ` Jason Cooper
2013-01-14 14:47 ` [PATCH v3] " Florian Fainelli
3 siblings, 2 replies; 18+ messages in thread
From: Florian Fainelli @ 2013-01-11 14:45 UTC (permalink / raw)
To: linux-arm-kernel
Thomas, Gregory,
Le 01/09/13 20:56, Florian Fainelli a ?crit :
> This patch adds the DTS file to support the Marvell RD-A370-A1
> (Reference Design board) also known as RD-88F6710 board. It is almost
> entirely similar to the DB-A370 board except that the first Ethernet PHY
> is SGMII-wired and the second is a switch which is RGMII-wired.
Who is going to take this patch? Since this is a new DTS file there are
little chances it breaks anything, could it be taken for an upcoming 3.8-rc?
Thanks.
--
Florian
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v2] arm: mvebu: add DTS file for Marvell RD-A370-A1 board
2013-01-11 14:45 ` Florian Fainelli
@ 2013-01-11 14:53 ` Gregory CLEMENT
2013-01-11 14:56 ` Jason Cooper
1 sibling, 0 replies; 18+ messages in thread
From: Gregory CLEMENT @ 2013-01-11 14:53 UTC (permalink / raw)
To: linux-arm-kernel
Florian,
On 01/11/2013 03:45 PM, Florian Fainelli wrote:
> Thomas, Gregory,
>
> Le 01/09/13 20:56, Florian Fainelli a ?crit :
>> This patch adds the DTS file to support the Marvell RD-A370-A1
>> (Reference Design board) also known as RD-88F6710 board. It is almost
>> entirely similar to the DB-A370 board except that the first Ethernet PHY
>> is SGMII-wired and the second is a switch which is RGMII-wired.
>
> Who is going to take this patch? Since this is a new DTS file there are
> little chances it breaks anything, could it be taken for an upcoming 3.8-rc?
It is Jason who takes care of gathering all the patches for mvebu.
As it is not a fix for a bug but more a new "feature" (at least a support
for a new board), I doubt that it could be part of 3.8-rc.
But as far as I am concerned I am not against having this patch in 3.8-rc.
Regards
>
> Thanks.
> --
> Florian
>
--
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] 18+ messages in thread
* [PATCH v2] arm: mvebu: add DTS file for Marvell RD-A370-A1 board
2013-01-11 14:45 ` Florian Fainelli
2013-01-11 14:53 ` Gregory CLEMENT
@ 2013-01-11 14:56 ` Jason Cooper
2013-01-13 19:40 ` Florian Fainelli
1 sibling, 1 reply; 18+ messages in thread
From: Jason Cooper @ 2013-01-11 14:56 UTC (permalink / raw)
To: linux-arm-kernel
Florian,
On Fri, Jan 11, 2013 at 03:45:04PM +0100, Florian Fainelli wrote:
> Le 01/09/13 20:56, Florian Fainelli a ?crit :
> >This patch adds the DTS file to support the Marvell RD-A370-A1
> >(Reference Design board) also known as RD-88F6710 board. It is almost
> >entirely similar to the DB-A370 board except that the first Ethernet PHY
> >is SGMII-wired and the second is a switch which is RGMII-wired.
>
> Who is going to take this patch? Since this is a new DTS file there
> are little chances it breaks anything, could it be taken for an
> upcoming 3.8-rc?
No, only fixes go in for the current -rc. It'll be included for v3.9, I
have it in my queue.
thx,
Jason.
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v2] arm: mvebu: add DTS file for Marvell RD-A370-A1 board
2013-01-11 14:56 ` Jason Cooper
@ 2013-01-13 19:40 ` Florian Fainelli
2013-01-13 20:12 ` Jason Cooper
0 siblings, 1 reply; 18+ messages in thread
From: Florian Fainelli @ 2013-01-13 19:40 UTC (permalink / raw)
To: linux-arm-kernel
Le 11/01/2013 15:56, Jason Cooper a ?crit :
> Florian,
>
> On Fri, Jan 11, 2013 at 03:45:04PM +0100, Florian Fainelli wrote:
>> Le 01/09/13 20:56, Florian Fainelli a ?crit :
>>> This patch adds the DTS file to support the Marvell RD-A370-A1
>>> (Reference Design board) also known as RD-88F6710 board. It is almost
>>> entirely similar to the DB-A370 board except that the first Ethernet PHY
>>> is SGMII-wired and the second is a switch which is RGMII-wired.
>>
>> Who is going to take this patch? Since this is a new DTS file there
>> are little chances it breaks anything, could it be taken for an
>> upcoming 3.8-rc?
>
> No, only fixes go in for the current -rc. It'll be included for v3.9, I
> have it in my queue.
Jason, I don't see this patch in the pull request you just sent for 3.9,
should it be? Thanks!
--
Florian
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v2] arm: mvebu: add DTS file for Marvell RD-A370-A1 board
2013-01-13 19:40 ` Florian Fainelli
@ 2013-01-13 20:12 ` Jason Cooper
2013-01-14 9:36 ` Florian Fainelli
0 siblings, 1 reply; 18+ messages in thread
From: Jason Cooper @ 2013-01-13 20:12 UTC (permalink / raw)
To: linux-arm-kernel
On Sun, Jan 13, 2013 at 08:40:29PM +0100, Florian Fainelli wrote:
> Le 11/01/2013 15:56, Jason Cooper a ?crit :
> >Florian,
> >
> >On Fri, Jan 11, 2013 at 03:45:04PM +0100, Florian Fainelli wrote:
> >>Le 01/09/13 20:56, Florian Fainelli a ?crit :
> >>>This patch adds the DTS file to support the Marvell RD-A370-A1
> >>>(Reference Design board) also known as RD-88F6710 board. It is almost
> >>>entirely similar to the DB-A370 board except that the first Ethernet PHY
> >>>is SGMII-wired and the second is a switch which is RGMII-wired.
> >>
> >>Who is going to take this patch? Since this is a new DTS file there
> >>are little chances it breaks anything, could it be taken for an
> >>upcoming 3.8-rc?
> >
> >No, only fixes go in for the current -rc. It'll be included for v3.9, I
> >have it in my queue.
>
> Jason, I don't see this patch in the pull request you just sent for
> 3.9, should it be? Thanks!
Not yet, I'm working my though the holiday backlog chronologically.
thx,
Jason.
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v2] arm: mvebu: add DTS file for Marvell RD-A370-A1 board
2013-01-13 20:12 ` Jason Cooper
@ 2013-01-14 9:36 ` Florian Fainelli
0 siblings, 0 replies; 18+ messages in thread
From: Florian Fainelli @ 2013-01-14 9:36 UTC (permalink / raw)
To: linux-arm-kernel
Le 01/13/13 21:12, Jason Cooper a ?crit :
> On Sun, Jan 13, 2013 at 08:40:29PM +0100, Florian Fainelli wrote:
>> Le 11/01/2013 15:56, Jason Cooper a ?crit :
>>> Florian,
>>>
>>> On Fri, Jan 11, 2013 at 03:45:04PM +0100, Florian Fainelli wrote:
>>>> Le 01/09/13 20:56, Florian Fainelli a ?crit :
>>>>> This patch adds the DTS file to support the Marvell RD-A370-A1
>>>>> (Reference Design board) also known as RD-88F6710 board. It is almost
>>>>> entirely similar to the DB-A370 board except that the first Ethernet PHY
>>>>> is SGMII-wired and the second is a switch which is RGMII-wired.
>>>> Who is going to take this patch? Since this is a new DTS file there
>>>> are little chances it breaks anything, could it be taken for an
>>>> upcoming 3.8-rc?
>>> No, only fixes go in for the current -rc. It'll be included for v3.9, I
>>> have it in my queue.
>> Jason, I don't see this patch in the pull request you just sent for
>> 3.9, should it be? Thanks!
> Not yet, I'm working my though the holiday backlog chronologically.
Ok, then you might want to hold this one a little, so I add mvsdio
bindings to this DTS file as well. Thanks!
--
Florian
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v3] arm: mvebu: add DTS file for Marvell RD-A370-A1 board
2013-01-09 19:56 ` [PATCH v2] " Florian Fainelli
` (2 preceding siblings ...)
2013-01-11 14:45 ` Florian Fainelli
@ 2013-01-14 14:47 ` Florian Fainelli
2013-01-23 14:29 ` Jason Cooper
2013-01-31 17:57 ` Jason Cooper
3 siblings, 2 replies; 18+ messages in thread
From: Florian Fainelli @ 2013-01-14 14:47 UTC (permalink / raw)
To: linux-arm-kernel
This patch adds the DTS file to support the Marvell RD-A370-A1
(Reference Design board) also known as RD-88F6710 board. It is almost
entirely similar to the DB-A370 except on the following points:
- second Ethernet MAC is connected to a switch using RGMII
- it only has 512MB of physical RAM
- SDIO interface is enabled and working by default (no need for jumpers)
Signed-off-by: Florian Fainelli <florian@openwrt.org>
---
Jason,
This is rebased against your mvebu/for-next branch, can you please drop v2 of
the previous patch and take this one instead? Thanks!
Changes since v2:
- rebased against Jason's mvebu/for-next
- added SDIO bindings
- changed commit message a bit
Changes since v1:
- added missing entry in arch/arm/boot/dts/Makefile to actually build the DTB
arch/arm/boot/dts/Makefile | 1 +
arch/arm/boot/dts/armada-370-rd.dts | 68 +++++++++++++++++++++++++++++++++++
2 files changed, 69 insertions(+)
create mode 100644 arch/arm/boot/dts/armada-370-rd.dts
diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index b7246d6..7a75171 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -78,6 +78,7 @@ dtb-$(CONFIG_ARCH_MSM) += msm8660-surf.dtb \
msm8960-cdp.dtb
dtb-$(CONFIG_ARCH_MVEBU) += armada-370-db.dtb \
armada-370-mirabox.dtb \
+ armada-370-rd.dtb \
armada-xp-db.dtb \
armada-xp-openblocks-ax3-4.dtb
dtb-$(CONFIG_ARCH_MXC) += imx51-babbage.dtb \
diff --git a/arch/arm/boot/dts/armada-370-rd.dts b/arch/arm/boot/dts/armada-370-rd.dts
new file mode 100644
index 0000000..f8e4855
--- /dev/null
+++ b/arch/arm/boot/dts/armada-370-rd.dts
@@ -0,0 +1,68 @@
+/*
+ * Device Tree file for Marvell Armada 370 Reference Design board
+ * (RD-88F6710-A1)
+ *
+ * Copied from arch/arm/boot/dts/armada-370-db.dts
+ *
+ * Copyright (C) 2013 Florian Fainelli <florian@openwrt.org>
+ *
+ * This file is licensed under the terms of the GNU General Public
+ * License version 2. This program is licensed "as is" without any
+ * warranty of any kind, whether express or implied.
+ */
+
+/dts-v1/;
+/include/ "armada-370.dtsi"
+
+/ {
+ model = "Marvell Armada 370 Reference Design";
+ compatible = "marvell,a370-rd", "marvell,armada370", "marvell,armada-370-xp";
+
+ chosen {
+ bootargs = "console=ttyS0,115200 earlyprintk";
+ };
+
+ memory {
+ device_type = "memory";
+ reg = <0x00000000 0x20000000>; /* 512 MB */
+ };
+
+ soc {
+ serial at d0012000 {
+ clock-frequency = <200000000>;
+ status = "okay";
+ };
+ sata at d00a0000 {
+ nr-ports = <2>;
+ status = "okay";
+ };
+
+ mdio {
+ phy0: ethernet-phy at 0 {
+ reg = <0>;
+ };
+
+ phy1: ethernet-phy at 1 {
+ reg = <1>;
+ };
+ };
+
+ ethernet at d0070000 {
+ status = "okay";
+ phy = <&phy0>;
+ phy-mode = "sgmii";
+ };
+ ethernet at d0074000 {
+ status = "okay";
+ phy = <&phy1>;
+ phy-mode = "rgmii-id";
+ };
+
+ mvsdio at d00d4000 {
+ pinctrl-0 = <&sdio_pins1>;
+ pinctrl-names = "default";
+ status = "okay";
+ /* No CD or WP GPIOs */
+ };
+ };
+};
--
1.7.10.4
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v3] arm: mvebu: add DTS file for Marvell RD-A370-A1 board
2013-01-14 14:47 ` [PATCH v3] " Florian Fainelli
@ 2013-01-23 14:29 ` Jason Cooper
2013-01-23 14:40 ` Jason Cooper
2013-01-31 17:57 ` Jason Cooper
1 sibling, 1 reply; 18+ messages in thread
From: Jason Cooper @ 2013-01-23 14:29 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Jan 14, 2013 at 03:47:32PM +0100, Florian Fainelli wrote:
> This patch adds the DTS file to support the Marvell RD-A370-A1
> (Reference Design board) also known as RD-88F6710 board. It is almost
> entirely similar to the DB-A370 except on the following points:
>
> - second Ethernet MAC is connected to a switch using RGMII
> - it only has 512MB of physical RAM
> - SDIO interface is enabled and working by default (no need for jumpers)
>
> Signed-off-by: Florian Fainelli <florian@openwrt.org>
> ---
> Jason,
>
> This is rebased against your mvebu/for-next branch, can you please drop v2 of
> the previous patch and take this one instead? Thanks!
>
> Changes since v2:
> - rebased against Jason's mvebu/for-next
> - added SDIO bindings
> - changed commit message a bit
>
> Changes since v1:
> - added missing entry in arch/arm/boot/dts/Makefile to actually build the DTB
>
> arch/arm/boot/dts/Makefile | 1 +
> arch/arm/boot/dts/armada-370-rd.dts | 68 +++++++++++++++++++++++++++++++++++
> 2 files changed, 69 insertions(+)
> create mode 100644 arch/arm/boot/dts/armada-370-rd.dts
Applied to mvebu/dt
thx,
Jason.
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v3] arm: mvebu: add DTS file for Marvell RD-A370-A1 board
2013-01-23 14:29 ` Jason Cooper
@ 2013-01-23 14:40 ` Jason Cooper
0 siblings, 0 replies; 18+ messages in thread
From: Jason Cooper @ 2013-01-23 14:40 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Jan 23, 2013 at 09:29:33AM -0500, Jason Cooper wrote:
> On Mon, Jan 14, 2013 at 03:47:32PM +0100, Florian Fainelli wrote:
> > This patch adds the DTS file to support the Marvell RD-A370-A1
> > (Reference Design board) also known as RD-88F6710 board. It is almost
> > entirely similar to the DB-A370 except on the following points:
> >
> > - second Ethernet MAC is connected to a switch using RGMII
> > - it only has 512MB of physical RAM
> > - SDIO interface is enabled and working by default (no need for jumpers)
> >
> > Signed-off-by: Florian Fainelli <florian@openwrt.org>
> > ---
> > Jason,
> >
> > This is rebased against your mvebu/for-next branch, can you please drop v2 of
> > the previous patch and take this one instead? Thanks!
> >
> > Changes since v2:
> > - rebased against Jason's mvebu/for-next
> > - added SDIO bindings
> > - changed commit message a bit
> >
> > Changes since v1:
> > - added missing entry in arch/arm/boot/dts/Makefile to actually build the DTB
> >
> > arch/arm/boot/dts/Makefile | 1 +
> > arch/arm/boot/dts/armada-370-rd.dts | 68 +++++++++++++++++++++++++++++++++++
> > 2 files changed, 69 insertions(+)
> > create mode 100644 arch/arm/boot/dts/armada-370-rd.dts
>
> Applied to mvebu/dt
oops, I meant mvebu/boards.
thx,
Jason.
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v3] arm: mvebu: add DTS file for Marvell RD-A370-A1 board
2013-01-14 14:47 ` [PATCH v3] " Florian Fainelli
2013-01-23 14:29 ` Jason Cooper
@ 2013-01-31 17:57 ` Jason Cooper
2013-01-31 19:28 ` Florian Fainelli
2013-02-01 10:38 ` [PATCH] arm: mvebu: enable the SD card slot on Armada 370 Reference Design board Florian Fainelli
1 sibling, 2 replies; 18+ messages in thread
From: Jason Cooper @ 2013-01-31 17:57 UTC (permalink / raw)
To: linux-arm-kernel
Florian,
On Mon, Jan 14, 2013 at 03:47:32PM +0100, Florian Fainelli wrote:
> This patch adds the DTS file to support the Marvell RD-A370-A1
> (Reference Design board) also known as RD-88F6710 board. It is almost
> entirely similar to the DB-A370 except on the following points:
>
> - second Ethernet MAC is connected to a switch using RGMII
> - it only has 512MB of physical RAM
> - SDIO interface is enabled and working by default (no need for jumpers)
>
> Signed-off-by: Florian Fainelli <florian@openwrt.org>
> ---
> Jason,
>
> This is rebased against your mvebu/for-next branch, can you please drop v2 of
> the previous patch and take this one instead? Thanks!
>
> Changes since v2:
> - rebased against Jason's mvebu/for-next
> - added SDIO bindings
> - changed commit message a bit
This breaks badly as sdio bindings aren't added until mvebu/dt. Since
this is a new board, it goes in *before* mvebu/dt (mvebu/boards). I'm
going to go ahead and pull v2 instead as there seems to be no
significant difference other than mvsdio being added.
If you want to get sdio support in for v3.9, please submit a separate
patch on top of your v2.
thx,
Jason.
>
> Changes since v1:
> - added missing entry in arch/arm/boot/dts/Makefile to actually build the DTB
>
> arch/arm/boot/dts/Makefile | 1 +
> arch/arm/boot/dts/armada-370-rd.dts | 68 +++++++++++++++++++++++++++++++++++
> 2 files changed, 69 insertions(+)
> create mode 100644 arch/arm/boot/dts/armada-370-rd.dts
>
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index b7246d6..7a75171 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -78,6 +78,7 @@ dtb-$(CONFIG_ARCH_MSM) += msm8660-surf.dtb \
> msm8960-cdp.dtb
> dtb-$(CONFIG_ARCH_MVEBU) += armada-370-db.dtb \
> armada-370-mirabox.dtb \
> + armada-370-rd.dtb \
> armada-xp-db.dtb \
> armada-xp-openblocks-ax3-4.dtb
> dtb-$(CONFIG_ARCH_MXC) += imx51-babbage.dtb \
> diff --git a/arch/arm/boot/dts/armada-370-rd.dts b/arch/arm/boot/dts/armada-370-rd.dts
> new file mode 100644
> index 0000000..f8e4855
> --- /dev/null
> +++ b/arch/arm/boot/dts/armada-370-rd.dts
> @@ -0,0 +1,68 @@
> +/*
> + * Device Tree file for Marvell Armada 370 Reference Design board
> + * (RD-88F6710-A1)
> + *
> + * Copied from arch/arm/boot/dts/armada-370-db.dts
> + *
> + * Copyright (C) 2013 Florian Fainelli <florian@openwrt.org>
> + *
> + * This file is licensed under the terms of the GNU General Public
> + * License version 2. This program is licensed "as is" without any
> + * warranty of any kind, whether express or implied.
> + */
> +
> +/dts-v1/;
> +/include/ "armada-370.dtsi"
> +
> +/ {
> + model = "Marvell Armada 370 Reference Design";
> + compatible = "marvell,a370-rd", "marvell,armada370", "marvell,armada-370-xp";
> +
> + chosen {
> + bootargs = "console=ttyS0,115200 earlyprintk";
> + };
> +
> + memory {
> + device_type = "memory";
> + reg = <0x00000000 0x20000000>; /* 512 MB */
> + };
> +
> + soc {
> + serial at d0012000 {
> + clock-frequency = <200000000>;
> + status = "okay";
> + };
> + sata at d00a0000 {
> + nr-ports = <2>;
> + status = "okay";
> + };
> +
> + mdio {
> + phy0: ethernet-phy at 0 {
> + reg = <0>;
> + };
> +
> + phy1: ethernet-phy at 1 {
> + reg = <1>;
> + };
> + };
> +
> + ethernet at d0070000 {
> + status = "okay";
> + phy = <&phy0>;
> + phy-mode = "sgmii";
> + };
> + ethernet at d0074000 {
> + status = "okay";
> + phy = <&phy1>;
> + phy-mode = "rgmii-id";
> + };
> +
> + mvsdio at d00d4000 {
> + pinctrl-0 = <&sdio_pins1>;
> + pinctrl-names = "default";
> + status = "okay";
> + /* No CD or WP GPIOs */
> + };
> + };
> +};
> --
> 1.7.10.4
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v3] arm: mvebu: add DTS file for Marvell RD-A370-A1 board
2013-01-31 17:57 ` Jason Cooper
@ 2013-01-31 19:28 ` Florian Fainelli
2013-02-01 10:38 ` [PATCH] arm: mvebu: enable the SD card slot on Armada 370 Reference Design board Florian Fainelli
1 sibling, 0 replies; 18+ messages in thread
From: Florian Fainelli @ 2013-01-31 19:28 UTC (permalink / raw)
To: linux-arm-kernel
Le 31/01/2013 18:57, Jason Cooper a ?crit :
> Florian,
>
> On Mon, Jan 14, 2013 at 03:47:32PM +0100, Florian Fainelli wrote:
>> This patch adds the DTS file to support the Marvell RD-A370-A1
>> (Reference Design board) also known as RD-88F6710 board. It is almost
>> entirely similar to the DB-A370 except on the following points:
>>
>> - second Ethernet MAC is connected to a switch using RGMII
>> - it only has 512MB of physical RAM
>> - SDIO interface is enabled and working by default (no need for jumpers)
>>
>> Signed-off-by: Florian Fainelli <florian@openwrt.org>
>> ---
>> Jason,
>>
>> This is rebased against your mvebu/for-next branch, can you please drop v2 of
>> the previous patch and take this one instead? Thanks!
>>
>> Changes since v2:
>> - rebased against Jason's mvebu/for-next
>> - added SDIO bindings
>> - changed commit message a bit
>
> This breaks badly as sdio bindings aren't added until mvebu/dt. Since
> this is a new board, it goes in *before* mvebu/dt (mvebu/boards). I'm
> going to go ahead and pull v2 instead as there seems to be no
> significant difference other than mvsdio being added.
Whoops, sounds like the right thing to do, thanks!
>
> If you want to get sdio support in for v3.9, please submit a separate
> patch on top of your v2.
Will do.
--
Florian
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] arm: mvebu: enable the SD card slot on Armada 370 Reference Design board
2013-01-31 17:57 ` Jason Cooper
2013-01-31 19:28 ` Florian Fainelli
@ 2013-02-01 10:38 ` Florian Fainelli
2013-02-16 16:29 ` Jason Cooper
1 sibling, 1 reply; 18+ messages in thread
From: Florian Fainelli @ 2013-02-01 10:38 UTC (permalink / raw)
To: linux-arm-kernel
The Armada 370 Reference Design board has one SD card slot, directly
connected to the SDIO IP of the SoC, so we enable this IP. there are no
GPIOs for card-detect and write-protect so we do not specify any.
Signed-off-by: Florian Fainelli <florian@openwrt.org>
---
Jason,
This is against your mvebu/dt branch
arch/arm/boot/dts/armada-370-rd.dts | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/arch/arm/boot/dts/armada-370-rd.dts b/arch/arm/boot/dts/armada-370-rd.dts
index d62dfac..f8e4855 100644
--- a/arch/arm/boot/dts/armada-370-rd.dts
+++ b/arch/arm/boot/dts/armada-370-rd.dts
@@ -57,5 +57,12 @@
phy = <&phy1>;
phy-mode = "rgmii-id";
};
+
+ mvsdio at d00d4000 {
+ pinctrl-0 = <&sdio_pins1>;
+ pinctrl-names = "default";
+ status = "okay";
+ /* No CD or WP GPIOs */
+ };
};
};
--
1.7.10.4
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH] arm: mvebu: enable the SD card slot on Armada 370 Reference Design board
2013-02-01 10:38 ` [PATCH] arm: mvebu: enable the SD card slot on Armada 370 Reference Design board Florian Fainelli
@ 2013-02-16 16:29 ` Jason Cooper
0 siblings, 0 replies; 18+ messages in thread
From: Jason Cooper @ 2013-02-16 16:29 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Feb 01, 2013 at 11:38:28AM +0100, Florian Fainelli wrote:
> The Armada 370 Reference Design board has one SD card slot, directly
> connected to the SDIO IP of the SoC, so we enable this IP. there are no
> GPIOs for card-detect and write-protect so we do not specify any.
>
> Signed-off-by: Florian Fainelli <florian@openwrt.org>
> ---
> Jason,
>
> This is against your mvebu/dt branch
>
> arch/arm/boot/dts/armada-370-rd.dts | 7 +++++++
> 1 file changed, 7 insertions(+)
Applied to mvebu/dt
thx,
Jason.
^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2013-02-16 16:29 UTC | newest]
Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-01-09 19:48 [PATCH] arm: mvebu: add DTS file for Marvell RD-A370-A1 board Florian Fainelli
2013-01-09 19:56 ` [PATCH v2] " Florian Fainelli
2013-01-10 9:38 ` Gregory CLEMENT
2013-01-10 12:51 ` Gregory CLEMENT
2013-01-10 12:57 ` Florian Fainelli
2013-01-11 14:45 ` Florian Fainelli
2013-01-11 14:53 ` Gregory CLEMENT
2013-01-11 14:56 ` Jason Cooper
2013-01-13 19:40 ` Florian Fainelli
2013-01-13 20:12 ` Jason Cooper
2013-01-14 9:36 ` Florian Fainelli
2013-01-14 14:47 ` [PATCH v3] " Florian Fainelli
2013-01-23 14:29 ` Jason Cooper
2013-01-23 14:40 ` Jason Cooper
2013-01-31 17:57 ` Jason Cooper
2013-01-31 19:28 ` Florian Fainelli
2013-02-01 10:38 ` [PATCH] arm: mvebu: enable the SD card slot on Armada 370 Reference Design board Florian Fainelli
2013-02-16 16:29 ` Jason Cooper
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.