From ff5c3960017cf4625581e2a03cead35cbd3e6744 Mon Sep 17 00:00:00 2001 From: Helin Zhang Date: Mon, 18 May 2015 23:40:56 +0800 Subject: [PATCH] i40e/base: fix build with clang When compiling with clang, it detects that a variable used for debug is unused (debug_mask is never set): lib/librte_pmd_i40e/i40e/i40e_nvm.c:708:20: error: unused variable 'i40e_nvm_update_state_str' [-Werror,-Wunused-variable] STATIC const char *i40e_nvm_update_state_str[] = { ^ As we are not supposed to modify the base driver (except for update synchronization from Intel), it was ignored with GCC thanks to the option -Wno-unused-variable but clang was forgotten. Fixes: 6b5aaf47d571 ("i40e/base: replace NVM debug logs") Signed-off-by: Thomas Monjalon Signed-off-by: Helin Zhang Acked-by: Bruce Richardson Acked-by: Tetsuya Mukawa --- lib/librte_pmd_i40e/Makefile | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/librte_pmd_i40e/Makefile b/lib/librte_pmd_i40e/Makefile index 911e4f5147..4a5635bc10 100644 --- a/lib/librte_pmd_i40e/Makefile +++ b/lib/librte_pmd_i40e/Makefile @@ -58,6 +58,7 @@ CFLAGS_BASE_DRIVER += -Wno-format CFLAGS_BASE_DRIVER += -Wno-missing-field-initializers CFLAGS_BASE_DRIVER += -Wno-pointer-to-int-cast CFLAGS_BASE_DRIVER += -Wno-format-nonliteral +CFLAGS_BASE_DRIVER += -Wno-unused-variable else CFLAGS_BASE_DRIVER = -Wno-sign-compare CFLAGS_BASE_DRIVER += -Wno-unused-value -- 2.20.1