On Fri, Mar 04, 2016 at 01:05:23PM -0800, Stephen Hurd wrote: > Add Broadcom Vendor ID and RTE_PCI_DEV_ID_DECL_BNXT() macro. > Add Device IDs for Broadcom bnxt devices. > > Signed-off-by: Stephen Hurd <stephen.hurd at broadcom.com> > --- > lib/librte_eal/common/include/rte_pci_dev_ids.h | 45 > ++++++++++++++++++++++--- > 1 file changed, 40 insertions(+), 5 deletions(-) > > diff --git a/lib/librte_eal/common/include/rte_pci_dev_ids.h > b/lib/librte_eal/common/include/rte_pci_dev_ids.h > index d088191..9a8f254 100644 > --- a/lib/librte_eal/common/include/rte_pci_dev_ids.h > +++ b/lib/librte_eal/common/include/rte_pci_dev_ids.h > @@ -63,11 +63,11 @@ > * This file contains a list of the PCI device IDs recognised by DPDK, which > * can be used to fill out an array of structures describing the devices. > * > - * Currently four families of devices are recognised: those supported by the > - * IGB driver, by EM driver, those supported by the IXGBE driver, and by > virtio > - * driver which is a para virtualization driver running in guest virtual > machine. > - * The inclusion of these in an array built using this file depends on the > - * definition of > + * Currently five families of devices are recognised: those supported by the > + * IGB driver, by EM driver, those supported by the IXGBE driver, by BNXT > + * driver, and by virtio driver which is a para virtualization driver running > + * in guest virtual machine. The inclusion of these in an array built using > + * this file depends on the definition of > * RTE_PCI_DEV_ID_DECL_EM > * RTE_PCI_DEV_ID_DECL_IGB > * RTE_PCI_DEV_ID_DECL_IGBVF > @@ -76,6 +76,7 @@ > * RTE_PCI_DEV_ID_DECL_I40E > * RTE_PCI_DEV_ID_DECL_I40EVF > * RTE_PCI_DEV_ID_DECL_VIRTIO > + * RTE_PCI_DEV_ID_DECL_BNXT > * at the time when this file is included. > * > * In order to populate an array, the user of this file must define this > macro: > @@ -167,6 +168,15 @@ > #define PCI_VENDOR_ID_VMWARE 0x15AD > #endif > > +#ifndef PCI_VENDOR_ID_BROADCOM > +/** Vendor ID used by Broadcom devices */ > +#define PCI_VENDOR_ID_BROADCOM 0x14E4 > +#endif > + > +#ifndef RTE_PCI_DEV_ID_DECL_BNXT > +#define RTE_PCI_DEV_ID_DECL_BNXT(vendor, dev) > +#endif > +
This macro definition looks to be in the wrong place in the file. The device definitions are all placed above the vendor declarations. /Bruce
