Re: [PATCH 01/17] ARM: dts: aspeed-g5: Move EDAC node to APB

[Stefan Schaeckeler (sschaeck)]

On Thursday, July 25, 2019 at 10:40 PM, Andrew Jeffery wrote:

> Previously the register interface was not attached to any internal bus,
> which is not correct - it lives on the APB.
>
> Cc: Stefan M Schaeckeler 
> Signed-off-by: Andrew Jeffery 
> ---
>  arch/arm/boot/dts/aspeed-g5.dtsi | 14 +++---
>  1 file changed, 7 insertions(+), 7 deletions(-)

Reviewed-by: Stefan Schaeckeler 

[PATCH 01/17] ARM: dts: aspeed-g5: Move EDAC node to APB

[Andrew Jeffery]

Previously the register interface was not attached to any internal bus,
which is not correct - it lives on the APB.

Cc: Stefan M Schaeckeler 
Signed-off-by: Andrew Jeffery 
---
 arch/arm/boot/dts/aspeed-g5.dtsi | 14 +++---
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/arch/arm/boot/dts/aspeed-g5.dtsi b/arch/arm/boot/dts/aspeed-g5.dtsi
index 5b1ca265c2ce..7723afc7c249 100644
--- a/arch/arm/boot/dts/aspeed-g5.dtsi
+++ b/arch/arm/boot/dts/aspeed-g5.dtsi
@@ -47,13 +47,6 @@
reg = <0x8000 0>;
};
 
-   edac: sdram@1e6e {
-   compatible = "aspeed,ast2500-sdram-edac";
-   reg = <0x1e6e 0x174>;
-   interrupts = <0>;
-   status = "disabled";
-   };
-
ahb {
compatible = "simple-bus";
#address-cells = <1>;
@@ -206,6 +199,13 @@
#size-cells = <1>;
ranges;
 
+   edac: sdram@1e6e {
+   compatible = "aspeed,ast2500-sdram-edac";
+   reg = <0x1e6e 0x174>;
+   interrupts = <0>;
+   status = "disabled";
+   };
+
syscon: syscon@1e6e2000 {
compatible = "aspeed,ast2500-scu", "syscon", 
"simple-mfd";
reg = <0x1e6e2000 0x1a8>;
-- 
2.20.1