On Mon, Jun 06, 2016 at 03:08:34PM -0700, Stephen Hurd wrote: > From: Ajit Khaparde <ajit.khaparde at broadcom.com> > > This patch adds code to add the start, stop and link update dev_ops. > The BNXT driver will now minimally pass traffic with testpmd. > > v4: > - Fix issues pointed out by checkpatch. > - Shorten the string passed for reserving memzone > when default completion ring is created. > > Signed-off-by: Ajit Khaparde <ajit.khaparde at broadcom.com> > Reviewed-by: David Christensen <david.christensen at broadcom.com> > Signed-off-by: Stephen Hurd <stephen.hurd at broadcom.com> > --- > drivers/net/bnxt/bnxt_ethdev.c | 269 > +++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 269 insertions(+) > I get compilation errors after applying this patch:
== Build drivers/net/bnxt CC bnxt_ethdev.o /home/bruce/next-net/dpdk-next-net/drivers/net/bnxt/bnxt_ethdev.c: In function ?bnxt_init_chip?: /home/bruce/next-net/dpdk-next-net/drivers/net/bnxt/bnxt_ethdev.c:135:7: error: implicit declaration of function ?bnxt_alloc_hwrm_rings? [-Werror=implicit-function-declaration] rc = bnxt_alloc_hwrm_rings(bp); ^ /home/bruce/next-net/dpdk-next-net/drivers/net/bnxt/bnxt_ethdev.c:135:2: error: nested extern declaration of ?bnxt_alloc_hwrm_rings? [-Werror=nested-externs] rc = bnxt_alloc_hwrm_rings(bp); ^ /home/bruce/next-net/dpdk-next-net/drivers/net/bnxt/bnxt_ethdev.c: In function ?bnxt_init_nic?: /home/bruce/next-net/dpdk-next-net/drivers/net/bnxt/bnxt_ethdev.c:233:2: error: implicit declaration of function ?bnxt_init_ring_grps? [-Werror=implicit-function-declaration] bnxt_init_ring_grps(bp); ^ /home/bruce/next-net/dpdk-next-net/drivers/net/bnxt/bnxt_ethdev.c:233:2: error: nested extern declaration of ?bnxt_init_ring_grps? [-Werror=nested-externs] cc1: all warnings being treated as errors /home/bruce/next-net/dpdk-next-net/mk/internal/rte.compile-pre.mk:126: recipe for target 'bnxt_ethdev.o' failed make[5]: *** [bnxt_ethdev.o] Error 1