From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michael Schwingen Date: Wed, 26 Jan 2011 00:37:16 +0100 Subject: [U-Boot] [PATCH 07/13] update/fix AcTux1 board In-Reply-To: <20110125204449.69815B187@gemini.denx.de> References: <1294062338-21084-1-git-send-email-michael@schwingen.org> <1294062338-21084-8-git-send-email-michael@schwingen.org> <20110125204449.69815B187@gemini.denx.de> Message-ID: <4D3F5EAC.9000002@discworld.dascon.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Am 01/25/2011 09:44 PM, schrieb Wolfgang Denk: > >> diff --git a/board/actux1/config.mk b/board/actux1/config.mk >> index 88634f7..a370337 100644 >> --- a/board/actux1/config.mk >> +++ b/board/actux1/config.mk >> @@ -1,6 +1,3 @@ >> -CONFIG_SYS_TEXT_BASE = 0x00e00000 >> - >> -# include NPE ethernet driver >> -BOARDLIBS = arch/arm/cpu/ixp/npe/libnpe.o >> - >> LDSCRIPT := $(SRCTREE)/board/$(BOARDDIR)/u-boot.lds >> +PLATFORM_RELFLAGS += -ffunction-sections -fdata-sections >> +PLATFORM_LDFLAGS += --gc-sections > Can we please get rid of this file completely? If you have a better proposal where to add -ffunction-sections and --gc-sections? In my previous patch, I proposed a generic config switch (which added those flags in the generic Makefile) for that, but the only comment I got was from Mike Frysinger, stating that an extra config switch was unnecessary when I could simply set those variables in the board config.mk. Those two switches make for a significant reduction in code size (~61000 bytes, nearly one flash sector!), so dropping them is no real option. Which solution would you prefer? cu Michael