From mboxrd@z Thu Jan 1 00:00:00 1970 From: Scott Wood Date: Thu, 27 May 2010 15:10:40 -0500 Subject: [U-Boot] [PATCH] powerpc: add support for the Freescale P1022DS reference board In-Reply-To: <20100527200022.D23CCEAC238@gemini.denx.de> References: <1274392909-16422-1-git-send-email-timur@freescale.com> <20100520223324.50594CCF026@gemini.denx.de> <4BFC1736.5030902@freescale.com> <20100526201014.97886EAC238@gemini.denx.de> <4BFD837D.2040508@freescale.com> <20100527070235.E8812EAC238@gemini.denx.de> <4BFE823A.1080409@freescale.com> <20100527181118.3C9F7EAC238@gemini.denx.de> <4BFEB922.9040106@freescale.com> <20100527194555.CAAD0EAC238@gemini.denx.de> <4BFECDF7.4070405@freescale.com> <20100527200022.D23CCEAC238@gemini.denx.de> Message-ID: <4BFED1C0.1030105@freescale.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 05/27/2010 03:00 PM, Wolfgang Denk wrote: > Dear Timur Tabi, > > In message<4BFECDF7.4070405@freescale.com> you wrote: >> >> What if the mapping is too big? Isn't the whole point of get_ram_size() >> that there is less RAM than we say there is? That is, if there's only 1GB >> of RAM, and we do this: > > get_ram_size() will also detect a number of hardware problems list > stuck / swapped / mirrored address lines etc. (actually a pretty > large percentage of the real world hardware problems). How about splitting out the testing bit into something that will just test a specified range, and report any errors, rather than quietly pretend you have less RAM if the test fails? -Scott