[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