On 4/3/2017 3:22 PM, Jerin Jacob wrote: > On Mon, Apr 03, 2017 at 02:10:41PM +0000, Legacy, Allain wrote: >>> -----Original Message----- >>> From: Jerin Jacob [mailto:jerin.ja...@caviumnetworks.com] >>> Sent: Monday, April 03, 2017 9:59 AM >>> To: dev@dpdk.org >>> Cc: YIGIT, FERRUH; Legacy, Allain; Jerin Jacob >>> Subject: [dpdk-dev] [PATCH] net/avp: fix build with non x86 >>> >>> sys/io.h is an x86 specific header file and AVP driver is not using any x86 >>> specific IO operation. Hence removing the sys/io.h reference to fix the >>> build >>> issue on non x86 platforms. >>> >>> Fixes: 5567f9689cf8 ("net/avp: support driver registration") >>> >>> Signed-off-by: Jerin Jacob <jerin.ja...@caviumnetworks.com> >> >> In addition to this, since we do not support AVP devices on anything but >> x86_64 perhaps it would be appropriate to avoid compilation altogether on >> other architectures. That is, by disabling CONFIG_RTE_LIBRTE_AVP_PMD by >> default and only enabling for x86_64 on icc/gcc/clang for linuxapp? > > If that is the case then Please send a patch to fix it appropriately.
+1
