From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH] config: compile avp driver only on intel 64 bit Date: Tue, 4 Apr 2017 18:12:12 +0100 Message-ID: <4712ffa9-b81c-bdef-ccc0-ae508d5b327d@intel.com> References: <20170404163654.86100-1-allain.legacy@windriver.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org To: Allain Legacy Return-path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id A0B182FDD for ; Tue, 4 Apr 2017 19:12:15 +0200 (CEST) In-Reply-To: <20170404163654.86100-1-allain.legacy@windriver.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 4/4/2017 5:36 PM, Allain Legacy wrote: > The AVP devices are only supported on Intel 64-bit architectures so > adjusting the defconfig attributes accordingly. > > Fixes: c5ab4707ba97 ("net/avp: support driver registration") > > Signed-off-by: Allain Legacy > --- > config/common_linuxapp | 1 - > config/defconfig_i686-native-linuxapp-gcc | 5 ----- > config/defconfig_i686-native-linuxapp-icc | 5 ----- > config/defconfig_x86_64-native-linuxapp-clang | 5 +++++ Right now compiler config files only have compiler and architecture configs, although it is OK to update them, to be consistent with what other PMDs did, what do you think: Keep AVP config enabled in "common_linuxapp" Disable AVP in following configs: defconfig_arm64-armv8a-linuxapp-gcc defconfig_arm-armv7a-linuxapp-gcc defconfig_ppc_64-power8-linuxapp-gcc > config/defconfig_x86_64-native-linuxapp-gcc | 6 ++++++ > config/defconfig_x86_64-native-linuxapp-icc | 5 +++++ > config/defconfig_x86_x32-native-linuxapp-gcc | 5 ----- > 7 files changed, 16 insertions(+), 16 deletions(-) > <...>