diff mbox

[FIX] ARM: mvebu: add missing DT Mbus ranges and relocate PCIe DT nodes for RN102

Message ID 87a9iuq7ff.fsf@natisbad.org (mailing list archive)
State New, archived
Headers show

Commit Message

Arnaud Ebalard Sept. 30, 2013, 10:19 p.m. UTC
When 5e12a613 and 0cd3754a were introduced, Netgear ReadyNAS 102 .dts
file was queued for inclusion and missed the update to have Mbus (and
then BootROM) ranges properties declared. It also missed the relocation
of Armada 370/XP PCIe DT nodes introduced by 14fd8ed0 after de1af8d4.
This patch fixes that which makes 3.12-rc3 bootable on the NAS.

Signed-off-by: Arnaud Ebalard <arno@natisbad.org>
---
 arch/arm/boot/dts/armada-370-netgear-rn102.dts | 35 ++++++++++++++------------
 1 file changed, 19 insertions(+), 16 deletions(-)

Comments

Jason Cooper Oct. 1, 2013, 12:50 a.m. UTC | #1
On Tue, Oct 01, 2013 at 12:19:16AM +0200, Arnaud Ebalard wrote:
> 
> When 5e12a613 and 0cd3754a were introduced, Netgear ReadyNAS 102 .dts
> file was queued for inclusion and missed the update to have Mbus (and
> then BootROM) ranges properties declared. It also missed the relocation
> of Armada 370/XP PCIe DT nodes introduced by 14fd8ed0 after de1af8d4.
> This patch fixes that which makes 3.12-rc3 bootable on the NAS.
> 
> Signed-off-by: Arnaud Ebalard <arno@natisbad.org>
> ---
>  arch/arm/boot/dts/armada-370-netgear-rn102.dts | 35 ++++++++++++++------------
>  1 file changed, 19 insertions(+), 16 deletions(-)

Applied to mvebu/fixes

thx,

Jason.
Olof Johansson Oct. 3, 2013, 3:57 a.m. UTC | #2
Hi,

On Mon, Sep 30, 2013 at 3:19 PM, Arnaud Ebalard <arno@natisbad.org> wrote:
>
> When 5e12a613 and 0cd3754a were introduced, Netgear ReadyNAS 102 .dts
> file was queued for inclusion and missed the update to have Mbus (and
> then BootROM) ranges properties declared. It also missed the relocation
> of Armada 370/XP PCIe DT nodes introduced by 14fd8ed0 after de1af8d4.
> This patch fixes that which makes 3.12-rc3 bootable on the NAS.

In the future please don't just quote the SHAs, give the patch subject
as well. It makes it easier to follow the patch description without
looking them up (besides, there's a chance that the 8-character SHAs
will not be unique at some point in the future, which makes the lookup
harder).


-Olof
Arnaud Ebalard Oct. 3, 2013, 5:54 a.m. UTC | #3
Hi,

Olof Johansson <olof@lixom.net> writes:

> Hi,
>
> On Mon, Sep 30, 2013 at 3:19 PM, Arnaud Ebalard <arno@natisbad.org> wrote:
>>
>> When 5e12a613 and 0cd3754a were introduced, Netgear ReadyNAS 102 .dts
>> file was queued for inclusion and missed the update to have Mbus (and
>> then BootROM) ranges properties declared. It also missed the relocation
>> of Armada 370/XP PCIe DT nodes introduced by 14fd8ed0 after de1af8d4.
>> This patch fixes that which makes 3.12-rc3 bootable on the NAS.
>
> In the future please don't just quote the SHAs, give the patch subject
> as well. It makes it easier to follow the patch description without
> looking them up (besides, there's a chance that the 8-character SHAs
> will not be unique at some point in the future, which makes the lookup
> harder).

Dully noted. Will do it next time, olof.

Cheers,

a+
diff mbox

Patch

diff --git a/arch/arm/boot/dts/armada-370-netgear-rn102.dts b/arch/arm/boot/dts/armada-370-netgear-rn102.dts
index 05e4485..aef5ac5 100644
--- a/arch/arm/boot/dts/armada-370-netgear-rn102.dts
+++ b/arch/arm/boot/dts/armada-370-netgear-rn102.dts
@@ -27,6 +27,25 @@ 
 	};
 
 	soc {
+		ranges = <MBUS_ID(0xf0, 0x01) 0 0xd0000000 0x100000
+			  MBUS_ID(0x01, 0xe0) 0 0xfff00000 0x100000>;
+
+		pcie-controller {
+			status = "okay";
+
+			/* Connected to Marvell SATA controller */
+			pcie@1,0 {
+				/* Port 0, Lane 0 */
+				status = "okay";
+			};
+
+			/* Connected to FL1009 USB 3.0 controller */
+			pcie@2,0 {
+				/* Port 1, Lane 0 */
+				status = "okay";
+			};
+		};
+
 		internal-regs {
 			serial@12000 {
 				clock-frequency = <200000000>;
@@ -89,22 +108,6 @@ 
 					pwm_polarity = <0>;
 				};
 			};
-
-			pcie-controller {
-				status = "okay";
-
-				/* Connected to Marvell SATA controller */
-				pcie@1,0 {
-					/* Port 0, Lane 0 */
-					status = "okay";
-				};
-
-				/* Connected to FL1009 USB 3.0 controller */
-				pcie@2,0 {
-					/* Port 1, Lane 0 */
-					status = "okay";
-				};
-			};
 		};
 	};