[U-Boot] [PATCH 12/13] add support for function-sections

[Mike Frysinger <vapier@gentoo.org>]

On Wednesday, December 29, 2010 03:28:51 Michael Schwingen wrote:
> Am 12/29/2010 05:47 AM, schrieb Mike Frysinger:
> > On Tuesday, December 28, 2010 07:43:30 Michael Schwingen wrote:
> >> Signed-off-by: Michael Schwingen <michael@schwingen.org>
> >> ---
> >> 
> >>  config.mk                |    6 ++++++
> >>  include/configs/actux1.h |    1 +
> >>  include/configs/actux2.h |    1 +
> >>  include/configs/actux3.h |    1 +
> >>  include/configs/actux4.h |    1 +
> >>  5 files changed, 10 insertions(+), 0 deletions(-)
> >> 
> >> diff --git a/config.mk b/config.mk
> >> index 66f8fe6..f7c7760 100644
> >> --- a/config.mk
> >> +++ b/config.mk
> >> @@ -152,6 +152,9 @@ endif
> >> 
> >>  RELFLAGS= $(PLATFORM_RELFLAGS)
> >>  DBGFLAGS= -g # -DDEBUG
> >>  OPTFLAGS= -Os #-fomit-frame-pointer
> >> 
> >> +ifeq ($(CONFIG_USE_FUNCTION_SECTIONS),y)
> >> +OPTFLAGS += -ffunction-sections
> >> +endif
> > 
> > you can already do this with board config.mk and PLATFORM_RELFLAGS.  no
> > need to introduce some random new config name for a random compiler
> > flag.
> 
> OK, I had not seen that possibility.
> Setting a platform variable in a board-dependant way sounds a bit wrong,
> but it works.

you should be appending it, not setting it
-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/20101229/658d61f5/attachment.pgp