From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gabor Juhos Date: Fri, 01 Feb 2013 23:17:45 +0100 Subject: [U-Boot] [PATCH v2] MIPS: start.S: unify and simplify reset vector handling In-Reply-To: <1359752134-9547-1-git-send-email-daniel.schwierzeck@gmail.com> References: <1359591991-15156-1-git-send-email-daniel.schwierzeck@gmail.com> <1359752134-9547-1-git-send-email-daniel.schwierzeck@gmail.com> Message-ID: <510C3F09.8030607@openwrt.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Daniel, > From: Daniel Schwierzeck > > Adopt reset vector handling from Yamon. > > Signed-off-by: Daniel Schwierzeck 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