From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Fri, 11 Jan 2008 07:44:03 +0100 Subject: [U-Boot-Users] [PATCH] Fix "i2c sdram" command for DDR2 DIMMs In-Reply-To: <4786E13B.3030105@acm.org> References: <4786E13B.3030105@acm.org> Message-ID: <200801110744.04262.sr@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > --- > 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 at denx.de =====================================================================