From d5cc74ebd86a753547349cefcea3ed552e94c5dd Mon Sep 17 00:00:00 2001 From: Solomon Peachy Date: Thu, 25 Mar 2021 12:14:28 -0400 Subject: build: Get rid of suplerflous '-Os' references in linker calls We should be using global optimization flags, and -Os is already our default. Change-Id: Iaba511b2b4fa2ee0f1eb1ad28024be954f4c5f03 --- firmware/target/hosted/agptek/rocker.make | 4 ++-- firmware/target/hosted/aigo/erosq.make | 4 ++-- firmware/target/hosted/fiio/fiio.make | 4 ++-- firmware/target/hosted/samsungypr/ypr0/ypr0.make | 2 +- firmware/target/hosted/samsungypr/ypr1/ypr1.make | 2 +- firmware/target/hosted/sonynwz/sonynwz.make | 4 ++-- firmware/target/hosted/xduoo/xduoo.make | 4 ++-- tools/root.make | 4 ++-- 8 files changed, 14 insertions(+), 14 deletions(-) diff --git a/firmware/target/hosted/agptek/rocker.make b/firmware/target/hosted/agptek/rocker.make index d159db77f3..117ecb8eac 100644 --- a/firmware/target/hosted/agptek/rocker.make +++ b/firmware/target/hosted/agptek/rocker.make @@ -25,7 +25,7 @@ ifneq (,$(findstring bootloader,$(APPSDIR))) # bootloader build $(BUILDDIR)/bootloader.elf : $$(OBJ) $(FIRMLIB) $(CORE_LIBS) - $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -Os -o $@ $(OBJ) \ + $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -o $@ $(OBJ) \ -L$(BUILDDIR)/firmware -lfirmware \ -L$(BUILDDIR)/lib $(call a2lnk,$(CORE_LIBS)) \ $(LDOPTS) $(GLOBAL_LDOPTS) -Wl,--gc-sections -Wl,-Map,$(BUILDDIR)/bootloader.map @@ -37,7 +37,7 @@ else # rockbox app build $(BUILDDIR)/rockbox.elf : $$(OBJ) $(FIRMLIB) $(VOICESPEEXLIB) $(CORE_LIBS) - $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -Os -o $@ $(OBJ) \ + $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -o $@ $(OBJ) \ -L$(BUILDDIR)/firmware -lfirmware \ -L$(RBCODEC_BLD)/codecs $(call a2lnk, $(VOICESPEEXLIB)) \ -L$(BUILDDIR)/lib $(call a2lnk,$(CORE_LIBS)) \ diff --git a/firmware/target/hosted/aigo/erosq.make b/firmware/target/hosted/aigo/erosq.make index d159db77f3..117ecb8eac 100644 --- a/firmware/target/hosted/aigo/erosq.make +++ b/firmware/target/hosted/aigo/erosq.make @@ -25,7 +25,7 @@ ifneq (,$(findstring bootloader,$(APPSDIR))) # bootloader build $(BUILDDIR)/bootloader.elf : $$(OBJ) $(FIRMLIB) $(CORE_LIBS) - $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -Os -o $@ $(OBJ) \ + $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -o $@ $(OBJ) \ -L$(BUILDDIR)/firmware -lfirmware \ -L$(BUILDDIR)/lib $(call a2lnk,$(CORE_LIBS)) \ $(LDOPTS) $(GLOBAL_LDOPTS) -Wl,--gc-sections -Wl,-Map,$(BUILDDIR)/bootloader.map @@ -37,7 +37,7 @@ else # rockbox app build $(BUILDDIR)/rockbox.elf : $$(OBJ) $(FIRMLIB) $(VOICESPEEXLIB) $(CORE_LIBS) - $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -Os -o $@ $(OBJ) \ + $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -o $@ $(OBJ) \ -L$(BUILDDIR)/firmware -lfirmware \ -L$(RBCODEC_BLD)/codecs $(call a2lnk, $(VOICESPEEXLIB)) \ -L$(BUILDDIR)/lib $(call a2lnk,$(CORE_LIBS)) \ diff --git a/firmware/target/hosted/fiio/fiio.make b/firmware/target/hosted/fiio/fiio.make index d159db77f3..117ecb8eac 100644 --- a/firmware/target/hosted/fiio/fiio.make +++ b/firmware/target/hosted/fiio/fiio.make @@ -25,7 +25,7 @@ ifneq (,$(findstring bootloader,$(APPSDIR))) # bootloader build $(BUILDDIR)/bootloader.elf : $$(OBJ) $(FIRMLIB) $(CORE_LIBS) - $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -Os -o $@ $(OBJ) \ + $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -o $@ $(OBJ) \ -L$(BUILDDIR)/firmware -lfirmware \ -L$(BUILDDIR)/lib $(call a2lnk,$(CORE_LIBS)) \ $(LDOPTS) $(GLOBAL_LDOPTS) -Wl,--gc-sections -Wl,-Map,$(BUILDDIR)/bootloader.map @@ -37,7 +37,7 @@ else # rockbox app build $(BUILDDIR)/rockbox.elf : $$(OBJ) $(FIRMLIB) $(VOICESPEEXLIB) $(CORE_LIBS) - $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -Os -o $@ $(OBJ) \ + $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -o $@ $(OBJ) \ -L$(BUILDDIR)/firmware -lfirmware \ -L$(RBCODEC_BLD)/codecs $(call a2lnk, $(VOICESPEEXLIB)) \ -L$(BUILDDIR)/lib $(call a2lnk,$(CORE_LIBS)) \ diff --git a/firmware/target/hosted/samsungypr/ypr0/ypr0.make b/firmware/target/hosted/samsungypr/ypr0/ypr0.make index 67ff326de0..3583293a0f 100644 --- a/firmware/target/hosted/samsungypr/ypr0/ypr0.make +++ b/firmware/target/hosted/samsungypr/ypr0/ypr0.make @@ -14,7 +14,7 @@ SIMFLAGS += $(INCLUDES) $(DEFINES) -DHAVE_CONFIG_H $(GCCOPTS) .SECONDEXPANSION: # $$(OBJ) is not populated until after this $(BUILDDIR)/rockbox.elf : $$(OBJ) $(FIRMLIB) $(VOICESPEEXLIB) $(CORE_LIBS) - $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -Os -o $@ $(OBJ) \ + $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -o $@ $(OBJ) \ -L$(BUILDDIR)/firmware -lfirmware \ -L$(RBCODEC_BLD)/codecs $(call a2lnk, $(VOICESPEEXLIB)) \ -L$(BUILDDIR)/lib $(call a2lnk,$(CORE_LIBS)) \ diff --git a/firmware/target/hosted/samsungypr/ypr1/ypr1.make b/firmware/target/hosted/samsungypr/ypr1/ypr1.make index 67ff326de0..3583293a0f 100644 --- a/firmware/target/hosted/samsungypr/ypr1/ypr1.make +++ b/firmware/target/hosted/samsungypr/ypr1/ypr1.make @@ -14,7 +14,7 @@ SIMFLAGS += $(INCLUDES) $(DEFINES) -DHAVE_CONFIG_H $(GCCOPTS) .SECONDEXPANSION: # $$(OBJ) is not populated until after this $(BUILDDIR)/rockbox.elf : $$(OBJ) $(FIRMLIB) $(VOICESPEEXLIB) $(CORE_LIBS) - $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -Os -o $@ $(OBJ) \ + $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -o $@ $(OBJ) \ -L$(BUILDDIR)/firmware -lfirmware \ -L$(RBCODEC_BLD)/codecs $(call a2lnk, $(VOICESPEEXLIB)) \ -L$(BUILDDIR)/lib $(call a2lnk,$(CORE_LIBS)) \ diff --git a/firmware/target/hosted/sonynwz/sonynwz.make b/firmware/target/hosted/sonynwz/sonynwz.make index 6c58e1a709..bf2d0aa53c 100644 --- a/firmware/target/hosted/sonynwz/sonynwz.make +++ b/firmware/target/hosted/sonynwz/sonynwz.make @@ -45,7 +45,7 @@ $(BUILDDIR)/bootloader.fnt: $(BL_FONT) $(TOOLS) $(call PRINTS,CONVBDF $(subst $(ROOTDIR)/,,$<))$(TOOLSDIR)/convbdf -l $(MAXCHAR) -f -o $@ $< $(BUILDDIR)/bootloader.elf : $$(OBJ) $(FIRMLIB) $(CORE_LIBS) - $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -Os -o $@ $(OBJ) \ + $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -o $@ $(OBJ) \ -L$(BUILDDIR)/firmware -lfirmware \ -L$(BUILDDIR)/lib $(call a2lnk,$(CORE_LIBS)) \ $(LDOPTS) $(GLOBAL_LDOPTS) -Wl,--gc-sections -Wl,-Map,$(BUILDDIR)/bootloader.map @@ -69,7 +69,7 @@ $(BUILDDIR)/$(BINARY): $(BUILDDIR)/bootloader.elf $(BUILDDIR)/bootloader.fnt else # bootloader $(BUILDDIR)/rockbox.elf : $$(OBJ) $(FIRMLIB) $(VOICESPEEXLIB) $(CORE_LIBS) - $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -Os -o $@ $(OBJ) \ + $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -o $@ $(OBJ) \ -L$(BUILDDIR)/firmware -lfirmware \ -L$(RBCODEC_BLD)/codecs $(call a2lnk, $(VOICESPEEXLIB)) \ -L$(BUILDDIR)/lib $(call a2lnk,$(CORE_LIBS)) \ diff --git a/firmware/target/hosted/xduoo/xduoo.make b/firmware/target/hosted/xduoo/xduoo.make index d159db77f3..117ecb8eac 100644 --- a/firmware/target/hosted/xduoo/xduoo.make +++ b/firmware/target/hosted/xduoo/xduoo.make @@ -25,7 +25,7 @@ ifneq (,$(findstring bootloader,$(APPSDIR))) # bootloader build $(BUILDDIR)/bootloader.elf : $$(OBJ) $(FIRMLIB) $(CORE_LIBS) - $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -Os -o $@ $(OBJ) \ + $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -o $@ $(OBJ) \ -L$(BUILDDIR)/firmware -lfirmware \ -L$(BUILDDIR)/lib $(call a2lnk,$(CORE_LIBS)) \ $(LDOPTS) $(GLOBAL_LDOPTS) -Wl,--gc-sections -Wl,-Map,$(BUILDDIR)/bootloader.map @@ -37,7 +37,7 @@ else # rockbox app build $(BUILDDIR)/rockbox.elf : $$(OBJ) $(FIRMLIB) $(VOICESPEEXLIB) $(CORE_LIBS) - $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -Os -o $@ $(OBJ) \ + $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -o $@ $(OBJ) \ -L$(BUILDDIR)/firmware -lfirmware \ -L$(RBCODEC_BLD)/codecs $(call a2lnk, $(VOICESPEEXLIB)) \ -L$(BUILDDIR)/lib $(call a2lnk,$(CORE_LIBS)) \ diff --git a/tools/root.make b/tools/root.make index 296ca849de..2c6c8474e7 100644 --- a/tools/root.make +++ b/tools/root.make @@ -260,7 +260,7 @@ $(LINKROM): $(ROMLDS) # Note: make sure -Wl,--gc-sections comes before -T in the linker options. # Having the latter first caused crashes on (at least) mini2g. $(BUILDDIR)/rockbox.elf : $$(OBJ) $(FIRMLIB) $(VOICESPEEXLIB) $(CORE_LIBS) $$(LINKRAM) - $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -Os -nostdlib -o $@ $(OBJ) \ + $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -nostdlib -o $@ $(OBJ) \ -L$(BUILDDIR)/firmware \ -L$(RBCODEC_BLD)/codecs $(call a2lnk, $(VOICESPEEXLIB)) \ -L$(BUILDDIR)/lib $(call a2lnk, $(CORE_LIBS)) \ @@ -268,7 +268,7 @@ $(BUILDDIR)/rockbox.elf : $$(OBJ) $(FIRMLIB) $(VOICESPEEXLIB) $(CORE_LIBS) $$(LI -Wl,-Map,$(BUILDDIR)/rockbox.map $(BUILDDIR)/rombox.elf : $$(OBJ) $(FIRMLIB) $(VOICESPEEXLIB) $(CORE_LIBS) $$(LINKROM) - $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -Os -nostdlib -o $@ $(OBJ) \ + $(call PRINTS,LD $(@F))$(CC) $(GCCOPTS) -nostdlib -o $@ $(OBJ) \ -L$(BUILDDIR)/firmware \ -L$(RBCODEC_BLD)/codecs $(call a2lnk, $(VOICESPEEXLIB)) \ -L$(BUILDDIR)/lib $(call a2lnk, $(CORE_LIBS)) \ -- cgit v1.2.3