[PATCH] ARM: mvebu: fix RAM size for Armada XP board DB-MV784MP-GP

On Wed, Mar 13, 2013 at 03:37:00PM +0100, Gregory CLEMENT wrote:

On 03/13/2013 03:30 PM, Sergei Shtylyov wrote:

quoted

Hello.

On 12-03-2013 19:12, Jason Cooper wrote:

quoted

The board is supplied with a 4GB RAM module.  This value can be
overridden by the bootloader based on probed memory size.  We set it to
a reasonable value here.

quoted

Signed-off-by: Jason Cooper <redacted>
---
As promised, catching this fix in the -rc cycle.

quoted

For those not familiar, earlier versions of the patch adding this board listed
3GB because that is all that was visible.  I mistaken applied v3 of the patch
instead of v4 which properly listed 4GB.  This patch cleans up my error.

quoted

  arch/arm/boot/dts/armada-xp-gp.dts | 9 ++++-----
  1 file changed, 4 insertions(+), 5 deletions(-)

quoted

diff --git a/arch/arm/boot/dts/armada-xp-gp.dts b/arch/arm/boot/dts/armada-xp-gp.dts
index 1c8afe2..4a2776a 100644
--- a/arch/arm/boot/dts/armada-xp-gp.dts
+++ b/arch/arm/boot/dts/armada-xp-gp.dts

@@ -28,12 +28,11 @@
  		device_type = "memory";

  		/*
-		 * 4 GB of plug-in RAM modules by default but only 3GB
-		 * are visible, the amount of memory available can be
-		 * changed by the bootloader according the size of the
-		 * module actually plugged
+		 * 4 GB of plug-in RAM modules by default. The amount of memory
+		 * available can be changed by the bootloader according the

    "According to the size".

quoted

+		 * size of the module actually plugged
  		 */
-		reg = <0x00000000 0xC0000000>;
+		reg = <0x00000000 0xD0000000>;

    But this is not 4G?

You're totally right!
It should be reg = <0x00000000 0xF0000000>;

$ printf "0x%08x\n" `echo "4 * (1024 ^ 3)" | bc`
0x100000000

?

thx,

Jason.