[U-Boot] [PATCH 12/26 v2][NEXT] ARM: add relocation support

Albert ARIBAUD albert.aribaud at free.fr
Sun Sep 19 10:28:12 CEST 2010


Le 17/09/2010 13:10, Heiko Schocher a écrit :

> diff --git a/arch/arm/lib/board.c b/arch/arm/lib/board.c
> index 54519b0..88c6427 100644
> --- a/arch/arm/lib/board.c
> +++ b/arch/arm/lib/board.c

> +void board_init_f (ulong bootflag)

A recursive grep found 'bootflag' nowhere else in the whole u-boot code; 
it always receives the value 0 from start.S; and is not used in 
board_init_f. What was it supposed to be for?

(if it has no use, no need to remove it: I am preparing a patch which 
will need to pass an argument from _start to board_init_f, and bootflag 
should be alright).

> +	relocate_code (addr_sp, id, addr);
> +	/* NOTREACHED - relocate_code() does not return */
> +}

Why call relocate_code() here when board_init_f could simply return to 
it? (passing a success/failure through a return value if necessary).

Amicalement,
-- 
Albert.


More information about the U-Boot mailing list