Skip to contentSkip to conversation

ARM: dts: aspeed: Set correct memory size for 2400 based opp platforms

PATCH In review Cyril Bur 2 participants 4 messages Dec 13, 2016

Subsystems (2)
Device tree bindings Arm/Aspeed machine support
lore
Share & tools
lore

Cyril Bur

author
#0 lore raw

PATCH 1/2 ARM: dts: aspeed: Set correct memory size for 2400 based opp platforms

These platforms all have 512MB of memory as opposed to the previously thought 256MB.

Update the device trees accordingly

Signed-off-by Cyril Bur <[email protected]>
---
 arch/arm/boot/dts/aspeed-bmc-opp-barreleye.dts | 2 +-
 arch/arm/boot/dts/aspeed-bmc-opp-firestone.dts | 2 +-
 arch/arm/boot/dts/aspeed-bmc-opp-garrison.dts  | 2 +-
 arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts  | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)
arch/arm/boot/dts/aspeed-bmc-opp-barreleye.dts +1 -1
index 802ce95..353033b 100644
@@ -17,7 +17,7 @@
 	};
 
 	memory {
-		reg = <0x40000000 0x10000000>;
+		reg = <0x40000000 0x20000000>;
 	};
 
 	ahb {
arch/arm/boot/dts/aspeed-bmc-opp-firestone.dts +1 -1
index 005ff89..ee7b569 100644
@@ -21,7 +21,7 @@
 	};
 
 	memory {
-		reg = < 0x40000000 0x10000000 >;
+		reg = <0x40000000 0x20000000>;
 	};
 
 	leds {
arch/arm/boot/dts/aspeed-bmc-opp-garrison.dts +1 -1
index 92b29d1..3f3fd1f 100644
@@ -17,7 +17,7 @@
 	};
 
 	memory {
-		reg = < 0x40000000 0x10000000 >;
+		reg = <0x40000000 0x20000000>;
 	};
 
 	ahb {
arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts +1 -1
index acaceda2..816c004 100644
@@ -17,7 +17,7 @@
 	};
 
 	memory {
-		reg = <0x40000000 0x10000000>;
+		reg = <0x40000000 0x20000000>;
 	};
 
         leds {
Signature · 1 line
2.10.2
author
#1 lore raw

PATCH 2/2 ARM: dts: aspeed: Reserve framebuffer memory 2400 based opp platforms

Use 16MB of framebuffer memory at the top of RAM

Signed-off-by Cyril Bur <[email protected]>
---
 arch/arm/boot/dts/aspeed-bmc-opp-barreleye.dts | 11 +++++++++++
 arch/arm/boot/dts/aspeed-bmc-opp-firestone.dts | 11 +++++++++++
 arch/arm/boot/dts/aspeed-bmc-opp-garrison.dts  | 11 +++++++++++
 arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts  | 11 +++++++++++
 4 files changed, 44 insertions(+)
arch/arm/boot/dts/aspeed-bmc-opp-barreleye.dts +11 -0
index 353033b..0f1ff31 100644
@@ -20,6 +20,17 @@
 		reg = <0x40000000 0x20000000>;
 	};
 
