From 317523ea7616331f189596a0749fd2a627d68756 Mon Sep 17 00:00:00 2001 From: Sanjeev N Date: Tue, 9 Jul 2019 16:42:16 +0530 Subject: [PATCH] uboot: Treat warnings as errors To treat warnings as errors, enable Werror in CFLAGS. sysupgrade-helper will compile separately only for 64 bit target, since it has architecture specific registers which throws warning on 32 bit compilation. For 32 bit target sysupgrade tools will compile as part of uboot-2016 and installed to target. Change-Id: I2f9bc0610d1738c17b294ddd79a9cadd2eb2f6b0 Signed-off-by: Sanjeev N --- Makefile | 16 +--------------- 1 file changed, 1 insertion(+), 15 deletions(-) diff --git a/Makefile b/Makefile index 85cae565c9..94e5a36ec4 100644 --- a/Makefile +++ b/Makefile @@ -358,7 +358,7 @@ KBUILD_CPPFLAGS := -D__KERNEL__ -D__UBOOT__ KBUILD_CFLAGS := -Wall -Wstrict-prototypes \ -Wno-format-security \ - -fno-builtin -ffreestanding + -fno-builtin -ffreestanding -Werror KBUILD_AFLAGS := -D__ASSEMBLY__ # Read UBOOTRELEASE from include/config/uboot.release (if it exists) @@ -576,20 +576,6 @@ KBUILD_CFLAGS += $(call cc-option,-fstack-protector) endif KBUILD_CFLAGS += $(call cc-option,-fno-delete-null-pointer-checks) -#FIXME: Added to avoid the warning from sysupgrade-helper -#To be modified once the config is fixed -ifdef CONFIG_ARCH_IPQ807x -KBUILD_CFLAGS += -Werror -endif -ifdef CONFIG_ARCH_IPQ40xx -KBUILD_CFLAGS += -Werror -endif -ifdef CONFIG_ARCH_IPQ806x -KBUILD_CFLAGS += -Werror -endif -ifdef CONFIG_ARCH_IPQ6018 -KBUILD_CFLAGS += -Werror -endif KBUILD_CFLAGS += -g # $(KBUILD_AFLAGS) sets -g, which causes gcc to pass a suitable -g # option to the assembler.