[U-Boot] [PATCH] PowerPC: Move -fPIC flag to common place

[Joakim Tjernlund]

>
> Dear Joakim Tjernlund,
>
> In message <1291642606-4290-1-git-send-email-Joakim.Tjernlund at transmode.se> you wrote:
> > The -fPIC flag belongs with -mrelocatable, move it there.
> > Also change -fPIC to -fpic as this produces smaller
> > binaries.
> > However, currently -mrelocatable promotes -fpic to -fPIC, a
> > fix for this is in upcoming gcc 4.6 or you can apply this small
> > patch to gcc:
>
> Unfortunately this patch breaks a number of boards:
>
> Bisecting: 0 revisions left to test after this (roughly 0 steps)
> [33ee4c92339ee386662c0ee2d221098c5cc8b07e] PowerPC: Move -fPIC flag to
> common place
>
>
> They fail to build like this:
>
> ...
> Configuring for TQM855L board...
> /work/tmp-8xx/u-boot.lds:74 cannot move location counter backwards (from 4000800c to 40008000)
> make: *** [/work/tmp-8xx/u-boot] Error 1
> ppc_6xx-size: '/work/tmp-8xx/u-boot': No such file
> ...
> Configuring for TQM860L board...
> /work/tmp-8xx/u-boot.lds:74 cannot move location counter backwards (from 4000802c to 40008000)
> make: *** [/work/tmp-8xx/u-boot] Error 1
> ppc_6xx-size: '/work/tmp-8xx/u-boot': No such file
>
> It seems this change increases the code size somehow - which I would
> not expect from your comment above.  Please have a look.

Very strange, I had a quick look(that is all I can do ATM). It appers
that 8xx is the only ppc arch which doesn't have its own u-boot.lds under
arch/powerpc/cpu/mpc8xx, is that intentional? Is it only 8xx that has this
problem?
My guess would be that 8xx missed out on an earlier patch
http://git.denx.de/?p=u-boot.git;a=commitdiff;h=337f5f50f539cc1ea1e0533c096e237228f12cae

 Jocke