aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJan Kiszka <jan.kiszka@siemens.com>2018-10-02 09:28:00 +0200
committerKonstantin Porotchkin <kostap@marvell.com>2018-10-03 15:31:19 +0300
commitc0f73edc9b04023a24a1e2103b442a94297921a7 (patch)
treef8c437e003beb8d6f87912a9accccad6f4401b40
parent37c4341b1064de53c4271c1459c489ea3cb9d878 (diff)
downloadtrusted-firmware-a-c0f73edc9b04023a24a1e2103b442a94297921a7.tar.gz
tools: Fix broken object compilation rules
As these rules depend on non-existing headers as well (likely copy & pasted from fiptool), they never matched, and the built-in rules were used. That led to random breakages when e.g. CPPFLAGS was suddenly evaluated and contained invalid options. For the stm32image, this reveals that we were relying on the built-in rules by passing -D_GNU_SOURCE via CPPFLAGS, rather than using CFLAGS as used in the local rule. Fix that as well. Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
-rw-r--r--tools/doimage/Makefile2
-rw-r--r--tools/stm32image/Makefile5
2 files changed, 3 insertions, 4 deletions
diff --git a/tools/doimage/Makefile b/tools/doimage/Makefile
index bc74369ff5..9cec681a49 100644
--- a/tools/doimage/Makefile
+++ b/tools/doimage/Makefile
@@ -39,7 +39,7 @@ ${PROJECT}: ${OBJECTS} Makefile
@echo "Built $@ successfully"
@echo
-%.o: %.c %.h Makefile
+%.o: %.c Makefile
@echo " CC $<"
${Q}${CC} -c ${CFLAGS} ${INCLUDE_PATHS} $< -o $@
diff --git a/tools/stm32image/Makefile b/tools/stm32image/Makefile
index 80dfbecf06..a593d31637 100644
--- a/tools/stm32image/Makefile
+++ b/tools/stm32image/Makefile
@@ -12,8 +12,7 @@ PROJECT := stm32image${BIN_EXT}
OBJECTS := stm32image.o
V := 0
-override CPPFLAGS += -D_GNU_SOURCE
-CFLAGS := -Wall -Werror -pedantic -std=c99
+CFLAGS := -Wall -Werror -pedantic -std=c99 -D_GNU_SOURCE
ifeq (${DEBUG},1)
CFLAGS += -g -O0 -DDEBUG
else
@@ -39,7 +38,7 @@ ${PROJECT}: ${OBJECTS} Makefile
@echo "Built $@ successfully"
@${ECHO_BLANK_LINE}
-%.o: %.c %.h Makefile
+%.o: %.c Makefile
@echo " CC $<"
${Q}${CC} -c ${CFLAGS} $< -o $@