All of lore.kernel.org
 help / color / mirror / Atom feed
* [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.