On Monday 26 January 2009, carolyn.j.sm...@tektronix.com wrote: > This fixes the initialization of the SDRAM_CODT register in the ppc4xx DDR2 > initialization code. The changes were discussed with Stefan Roese and Adam > Graham and tested by Adam.
General comment: The line "The changes were discussed..." should be placed below the "---" since it doesn't really belong to the patch description. > Signed-off-by: Carolyn Smith <carolyn.sm...@tektronix.com> > --- > diff -w -uprN old/cpu/ppc4xx/44x_spd_ddr2.c new/cpu/ppc4xx/44x_spd_ddr2.c > --- old/cpu/ppc4xx/44x_spd_ddr2.c 2009-01-20 13:00:33.000000000 -0800 > +++ new/cpu/ppc4xx/44x_spd_ddr2.c 2009-01-23 16:00:49.000000000 -0800 > @@ -1104,11 +1104,14 @@ static void program_codt(unsigned long * > * Set the SDRAM Controller On Die Termination Register > > *-----------------------------------------------------------------*/ > mfsdram(SDRAM_CODT, codt); > - codt |= (SDRAM_CODT_IO_NMODE > - & (~SDRAM_CODT_DQS_SINGLE_END > - & ~SDRAM_CODT_CKSE_SINGLE_END > - & ~SDRAM_CODT_FEEBBACK_RCV_SINGLE_END > - & ~SDRAM_CODT_FEEBBACK_DRV_SINGLE_END)); > + > + codt |= SDRAM_CODT_IO_NMODE; > + > + codt &= ~(SDRAM_CODT_DQS_SINGLE_END | > + SDRAM_CODT_CKSE_SINGLE_END | > + SDRAM_CODT_FEEBBACK_RCV_SINGLE_END | > + SDRAM_CODT_FEEBBACK_DRV_SINGLE_END); > + I don't think that this is correct. As I understood from Adam, those bits it questions are undefined/reserevd on all 4xx PPC's. So we should better remove the code referencing those values. Here is what Adam wrote: > It is in fact the case for all the AMCC 4xx SoC chips that have the IBM > SDRAM memory controller core IP, the SDRAM_CODT register, bits 29 and 30 > are reserved bits. Thanks Carolyn for finding this issue. We will > update the 44x_spd_ddr2.c file and send out a patch shortly for this > SDRAM_CODT register bits 29-30 issue. Adam, do you still plan to send a patch for this? Or did I misunderstand you and the patch above is ok? 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: off...@denx.de ===================================================================== _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot