Re: [dpdk-dev] [PATCH 3/6] qede: add QLogic PCI ids

[Harish Patil <harish.patil@qlogic.com>]

>

>On Sat, 20 Feb 2016 07:40:28 -0800
>Harish Patil <harish.patil@qlogic.com> wrote:
>
>> 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..0c1a3fe 100644
>> --- a/lib/librte_eal/common/include/rte_pci_dev_ids.h
>> +++ b/lib/librte_eal/common/include/rte_pci_dev_ids.h
>> @@ -152,6 +152,14 @@
>>  #define RTE_PCI_DEV_ID_DECL_BNX2XVF(vend, dev)
>>  #endif
>>
>> +#ifndef RTE_PCI_DEV_ID_DECL_QEDE
>> +#define RTE_PCI_DEV_ID_DECL_QEDE(vend, dev)
>> +#endif
>> +
>> +#ifndef RTE_PCI_DEV_ID_DECL_QEDEVF
>> +#define RTE_PCI_DEV_ID_DECL_QEDEVF(vend, dev)
>> +#endif
>> +
>>  #ifndef PCI_VENDOR_ID_INTEL
>>  /** Vendor ID used by Intel devices */
>>  #define PCI_VENDOR_ID_INTEL 0x8086
>> @@ -177,6 +185,11 @@
>>  #define PCI_VENDOR_ID_BROADCOM 0x14E4
>>  #endif
>>
>> +#ifndef PCI_VENDOR_ID_QLOGIC
>> +/** Vendor ID used by QLogic devices */
>> +#define PCI_VENDOR_ID_QLOGIC 0x1077
>> +#endif
>> +
>
>The goal is get away from having every PCI id for every device
>stuffed into rte_pci_dev_ids.h. So it would be good not to add
>more now.
>

Okay, I shall make the PCI ids local to this driver.

Thanks,
Harish


________________________________

This message and any attached documents contain information from the sending company or its parent company(s), subsidiaries, divisions or branch offices that may be confidential. If you are not the intended recipient, you may not read, copy, distribute, or use this information. If you have received this transmission in error, please notify the sender immediately by reply e-mail and then delete this message.