Felix,
On Tuesday 08 September 2009 12:05:30 Felix Radensky wrote:
Most likely historic reasons. I added Grant Erickson to Cc, IIRC he added/tested this ecc_init() code. Maybe he can shed some more light into this.
But from looking at it, it seems to me that we should get rid of one of those routines. program_ecc_addr() seems more generic to me. Patches welcome. ;)
OK, unless Grant objects I'll try to cook up a patch, most probably next week. I have 405EX and 460EX boards with ECC enabled soldered DDR2, so I'll be able to test these cases. It would be great if someone could test SPD case, to see that I didn't break anything.
Great. I'll test it on Katmai (440SPe DDR2 with ECC enabled).
Thanks.
Cheers, 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