diff mbox

[v2] ARM: mvebu: fix ordering in Armada 370 .dtsi

Message ID 20141114204333.GS27002@pengutronix.de (mailing list archive)
State New, archived
Headers show

Commit Message

Uwe Kleine-König Nov. 14, 2014, 8:43 p.m. UTC
Commit a095b1c78a35 ("ARM: mvebu: sort DT nodes by address")
missed placing the system-controller in the correct order.                                                                                                               

Fixes: a095b1c78a35 ("ARM: mvebu: sort DT nodes by address")
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
Hello Andrew,

I guess your question concerning overlapping ranges doesn't stop this
patch from being useful, does it?

Also thanks for the suggested wording which I nearly took as is.

Best regards
Uwe

 arch/arm/boot/dts/armada-370.dtsi | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

Comments

Andrew Lunn Nov. 14, 2014, 8:50 p.m. UTC | #1
On Fri, Nov 14, 2014 at 09:43:33PM +0100, Uwe Kleine-König wrote:
> Commit a095b1c78a35 ("ARM: mvebu: sort DT nodes by address")
> missed placing the system-controller in the correct order.                                                                                                               
> 
> Fixes: a095b1c78a35 ("ARM: mvebu: sort DT nodes by address")
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> ---
> Hello Andrew,
> 
> I guess your question concerning overlapping ranges doesn't stop this
> patch from being useful, does it?

Not at all.

Acked-by: Andrew Lunn <andrew@lunn.ch>

	  Andrew
Jason Cooper Nov. 22, 2014, 3:37 a.m. UTC | #2
On Fri, Nov 14, 2014 at 09:43:33PM +0100, Uwe Kleine-König wrote:
> Commit a095b1c78a35 ("ARM: mvebu: sort DT nodes by address")
> missed placing the system-controller in the correct order.                                                                                                               
> 
> Fixes: a095b1c78a35 ("ARM: mvebu: sort DT nodes by address")
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> ---
> Hello Andrew,
> 
> I guess your question concerning overlapping ranges doesn't stop this
> patch from being useful, does it?
> 
> Also thanks for the suggested wording which I nearly took as is.
> 
> Best regards
> Uwe
> 
>  arch/arm/boot/dts/armada-370.dtsi | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)

Applied to mvebu/dt with Andrew's Ack.

thx,

Jason.
diff mbox

Patch

diff --git a/arch/arm/boot/dts/armada-370.dtsi b/arch/arm/boot/dts/armada-370.dtsi
index 6b3c23b1e138..7513410f7b89 100644
--- a/arch/arm/boot/dts/armada-370.dtsi
+++ b/arch/arm/boot/dts/armada-370.dtsi
@@ -106,11 +106,6 @@ 
 				reg = <0x11100 0x20>;
 			};
 
-			system-controller@18200 {
-				compatible = "marvell,armada-370-xp-system-controller";
-				reg = <0x18200 0x100>;
-			};
-
 			pinctrl {
 				compatible = "marvell,mv88f6710-pinctrl";
 				reg = <0x18000 0x38>;
@@ -205,6 +200,11 @@ 
 				interrupts = <91>;
 			};
 
+			system-controller@18200 {
+				compatible = "marvell,armada-370-xp-system-controller";
+				reg = <0x18200 0x100>;
+			};
+
 			gateclk: clock-gating-control@18220 {
 				compatible = "marvell,armada-370-gating-clock";
 				reg = <0x18220 0x4>;