From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id 47744BDC2 for ; Tue, 9 Feb 2016 12:08:48 +0100 (CET) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga104.fm.intel.com with ESMTP; 09 Feb 2016 03:08:28 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,420,1449561600"; d="scan'208";a="880303158" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.221.35]) by orsmga001.jf.intel.com with SMTP; 09 Feb 2016 03:08:27 -0800 Received: by (sSMTP sendmail emulation); Tue, 09 Feb 2016 11:08:25 +0025 Date: Tue, 9 Feb 2016 11:08:25 +0000 From: Bruce Richardson To: dev@dpdk.org Message-ID: <20160209110824.GA10444@bricha3-MOBL3> References: <1454969509-19157-1-git-send-email-bruce.richardson@intel.com> <20160209083926.GS27079@6wind.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160209083926.GS27079@6wind.com> Organization: Intel Shannon Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Subject: Re: [dpdk-dev] [PATCH] mlx4: fix unneeded function error with clang v3.6 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: Tue, 09 Feb 2016 11:08:48 -0000 On Tue, Feb 09, 2016 at 09:39:26AM +0100, Adrien Mazarguil wrote: > On Mon, Feb 08, 2016 at 10:11:49PM +0000, Bruce Richardson wrote: > > When compiling with clang 3.6, the mlx4 driver gives the following error > > message about an unneeded function. > > > > CC mlx4.o > > .../drivers/net/mlx4/mlx4.c:136:20: fatal error: function > > 'wr_id_t_check' is not needed and will not be emitted > > [-Wunneeded-internal-declaration] > > static inline void wr_id_t_check(void) > > ^ > > 1 error generated. > > > > The function is to compile-time check the size of wr_id_t, so use > > the standard DPDK BUILD_BUG_ON macro to do so in the init function > > instead. > > > > Fixes: 7fae69eeff13 ("mlx4: new poll mode driver") > > > > Signed-off-by: Bruce Richardson > > Thanks. > > Acked-by: Adrien Mazarguil > > -- Applied to dpdk-next-net/rel_16_04 /Bruce