From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id EA1072BA3 for ; Wed, 2 Nov 2016 11:21:16 +0100 (CET) Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga104.jf.intel.com with ESMTP; 02 Nov 2016 03:21:16 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.31,583,1473145200"; d="scan'208";a="26465750" Received: from yliu-dev.sh.intel.com ([10.239.67.162]) by fmsmga006.fm.intel.com with ESMTP; 02 Nov 2016 03:21:15 -0700 From: Yuanhan Liu To: Ferruh Yigit Date: Wed, 2 Nov 2016 18:21:24 +0800 Message-Id: <1478082097-16957-28-git-send-email-yuanhan.liu@linux.intel.com> X-Mailer: git-send-email 1.9.0 In-Reply-To: <1478082097-16957-1-git-send-email-yuanhan.liu@linux.intel.com> References: <1478082097-16957-1-git-send-email-yuanhan.liu@linux.intel.com> Cc: Rasesh Mody , dpdk stable Subject: [dpdk-stable] patch 'net/bnx2x: fix build with icc' has been queued to stable release 16.07.2 X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 02 Nov 2016 10:21:17 -0000 Hi, FYI, your patch has been queued to stable release 16.07.2 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 11/06/16. So please shout if anyone has objections. Thanks. --yliu --- >>From b5da31159f14fafef68c326c17d8cd68a0dac770 Mon Sep 17 00:00:00 2001 From: Ferruh Yigit Date: Fri, 30 Sep 2016 15:14:13 +0100 Subject: [PATCH] net/bnx2x: fix build with icc [ upstream commit 35050da740f0a031fb2c550f367d14db30ca8b9a ] When bnx2x debug config enabled, icc (ICC) 17.0.0 20160721 gives the following compilation error: .../drivers/net/bnx2x/elink.c(6682): error #3656: variable "fw_ver1" may be used before its value is set PMD_DRV_LOG(DEBUG, ^ According logic in the code, this error is a false positive, but since this is not in the fast path, fixing compiler error by assigning initial value to variable. Signed-off-by: Ferruh Yigit Acked-by: Rasesh Mody --- drivers/net/bnx2x/elink.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/bnx2x/elink.c b/drivers/net/bnx2x/elink.c index 149cc97..d9a72f0 100644 --- a/drivers/net/bnx2x/elink.c +++ b/drivers/net/bnx2x/elink.c @@ -6645,7 +6645,7 @@ static elink_status_t elink_8073_8727_external_rom_boot(struct bnx2x_softc *sc, uint8_t port) { uint32_t count = 0; - uint16_t fw_ver1, fw_msgout; + uint16_t fw_ver1 = 0, fw_msgout; elink_status_t rc = ELINK_STATUS_OK; /* Boot port from external ROM */ -- 1.9.0