From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <bruce.richardson@intel.com>
Received: from mga09.intel.com (mga09.intel.com [134.134.136.24])
 by dpdk.org (Postfix) with ESMTP id E8209DE5
 for <dev@dpdk.org>; Mon, 16 Jan 2017 17:48:45 +0100 (CET)
Received: from fmsmga001.fm.intel.com ([10.253.24.23])
 by orsmga102.jf.intel.com with ESMTP; 16 Jan 2017 08:48:19 -0800
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.33,240,1477983600"; d="scan'208";a="1094798509"
Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.221.61])
 by fmsmga001.fm.intel.com with SMTP; 16 Jan 2017 08:48:17 -0800
Received: by  (sSMTP sendmail emulation); Mon, 16 Jan 2017 16:48:16 +0000
Date: Mon, 16 Jan 2017 16:48:16 +0000
From: Bruce Richardson <bruce.richardson@intel.com>
To: Ferruh Yigit <ferruh.yigit@intel.com>
Cc: Alejandro Lucero <alejandro.lucero@netronome.com>, dev@dpdk.org,
 Thomas Monjalon <thomas.monjalon@6wind.com>
Message-ID: <20170116164816.GA25960@bricha3-MOBL3.ger.corp.intel.com>
References: <1484308202-5198-1-git-send-email-alejandro.lucero@netronome.com>
 <29aed2ee-78a9-05d5-b684-01f7b62d10f9@intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <29aed2ee-78a9-05d5-b684-01f7b62d10f9@intel.com>
Organization: Intel Research and =?iso-8859-1?Q?De=ACvel?=
 =?iso-8859-1?Q?opment?= Ireland Ltd.
User-Agent: Mutt/1.7.1 (2016-10-04)
Subject: Re: [dpdk-dev] [PATCH] config: build nfp pmd support by default
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Mon, 16 Jan 2017 16:48:46 -0000

On Mon, Jan 16, 2017 at 04:43:11PM +0000, Ferruh Yigit wrote:
> On 1/13/2017 11:50 AM, Alejandro Lucero wrote:
> > Because using a NFP PMD requires specific BSP installed, the PMD
> > support was not the default option before. This was just for making
> > people aware of such dependency, since there is no need for such a
> > BSP for just compiling DPDK with NFP PMD support.
> 
> Although NFP will compile fine, as far as I understand BSP dependency is
> still there.
> 
> Thomas,
> 
> What do you think, is compilation enough to be enabled by default?
> 
> > 
Although you haven't asked me, I think having it compiled by default is
ok, even if you can't run it without the BSP. Having it compiled means
that we can catch errors or problems with the driver sooner, e.g. when
doing updates across all drivers.

/Bruce