Hello.

On 04-10-2010 7:37, Graeme Russ wrote:

> To allow for 'load anywhere' images, the %ebp return pointer 'hack' must
> be removed, so we cannot have two 'calls' to get_mem_size

> Signed-off-by: Graeme Russ<graeme.r...@gmail.com>
[...]

> diff --git a/arch/i386/cpu/start.S b/arch/i386/cpu/start.S
> index cb47ce7..9e6e0bf 100644
> --- a/arch/i386/cpu/start.S
> +++ b/arch/i386/cpu/start.S
> @@ -93,6 +93,17 @@ mem_init_ret:
>       jmp     get_mem_size
>   get_mem_size_ret:
>
> +#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?

> +     mov     $init_ecc_ret, %ebp
> +     jmp     init_ecc
> +
> +init_ecc_ret:
> +#endif
> +
>       /* Check we have enough memory for stack */
>       movl    $CONFIG_SYS_STACK_SIZE, %ecx
>       cmpl    %ecx, %eax

WBR, Sergei
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to