On Friday 11 January 2008, Larry Johnson wrote:
Many of the SPD bytes for DDR2 SDRAM are not interpreted correctly by the "i2c sdram" command. This patch provides correct alternative interpretations when DDR2 memory is detected.
Signed-off-by: Larry Johnson lrj@acm.org
Hi Wolfgang and Stefan and group,
In this patch, I have fixed the DDR2 output for the SPD bytes currently displayed by the "i2c sdram". I have not added any display for the additional bytes defined in DDR2; I may do that later when I have time.
I don't have any way of testing the patched code for non-DDR2 memory, but if I force the code to treat the DDR2 memory type as "unknown", I get the same values as before the patch.
I'll give it a try on a DDR system in a short while.
Stefan, the added size of this causes a section overlap on the Luan board, and makes the existing overlap on the Ocotea board worse.
This should not be a real problem with the bss NOLOAD patch applied:
[PATCH] Fix linker scripts: add NOLOAD atribute to .bss/.sbss sections
Wolfgang: What's the status with this patch?
If memory size is critical, I could redo this patch using conditional compilation for different memory types, though I believe this would add complexity and reduce flexibility. If you would prefer this, please let me know.
Let's wait on the results with the NOLOAD patch first.
Thanks.
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@denx.de =====================================================================