From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bruce Richardson Subject: Re: [PATCH] net/enic: remove assert which causes compile error Date: Thu, 13 Oct 2016 14:37:42 +0100 Message-ID: <20161013133742.GB138296@bricha3-MOBL3> References: <20161012162442.GI104428@bricha3-MOBL3> <20161012180935.8366-1-johndale@cisco.com> <20161013132215.GA138296@bricha3-MOBL3> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev@dpdk.org To: John Daley Return-path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id 8E4F12B97 for ; Thu, 13 Oct 2016 15:37:46 +0200 (CEST) Content-Disposition: inline In-Reply-To: <20161013132215.GA138296@bricha3-MOBL3> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Thu, Oct 13, 2016 at 02:22:16PM +0100, Bruce Richardson wrote: > On Wed, Oct 12, 2016 at 11:09:35AM -0700, John Daley wrote: > > Remove an RTE_ASSERT which will not compile if enabled and is not needed. > > > > Fixes: a1f7c7b3b5b2 ("net/enic: extend fdir support for 1300 series adapters") > > > > Signed-off-by: John Daley > > --- > > Would have been nice if I caught this yesterday before you applied a1f7c7b3 :( > > > Since commit a1f7c7b3 has not been pulled into the dpdk.org mainline, I can still > merge this commit into the origin one that has the bug. Unless you object, I > shall do so now. > > /Bruce Done. Fix merged into original commit /Bruce