[U-Boot] [PATCH] PowerPC: Change -fpic flag to -fPIC flag in the config.mk

[Scott Wood <scottwood@freescale.com>]

On 04/18/2012 05:07 AM, Joakim Tjernlund wrote:
> Your linker file is buggy I think. I found u-boot-nand_spl.lds, is that the one?

That's the one for the SPL part.

> Check out that files reloc part:
> .reloc : {
> 		_GOT2_TABLE_ = .;
> 		KEEP(*(.got2))
> 		_FIXUP_TABLE_ = .;
> 		KEEP(*(.fixup))
> 	}
> 	__got2_entries = (_FIXUP_TABLE_ - _GOT2_TABLE_) >> 2;
> 	__fixup_entries = (. - _FIXUP_TABLE_) >> 2;
> 
> Compare that with(from u-boot.lds):
>  .reloc   :
>   {
>     _GOT2_TABLE_ = .;
>     KEEP(*(.got2))
>     KEEP(*(.got))
>     PROVIDE(_GLOBAL_OFFSET_TABLE_ = . + 4);
>     _FIXUP_TABLE_ = .;
>     KEEP(*(.fixup))
>   }
>   __got2_entries = ((_GLOBAL_OFFSET_TABLE_ - _GOT2_TABLE_) >> 2) - 1;
>   __fixup_entries = (. - _FIXUP_TABLE_) >> 2;

I notice a difference between u-boot.lds and u-boot-nand.lds -- the
latter (used for the main part of U-Boot when loaded from SPL) has:

>   .reloc   :
>   {
>     KEEP(*(.got))
>     _GOT2_TABLE_ = .;
>     KEEP(*(.got2))
>     _FIXUP_TABLE_ = .;
>     KEEP(*(.fixup))
>   }
>   __got2_entries = (_FIXUP_TABLE_ - _GOT2_TABLE_) >> 2;
>   __fixup_entries = (. - _FIXUP_TABLE_) >> 2;

Is this wrong as well?

-Scott