On Fri, 20 Nov 2015 15:17:58 +0800 Wenzhuo Lu <wenzhuo.lu at intel.com> wrote:
> Signed-off-by: Wenzhuo Lu <wenzhuo.lu at intel.com> > --- > lib/librte_eal/common/include/rte_pci_dev_ids.h | 2 ++ > 1 file changed, 2 insertions(+) > > 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 e31b934..d088191 100644 > --- a/lib/librte_eal/common/include/rte_pci_dev_ids.h > +++ b/lib/librte_eal/common/include/rte_pci_dev_ids.h > @@ -433,6 +433,7 @@ RTE_PCI_DEV_ID_DECL_IGB(PCI_VENDOR_ID_INTEL, > E1000_DEV_ID_DH89XXCC_SFP) > #define IXGBE_DEV_ID_X550EM_X_10G_T 0x15AD > #define IXGBE_DEV_ID_X550EM_X_1G_T 0x15AE > #define IXGBE_DEV_ID_X550T 0x1563 > +#define IXGBE_DEV_ID_X550T1 0x15D1 > #define IXGBE_DEV_ID_X550EM_X_KX4 0x15AA > #define IXGBE_DEV_ID_X550EM_X_KR 0x15AB > > @@ -483,6 +484,7 @@ RTE_PCI_DEV_ID_DECL_IXGBE(PCI_VENDOR_ID_INTEL, > IXGBE_DEV_ID_X550EM_X_SFP) > RTE_PCI_DEV_ID_DECL_IXGBE(PCI_VENDOR_ID_INTEL, IXGBE_DEV_ID_X550EM_X_10G_T) > RTE_PCI_DEV_ID_DECL_IXGBE(PCI_VENDOR_ID_INTEL, IXGBE_DEV_ID_X550EM_X_1G_T) > RTE_PCI_DEV_ID_DECL_IXGBE(PCI_VENDOR_ID_INTEL, IXGBE_DEV_ID_X550T) > +RTE_PCI_DEV_ID_DECL_IXGBE(PCI_VENDOR_ID_INTEL, IXGBE_DEV_ID_X550T1) > RTE_PCI_DEV_ID_DECL_IXGBE(PCI_VENDOR_ID_INTEL, IXGBE_DEV_ID_X550EM_X_KX4) > RTE_PCI_DEV_ID_DECL_IXGBE(PCI_VENDOR_ID_INTEL, IXGBE_DEV_ID_X550EM_X_KR) This patch is good, glad to see the base driver keep up with the other Intel driver code bases. Has anyone looked into cleaning up the management of PCI id's in the source base. Putting all the PCI'ids for all DPDK drivers in one file with #ifdef's is really not a scalable long term solution. It needs to be split out into each driver.
