[U-Boot] [PATCH v2] MIPS: start.S: unify and simplify reset vector handling
Gabor Juhos
juhosg at openwrt.org
Fri Feb 1 23:17:45 CET 2013
Hi Daniel,
> From: Daniel Schwierzeck <daniel.schwierzeck at gmail.com>
>
> Adopt reset vector handling from Yamon.
>
> Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck at gmail.com>
Looks good, I have only one minor comment below.
> ---
> Changes for v2:
> - optimized U-Boot entry point on MIPS32
> - fix indentation of .org
>
> ---
> arch/mips/cpu/mips32/start.S | 186 +++++++++----------------------------------
> arch/mips/cpu/mips64/start.S | 47 +++++------
> 2 files changed, 57 insertions(+), 176 deletions(-)
>
> diff --git a/arch/mips/cpu/mips32/start.S b/arch/mips/cpu/mips32/start.S
> index 65acf7d..7373d4e 100644
> --- a/arch/mips/cpu/mips32/start.S
> +++ b/arch/mips/cpu/mips32/start.S
> @@ -47,19 +47,16 @@
> .set pop
> .endm
>
> -#define RVECENT(f,n) \
> - b f; nop
> -#define XVECENT(f,bev) \
> - b f ; \
> - li k0,bev
> -
> .set noreorder
>
> .globl _start
> .text
> _start:
> - RVECENT(reset,0) # U-boot entry point
> - RVECENT(reset,1) # software reboot
> + /* U-boot entry point */
> + b reset
> + nop
> +
> + .org 0x10
> #ifdef CONFIG_SYS_XWAY_EBU_BOOTCFG
The .org should be within the ifdef. Although it does not change the generated
binary code, but logically it would be a more correct place. Sorry, I should
have mentioned that in the previous mail.
-Gabor
More information about the U-Boot
mailing list