From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id 6EA67677C for ; Wed, 11 May 2016 19:17:23 +0200 (CEST) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga101.jf.intel.com with ESMTP; 11 May 2016 10:17:22 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,608,1455004800"; d="scan'208";a="973653509" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.154]) ([10.237.220.154]) by orsmga002.jf.intel.com with ESMTP; 11 May 2016 10:17:21 -0700 To: Harish Patil , "dev@dpdk.org" References: <1462879407-27649-1-git-send-email-ferruh.yigit@intel.com> Cc: Rasesh Mody , Sony Chacko From: Ferruh Yigit Message-ID: <57336920.5060409@intel.com> Date: Wed, 11 May 2016 18:17:20 +0100 User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.7.2 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH] qede: fix icc build error X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 11 May 2016 17:17:23 -0000 On 5/10/2016 9:02 PM, Harish Patil wrote: > Hi Ferruh, > >> fix errors: >> icc: command line warning #10006: ignoring unknown option >> '-Wno-unused-value' >> icc: command line warning #10006: ignoring unknown option >> '-Wno-format-nonliteral' >> icc: command line warning #10006: ignoring unknown option >> '-Wno-shift-negative-value' >> qede/base/ecore_dev.c(1643): error #188: enumerated type mixed with >> another type >> return 0; >> ^ >> >> Signed-off-by: Ferruh Yigit >> --- >> drivers/net/qede/Makefile | 7 +++++-- >> 1 file changed, 5 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/net/qede/Makefile b/drivers/net/qede/Makefile >> index 47e01be..d0c1361 100644 >> --- a/drivers/net/qede/Makefile >> +++ b/drivers/net/qede/Makefile >> @@ -27,7 +27,6 @@ OS_TYPE := $(shell uname -s) >> # CFLAGS >> # >> CFLAGS_BASE_DRIVER = -Wno-unused-parameter >> -CFLAGS_BASE_DRIVER += -Wno-unused-value >> CFLAGS_BASE_DRIVER += -Wno-sign-compare >> CFLAGS_BASE_DRIVER += -Wno-missing-prototypes >> CFLAGS_BASE_DRIVER += -Wno-cast-qual >> @@ -35,12 +34,16 @@ CFLAGS_BASE_DRIVER += -Wno-unused-function >> CFLAGS_BASE_DRIVER += -Wno-unused-variable >> CFLAGS_BASE_DRIVER += -Wno-strict-aliasing >> CFLAGS_BASE_DRIVER += -Wno-missing-prototypes >> + >> +ifneq ($(CONFIG_RTE_TOOLCHAIN_ICC),y) >> +CFLAGS_BASE_DRIVER += -Wno-unused-value > > This change is not for the actual build error but to mask the warning: > > icc: command line warning #10006: ignoring unknown option > '-Wno-unused-value’, right? > > > If so, what about for Wno-format-nonliteral and Wno-shift-negative-value? These three flags are not supported by icc, please check commit log, it has three of them in warnings, so removing them for icc. This doesn't cause a compilation error, just warnings from compiler but it is too noisy, I believe good to remove. > >> CFLAGS_BASE_DRIVER += -Wno-format-nonliteral >> ifeq ($(OS_TYPE),Linux) >> ifeq ($(shell clang -Wno-shift-negative-value -Werror -E - < /dev/null > >> /dev/null 2>&1; echo $$?),0) >> CFLAGS_BASE_DRIVER += -Wno-shift-negative-value >> endif >> endif >> +endif >> >> ifneq (,$(filter gcc gcc48,$(CC))) >> CFLAGS_BASE_DRIVER += -Wno-unused-but-set-variable >> @@ -57,7 +60,7 @@ ifeq ($(shell clang -Wno-pointer-bool-conversion >> -Werror -E - < /dev/null > /dev >> CFLAGS_BASE_DRIVER += -Wno-pointer-bool-conversion >> endif >> else >> -#icc flags >> +CFLAGS_BASE_DRIVER += -wd188 #188: enumerated type mixed with another >> type >> endif >> >> # >> -- >> 2.5.5 >> >> > Looks good. > I hope its retested against gcc/clang also to make sure the change doesn’t > cause build errors. Yes, did a gcc and clang compile test. > Acked-by: Harish Patil > >