[U-Boot] [PATCH 10/19] x86: Move ECC initialisation outside RAM initialisation
Graeme Russ
graeme.russ at gmail.com
Mon Oct 4 23:49:25 CEST 2010
On Mon, Oct 4, 2010 at 9:21 PM, Wolfgang Denk <wd at denx.de> wrote:
> Dear Graeme Russ,
>
> In message <4CA9A70B.6030302 at 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?
>
Ah yes, now I see - thanks for picking this up. Will fix before applying
Regards,
Graeme
More information about the U-Boot
mailing list