[U-Boot] [PATCH 1/2 for 2010.12 ] Add support PLATFORM_LDFLAGS to cmd_link_o_target

[Mike Frysinger]

On Saturday, December 25, 2010 18:02:26 Nobuhiro Iwamatsu wrote:
> Current cmd_link_o_target function in config.mk does not support the set
> of the endian.
> 
> Some architecture is bi-endian (e.g. mips and sh).
> Therefore, there is case supporting big endian and little endian
> with one toolchain.
> For example, when user builds target of big endian in host of little
> endian, they need set endian.
> 
>  # If the list of objects to link is empty, just create an empty built-in.o
>  cmd_link_o_target = $(if $(strip $1),\
> -		      $(LD) -r -o $@ $1 ,\
> +		      $(LD) $(ENDIANNESS) -r -o $@ $1 ,\

i dont think we should start declaring random new variables with specific 
purposes.  better to split the "u-boot final" LDFLAGS out into their own 
variable (LDFLAGS_u-boot) and keep LDFLAGS as a "these are the flags that need 
to be used with $(LD)".
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20101226/ddb419ca/attachment.pgp