firmware-utils: unify some of the C flags for all utils
authorPetr Štetiar <ynezz@true.cz>
Sat, 20 Jul 2019 10:32:03 +0000 (12:32 +0200)
committerPetr Štetiar <ynezz@true.cz>
Tue, 23 Jul 2019 20:07:22 +0000 (22:07 +0200)
Enable -Wextra -Werror in order to catch and fix potential errors during
compilation.

Signed-off-by: Petr Štetiar <ynezz@true.cz>
tools/firmware-utils/Makefile

index d318788ac542801f9cbd671845b2c6aa4f2e299f..cecf916ba27ede90d0715d424a74ce204d792200 100644 (file)
@@ -14,6 +14,7 @@ include $(INCLUDE_DIR)/kernel.mk
 define cc
        $(HOSTCC) \
                $(HOST_CFLAGS) \
+               -Wall -Werror -Wextra \
                -Wno-unused-parameter \
                -include endian.h $(HOST_LDFLAGS) \
                -o $(HOST_BUILD_DIR)/bin/$(firstword $(1)) \
@@ -29,7 +30,7 @@ define Host/Compile
        $(call cc,otrx)
        $(call cc,motorola-bin)
        $(call cc,dgfirmware)
-       $(call cc,mksenaofw md5, -Wall --std=gnu99)
+       $(call cc,mksenaofw md5, --std=gnu99)
        $(call cc,trx2usr)
        $(call cc,ptgen)
        $(call cc,srec2bin)
@@ -38,7 +39,7 @@ define Host/Compile
        $(call cc,mkzynfw)
        $(call cc,lzma2eva,-lz)
        $(call cc,mkcasfw)
-       $(call cc,mkfwimage,-lz -Wall -Werror -Wextra -D_FILE_OFFSET_BITS=64)
+       $(call cc,mkfwimage,-lz -D_FILE_OFFSET_BITS=64)
        $(call cc,mkfwimage2,-lz)
        $(call cc,imagetag imagetag_cmdline cyg_crc32)
        $(call cc,add_header)
@@ -46,9 +47,9 @@ define Host/Compile
        $(call cc,encode_crc)
        $(call cc,nand_ecc)
        $(call cc,mkplanexfw sha1)
-       $(call cc,mktplinkfw mktplinkfw-lib md5, -Wall -fgnu89-inline)
+       $(call cc,mktplinkfw mktplinkfw-lib md5, -fgnu89-inline)
        $(call cc,mktplinkfw2 mktplinkfw-lib md5, -fgnu89-inline)
-       $(call cc,tplink-safeloader md5, -Wall --std=gnu99)
+       $(call cc,tplink-safeloader md5, --std=gnu99)
        $(call cc,pc1crypt)
        $(call cc,osbridge-crc)
        $(call cc,wrt400n cyg_crc32)
@@ -60,38 +61,38 @@ define Host/Compile
        $(call cc,zyxbcm)
        $(call cc,trx2edips)
        $(call cc,xorimage)
-       $(call cc,buffalo-enc buffalo-lib, -Wall)
-       $(call cc,buffalo-tag buffalo-lib, -Wall)
-       $(call cc,buffalo-tftp buffalo-lib, -Wall)
-       $(call cc,mkwrgimg md5, -Wall)
-       $(call cc,mkwrggimg md5, -Wall)
+       $(call cc,buffalo-enc buffalo-lib)
+       $(call cc,buffalo-tag buffalo-lib)
+       $(call cc,buffalo-tftp buffalo-lib)
+       $(call cc,mkwrgimg md5)
+       $(call cc,mkwrggimg md5)
        $(call cc,mkedimaximg)
        $(call cc,mkbrncmdline)
        $(call cc,mkbrnimg)
        $(call cc,mkdapimg)
        $(call cc,mkdapimg2)
-       $(call cc, mkcameofw, -Wall)
+       $(call cc, mkcameofw)
        $(call cc,seama md5)
-       $(call cc,oseama md5, -Wall)
-       $(call cc,fix-u-media-header cyg_crc32,-Wall)
+       $(call cc,oseama md5)
+       $(call cc,fix-u-media-header cyg_crc32)
        $(call cc,hcsmakeimage bcmalgo)
-       $(call cc,mkporayfw, -Wall)
+       $(call cc,mkporayfw)
        $(call cc,mkrasimage, --std=gnu99)
        $(call cc,mkhilinkfw, -lcrypto)
        $(call cc,mkheader_gemtek,-lz)
        $(call cc,mkrtn56uimg, -lz)
-       $(call cc,dgn3500sum, -Wall)
-       $(call cc,edimax_fw_header, -Wall)
-       $(call cc,mkmerakifw sha1, -Wall)
-       $(call cc,mkmerakifw-old, -Wall)
-       $(call cc,jcgimage, -lz -Wall)
-       $(call cc,mkbuffaloimg, -Wall)
-       $(call cc,zyimage, -Wall)
-       $(call cc,mkdhpimg buffalo-lib, -Wall)
-       $(call cc,mkdlinkfw mkdlinkfw-lib, -lz -Wall --std=c99)
-       $(call cc,dns313-header, -Wall)
-       $(call cc,mksercommfw, -Wall)
-       $(call cc,nec-enc, -Wall --std=gnu99)
+       $(call cc,dgn3500sum)
+       $(call cc,edimax_fw_header)
+       $(call cc,mkmerakifw sha1)
+       $(call cc,mkmerakifw-old)
+       $(call cc,jcgimage, -lz)
+       $(call cc,mkbuffaloimg)
+       $(call cc,zyimage)
+       $(call cc,mkdhpimg buffalo-lib)
+       $(call cc,mkdlinkfw mkdlinkfw-lib, -lz --std=c99)
+       $(call cc,dns313-header)
+       $(call cc,mksercommfw)
+       $(call cc,nec-enc, --std=gnu99)
 endef
 
 define Host/Install