Hi Phil, > -----Original Message----- > From: Phil Edworthy [mailto:phil.edwor...@renesas.com] > Sent: Thursday, May 25, 2017 6:58 AM > To: Vikas MANOCHA <vikas.mano...@st.com>; Albert Aribaud > <albert.u.b...@aribaud.net> > Cc: Tom Rini <tr...@konsulko.com>; Kamil Lulko <kamil.lu...@gmail.com>; > u-boot@lists.denx.de > Subject: RE: [PATCH] armv7m: Fix larger builds > > Hi Vikas, > > On 25 May 2017 10:16 Phil Edworthy wrote: > > > On 24 May 2017 18:32 Vikas MANOCHA wrote: > > > Hi Phil, > > > > > > > On Wednesday, May 24, 2017 7:34 AM Phil Edworthy wrote: > > > > The branch instruction only has an 11-bit relative target address, > > > > which is > > > sometimes not enough. > > > > > > > > Signed-off-by: Phil Edworthy <phil.edwor...@renesas.com> > > > > --- > > > > arch/arm/cpu/armv7m/start.S | 3 ++- > > > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > > > > > diff --git a/arch/arm/cpu/armv7m/start.S > > b/arch/arm/cpu/armv7m/start.S > > > > index 49f2720..d79adb5 100644 > > > > --- a/arch/arm/cpu/armv7m/start.S > > > > +++ b/arch/arm/cpu/armv7m/start.S > > > > @@ -8,7 +8,8 @@ > > > > .globl reset > > > > .type reset, %function > > > > reset: > > > > - b _main > > > > + ldr r0, =_main > > > > + mov pc, r0 > > > > > > How about using W(b) for wider range ? > > Yes, that makes better sense! > Hmm, if I use W(b) it complains with: > arch/arm/cpu/armv7m/start.S:14:(.text+0x0): relocation truncated to fit: > R_ARM_THM_JUMP11 against symbol `_main' defined in > .text section in arch/arm/lib/built-in.o
Seems like the generated branch instruction is still 16 bit, you can check the disassembly. Which compiler & version you are using ? Are you getting the same issue with "b _main" also. If no, compare the disassembly. Cheers, Vikas > > Any ideas why? > Phil > > > Thanks > > Phil > > > > > Cheers, > > > Vikas > > > > > > > > > > > .globl c_runtime_cpu_setup > > > > c_runtime_cpu_setup: > > > > -- > > > > 2.7.4 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
