Dear Graeme Russ, In message <4ca9a70b.6030...@gmail.com> you wrote: > > >> +#if CONFIG_SYS_SDRAM_ECC_ENABLE > >> + /* Skip ECC initialization if not starting from cold-reset */ > >> + movl %ebx, %ecx > >> + andl $GD_FLG_COLD_BOOT, %ecx > >> + jz ecc_init_ret > > > > But the label you added is called 'init_ecc_ret'. Are you sure the > > name is correct here? > > Yes, if the board has not been cold-booted (i.e. a new U-Boot image has > been loaded into RAM and executed) we must skip ECC initialization (it > destroys the contents of memory) > > > > >> + mov $init_ecc_ret, %ebp > >> + jmp init_ecc > >> + > >> +init_ecc_ret:
Sergei did not question the code itself, but the possible confusion between "ecc_init_ret" versus "init_ecc_ret", which both were added with the same commit, so it seems likely both should actually be the _same_ name? 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: w...@denx.de Crash programs fail because they are based on the theory that, with nine women pregnant, you can get a baby a month. - Wernher von Braun _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot