[dpdk-dev] [PATCH v3 2/7] lib/librte_eal: Add PCI IDs for Broadcom bnxt

Bruce Richardson 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
>  #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


More information about the dev mailing list