net/qede: fix gcc option checks
authorRasesh Mody <rasesh.mody@cavium.com>
Fri, 28 Oct 2016 06:37:57 +0000 (23:37 -0700)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Mon, 7 Nov 2016 20:08:54 +0000 (21:08 +0100)
Using GCC_VERSION to check gcc version and decide whether to include
that compiler option.

Fixes: ecc7a5a27ffe ("net/qede/base: fix 32-bit build")

Signed-off-by: Rasesh Mody <rasesh.mody@cavium.com>
drivers/net/qede/Makefile

index 39751e4..29b443d 100644 (file)
@@ -46,11 +46,11 @@ endif
 endif
 
 ifeq ($(CONFIG_RTE_TOOLCHAIN_GCC),y)
-ifeq ($(shell gcc -Wno-unused-but-set-variable -Werror -E - < /dev/null > /dev/null 2>&1; echo $$?),0)
+ifeq ($(shell test $(GCC_VERSION) -ge 44 && echo 1), 1)
 CFLAGS_BASE_DRIVER += -Wno-unused-but-set-variable
 endif
 CFLAGS_BASE_DRIVER += -Wno-missing-declarations
-ifeq ($(shell gcc -Wno-maybe-uninitialized -Werror -E - < /dev/null > /dev/null 2>&1; echo $$?),0)
+ifeq ($(shell test $(GCC_VERSION) -ge 46 && echo 1), 1)
 CFLAGS_BASE_DRIVER += -Wno-maybe-uninitialized
 endif
 CFLAGS_BASE_DRIVER += -Wno-strict-prototypes