[U-Boot] [PATCH] Add LDFLAGS-u-boot variable and move some linker option to this

Mike Frysinger vapier at gentoo.org
Mon Dec 27 17:46:21 CET 2010


On Monday, December 27, 2010 05:47:54 Wolfgang Denk wrote:
> Nobuhiro Iwamatsu wrote:
> > This move linker option used by the last of u-boot in LDFLAGS_u-boot
> > variable. And the option to use in ld uses LDFLAGS variable.
> 
> Can you please explain why this would be needed?

he explained in the previous thread why we need to split things.  there are 
flags that are needed for all linker options and there are flags needed just 
for the final u-boot link.

> > -LDFLAGS += --gc-sections -m elf32bfin
> > +LDFLAGS_u-boot += --gc-sections
> 
> The name "LDFLAGS_u-boot" may actually work here, but iut is
> inconsistent with other such variables names, and using '-' in a name
> is probably not a good idea either.

it isnt inconsistent.  the convention is $(XFLAGS_$(@F)) and we just happen to 
be creating a file here which is named "u-boot".  dashes in variable names 
isnt a problem for make.  otherwise you're basically banning creating files in 
the u-boot tree with dashes in their filenames.
-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/20101227/c9d76dcf/attachment.pgp 


More information about the U-Boot mailing list