[dpdk-dev] [PATCH v3 2/7] lib/librte_eal: Add PCI IDs for Broadcom bnxt
bruce.richardson at intel.com
Tue Apr 19 15:01:43 CEST 2016
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
> +#ifndef PCI_VENDOR_ID_BROADCOM
> +/** Vendor ID used by Broadcom devices */
> +#define PCI_VENDOR_ID_BROADCOM 0x14E4
> +#ifndef RTE_PCI_DEV_ID_DECL_BNXT
> +#define RTE_PCI_DEV_ID_DECL_BNXT(vendor, dev)
This macro definition looks to be in the wrong place in the file. The device
definitions are all placed above the vendor declarations.
More information about the dev