On Thu, Sep 13, 2018 at 02:27:54PM +0100, Luca Boccassi wrote: > On Thu, 2018-09-13 at 14:24 +0100, Bruce Richardson wrote: > > On Tue, Sep 11, 2018 at 05:08:58PM +0100, Luca Boccassi wrote: > > > Signed-off-by: Luca Boccassi <bl...@debian.org> > > > --- > > > drivers/net/avf/base/meson.build | 20 ++++++++++++++++++++ > > > drivers/net/avf/meson.build | 15 +++++++++++++++ > > > drivers/net/meson.build | 1 + > > > 3 files changed, 36 insertions(+) > > > create mode 100644 drivers/net/avf/base/meson.build > > > create mode 100644 drivers/net/avf/meson.build > > > > > > > After applying this patch, I get lots of unused variable warnings. > > Gcc 6.3 > > on debian, you'll be glad to know. > > > > /Bruce > > Yep got the same, but I'll leave that for the AVF maintainers to have > fun with :-) > Ok, I've sent a patch to remove the unused stuff. http://patches.dpdk.org/patch/44694/
I think this file still needs "cflags += '-Wno-strict-aliasing'" to make it work though, since those errors are tougher to remove. There also needs to be an "INC_VECTOR" define inside the x86-specific block, as is done for the IXGBE and I40E drivers. /Bruce
