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

Wolfgang Denk wd at denx.de
Thu Jul 15 11:07:06 CEST 2010


Dear Timur Tabi,

In message <AANLkTim6yjF6ORogtqE0vqiVqZqxu-EkkUhDTmJMgPqu at 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.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Programmer's Lament: (Shakespeare, Macbeth, Act I, Scene vii)
        "That we but teach bloody instructions,
        which, being taught, return to plague the inventor..."


More information about the U-Boot mailing list