[U-Boot] [PATCH] armv7m: Fix larger builds

[Phil Edworthy]

Hi Vikas,

On 26 May 2017 00:58 Vikas MANOCHA wrote:
> On Thursday, May 25, 2017 6:58 AM Phil Edworthy wrote:
> > 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.edworthy at 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.
I'm using Linaro GCC 6.3-2017.02

	b	_main
or
	W(b)	_main
Disassembly is the same:
   0:	e7fe      	b.n	0 <_main>

The problem appears to be that __ASSEMBLY__ is defined and so in
arch/arm/include/asm/unified.h, W(x) does nothing. Having said that,
if I simply change the code to
	b.w	_main
then I get a build error "Error: bad instruction `b.w _main'"

Sorry, I'm not very familiar with ARM asm :)

Thanks
Phil

> Cheers,
> Vikas
> 
> >
> > Any ideas why?
> > Phil
> >
> > > Thanks
> > > Phil
> > >
> > > > Cheers,
> > > > Vikas
> > > >
> > > > >
> > > > >  .globl	c_runtime_cpu_setup
> > > > >  c_runtime_cpu_setup:
> > > > > --
> > > > > 2.7.4