+	reserved-memory {
+		#address-cells = <1>;
+		#size-cells = <1>;
+		ranges;
+
+		vga_memory: framebuffer@5f000000 {
+			no-map;
+			reg = <0x5f000000 0x01000000>; /* 16MB */
+		};
+	};
+
 	ahb {
 		bmc_pnor: fmc@1e620000 {
 			reg = < 0x1e620000 0x94
arch/arm/boot/dts/aspeed-bmc-opp-firestone.dts +11 -0
index ee7b569..bd297d3 100644
@@ -24,6 +24,17 @@
 		reg = <0x40000000 0x20000000>;
 	};
 
+	reserved-memory {
+		#address-cells = <1>;
+		#size-cells = <1>;
+		ranges;
+
+		vga_memory: framebuffer@5f000000 {
+			no-map;
+			reg = <0x5f000000 0x01000000>; /* 16MB */
+		};
+	};
+
 	leds {
 		compatible = "gpio-leds";
arch/arm/boot/dts/aspeed-bmc-opp-garrison.dts +11 -0
index 3f3fd1f..2365483 100644
@@ -20,6 +20,17 @@
 		reg = <0x40000000 0x20000000>;
 	};
 
+	reserved-memory {
+		#address-cells = <1>;
+		#size-cells = <1>;
+		ranges;
+
+		vga_memory: framebuffer@5f000000 {
+			no-map;
+			reg = <0x5f000000 0x01000000>; /* 16MB */
+		};
+	};
+
 	ahb {
 
 		fmc@1e620000 {
arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts +11 -0
index 816c004..e05a56d 100644
@@ -20,6 +20,17 @@
 		reg = <0x40000000 0x20000000>;
 	};
 
+	reserved-memory {
+		#address-cells = <1>;
+		#size-cells = <1>;
+		ranges;
+
+		vga_memory: framebuffer@5f000000 {
+			no-map;
+			reg = <0x5f000000 0x01000000>; /* 16M */
+		};
+	};
+
         leds {
                 compatible = "gpio-leds";
Signature · 1 line
2.10.2
maintainer
#2 lore raw

Re: [PATCH linux 2/2] ARM: dts: aspeed: Reserve framebuffer memory 2400 based opp platforms

Quoted 3 lines · Cyril Bur

Use 16MB of framebuffer memory at the top of RAM

Signed-off-by Cyril Bur <[email protected]>

This looks sane. Did you boot the host to make sure it does what it says on the box?

Cheers,

Joel

maintainer
#3 lore raw
In reply to Cyril Bur
Quoted 6 lines · Cyril Bur

These platforms all have 512MB of memory as opposed to the previously thought 256MB.

Update the device trees accordingly

Signed-off-by Cyril Bur <[email protected]>

Thanks. I tweaked the commit message a little and applied to dev-4.7 as 580004c1267c90ff4a3c82d7c970e93609ae28ed.

Cheers,

Joel

Quoted 66 lines
---
 arch/arm/boot/dts/aspeed-bmc-opp-barreleye.dts | 2 +-
 arch/arm/boot/dts/aspeed-bmc-opp-firestone.dts | 2 +-
 arch/arm/boot/dts/aspeed-bmc-opp-garrison.dts  | 2 +-
 arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts  | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)
arch/arm/boot/dts/aspeed-bmc-opp-barreleye.dts +1 -1
index 802ce95..353033b 100644
@@ -17,7 +17,7 @@
        };
 
        memory {
-               reg = <0x40000000 0x10000000>;
+               reg = <0x40000000 0x20000000>;
        };
 
        ahb {
arch/arm/boot/dts/aspeed-bmc-opp-firestone.dts +1 -1
index 005ff89..ee7b569 100644
@@ -21,7 +21,7 @@
        };
 
        memory {
-               reg = < 0x40000000 0x10000000 >;
+               reg = <0x40000000 0x20000000>;
        };
 
        leds {
arch/arm/boot/dts/aspeed-bmc-opp-garrison.dts +1 -1
index 92b29d1..3f3fd1f 100644
@@ -17,7 +17,7 @@
        };
 
        memory {
-               reg = < 0x40000000 0x10000000 >;
+               reg = <0x40000000 0x20000000>;
        };
 
        ahb {
arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts +1 -2
index acaceda2..816c004 100644
@@ -17,7 +17,7 @@
        };
 
        memory {
-               reg = <0x40000000 0x10000000>;
+               reg = <0x40000000 0x20000000>;
        };
 
         leds {
--

2.10.2

_______________________________________________
openbmc mailing list
[email protected]
https://lists.ozlabs.org/listinfo/openbmc