[U-Boot] powerpc/mpc8xxx failed to compile: operand out of range

["Jérôme Arzel" <jerome.arzel@gnuside.com>]

On 05/24/2013 10:13:55 PM, Scott Wood wrote:
> 
> On 05/23/2013 04:52:26 AM, J?r?me Arzel wrote:
> > Hi all,
> > 
> > I have an issue when I compile U-Boot for my target machine
> > (P1022DS,
> > 36-bit).
> > Here is the error message:
> > 
> > release.S: Assembler messages:
> > release.S:154: Error: operand out of range (0xfffff144 is not
> > between
> > 0x00000000 and 0x0000ffff)
> > release.S:286: Error: operand out of range (0xfffff144 is not
> > between
> > 0x00000000 and 0x0000ffff)
> > release.S:311: Error: operand out of range (0xfffff140 is not
> > between
> > 0x00000000 and 0x0000ffff)
> > 
> > (release.S is located to arch/powerpc/cpu/mpc85xx/release.S)
> > 
> > And here is the code for the first error (line 150):
> > 
> > #define toreset(x) (x - __secondary_start_page + 0xfffff000)
> > 
> >         /* get our PIR to figure out our table entry */
> >         lis     r3,toreset(__spin_table_addr)@h
> >         ori     r3,r3,toreset(__spin_table_addr)@l
> > 
> > I don't really know why it doesn't work, but I think
> > that "ori" is inappropried, the immediate value must be a 16-bit
> > value.
> 
> The @l means take the low 16 bits of the constant.  Likewise, the @h
> in
> the lis takes the upper 16 bits.
> 
> Could you try to see what that instruction looks like after the
> preprocessor stage (i.e. use -E rather than -c in gcc)?
> 

OK, I tried and unsurprisingly it's:

 lis 3,(__spin_table_addr - __secondary_start_page + 0xfffff000)@h
 ori 3,3,(__spin_table_addr - __secondary_start_page + 0xfffff000)@l
 lwz 3,0(3)

If the @l takes the low 16 bits of
(__spin_table_addr - __secondary_start_page + 0xfffff000), so
the immediate value should not be 0xfffff144, but 0xf144.
The error is maybe in GCC...


J?r?me