[U-Boot] [PATCH 8/8] powerpc/85xx: Add memory test feature for mpc85xx.

[York Sun <yorksun@freescale.com>]

Wolfgang,

On Thu, 2010-07-15 at 11:07 +0200, Wolfgang Denk wrote: 
> Dear Timur Tabi,
> 
> In message <AANLkTim6yjF6ORogtqE0vqiVqZqxu-EkkUhDTmJMgPqu@mail.gmail.com> you wrote:
> > 
> > > NAK.
> > >
> > > Please do not reinvent the wheel and add yet another meory test. Use
> > > one of the existing memory tests we already have -
> > > post/drivers/memory.c comes to mind.
> > 
> > That code is not capable of testing more than 2GB of RAM.  It assumes
> > that memory has already been initialized, but on PowerPC parts, we
> > only ever map 2GB of DDR.  We need a memory test that covers all of
> > DDR, but that requries PowerPC-specific code to create sliding TLB
> > windows during the memory test.
> 
> Then please extend the existing code and add the missing features.
> 
> We already have too many different implementations of a memory test in
> U-Boot, and I will not accept adding yet another one.
> 
I can reuse your testing code but have to move the desired code out of
memory.c file to avoid the need for CONFIG_POST and
CONFIG_SYS_POST_MEMORY. I also add a progress indicator. My testing
target is 2GB at a time, up to physically memory size which is easily
over 8GB. Without progress indicator, it feels hung when it is actually
running.

Please take a look at the patch below.

Regards,

York