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

Mike Frysinger
2010-12-29T08:35:39Z

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