From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH 3/5] bnx2x: Add RTE_LIBRTE_BNX2X_REG_ACCESS config option Date: Tue, 20 Oct 2015 09:18:04 -0700 (PDT) Message-ID: <3670820.DENFCmAgGV@xps13> References: <1442018576-19981-1-git-send-email-rasesh.mody@qlogic.com> <1442018576-19981-4-git-send-email-rasesh.mody@qlogic.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, sony.chacko@qlogic.com To: harish.patil@qlogic.com Return-path: Received: from mail-wi0-f172.google.com (mail-wi0-f172.google.com [209.85.212.172]) by dpdk.org (Postfix) with ESMTP id 890078DAF for ; Tue, 20 Oct 2015 18:18:05 +0200 (CEST) Received: by wicfx6 with SMTP id fx6so53479879wic.1 for ; Tue, 20 Oct 2015 09:18:05 -0700 (PDT) In-Reply-To: <1442018576-19981-4-git-send-email-rasesh.mody@qlogic.com> 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" 2015-09-11 17:42, Rasesh Mody: > From: Harish Patil > --- a/config/common_linuxapp > +++ b/config/common_linuxapp > @@ -220,6 +220,7 @@ CONFIG_RTE_LIBRTE_BNX2X_DEBUG_INIT=n > CONFIG_RTE_LIBRTE_BNX2X_DEBUG_RX=n > CONFIG_RTE_LIBRTE_BNX2X_DEBUG_TX=n > CONFIG_RTE_LIBRTE_BNX2X_MF_SUPPORT=n > +CONFIG_RTE_LIBRTE_BNX2X_REG_ACCESS=n Why do you need this option? Why only for Linux? And last but not the least, why nobody else have reviewed carefully this patch after one month?