[U-Boot] [PATCH 10/19] x86: Move ECC initialisation outside RAM initialisation

Sergei Shtylyov sshtylyov at mvista.com
Mon Oct 4 11:59:30 CEST 2010


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.russ at 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


More information about the U-Boot mailing list