Well that's embarasing :). Forgot to add _done. Not enough coffee. On Tue, 2017-10-17 at 22:16 +0100, Kevin Darbyshire-Bryant wrote: > > On 17/10/17 21:52, ros...@gmail.com wrote: > > I'll take your word for it since I have no hardware to test on. > > I'd say it has a huge impact on performance irrespective of hardware > 'cos the patch as supplied doesn't actually compile ;-) > > > CC drivers/net/ethernet/atheros/ag71xx/ag71xx_main.o > drivers/net/ethernet/atheros/ag71xx/ag71xx_main.c: In function > 'ag71xx_poll': > drivers/net/ethernet/atheros/ag71xx/ag71xx_main.c:1163:2: error: too > many arguments to function 'napi_complete' > napi_complete(napi, rx_done); > ^~~~~~~~~~~~~ > In file included from ./include/linux/etherdevice.h:26:0, > from > drivers/net/ethernet/atheros/ag71xx/ag71xx.h:27, > from > drivers/net/ethernet/atheros/ag71xx/ag71xx_main.c:14: > ./include/linux/netdevice.h:465:20: note: declared here > static inline void napi_complete(struct napi_struct *n) > ^~~~~~~~~~~~~ > scripts/Makefile.build:293: recipe for target > 'drivers/net/ethernet/atheros/ag71xx/ag71xx_main.o' failed > make[10]: *** [drivers/net/ethernet/atheros/ag71xx/ag71xx_main.o] > Error 1 > > > Kevin > > _______________________________________________ > Lede-dev mailing list > Lede-dev@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/lede-dev
_______________________________________________ Lede-dev mailing list Lede-dev@lists.infradead.org http://lists.infradead.org/mailman/listinfo/lede-dev
