From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Roese <sr@denx.de>
Date: Wed, 30 Jul 2008 14:08:42 +0200
Subject: [U-Boot-Users] [PATCH] bdinfo not showing correct info about
	memory size
In-Reply-To: <20080730115412.1A91E248BF@gemini.denx.de>
References: <20080730115412.1A91E248BF@gemini.denx.de>
Message-ID: <200807301408.42246.sr@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Wednesday 30 July 2008, Wolfgang Denk wrote:
> => bdi
> memstart    = 0x00000000
> memsize     = 0x100000000
> ...
>
> As you can see, this board has 4 GB of RAM and everything works  just
> fine.

Yes. But unfortunately not for all boards. As it seems the patch 
b57ca3e128cc10a133ba79bc7ec3e7b50e7c8fbe [Change bd/gd memsize/ram_size to be 
phys_size_t.] introduced %llX unconditionally for all non-ARM boards. And 
many boards don't have support for 64bit printf formats enabled.

So what to do? Enable 64bit format on all boards? Or make this size printing 
conditionally somehow?

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================