From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] ixgbe: fix build with bypass and debug enabled Date: Tue, 02 Dec 2014 12:28:59 +0100 Message-ID: <7774520.abWCdkbC7k@xps13> References: <1417455659-24854-1-git-send-email-thomas.monjalon@6wind.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org To: David Marchand Return-path: In-Reply-To: List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" > > Since commit aae1047905621 ("use the right debug macro"), > > DEBUGOUT was replaced by PMD_DRV_LOG which requires at least > > 2 arguments. But the level argument was missing. > > > > Commit 7a10de5e27 fixed the logs but not the macros FUNC_PTR_OR_* > > which are not preprocessed if RTE_LIBRTE_IXGBE_DEBUG_DRIVER is disabled. > > > > Signed-off-by: Thomas Monjalon > > Argh ... good catch. > Looks like these were the only places with this error (I did some grep and > only found those). > And ok with the ERR level, it looks fine to me. > > Ack. Applied -- Thomas