In MHI driver, there are some companies product still
do not have their own PCI vendor macro. So we add it
here to make the code neat. Ref ID could be found in
link https://pcisig.com/membership/member-companies.
Signed-off-by: Slark Xiao <[email protected]>
---
include/linux/pci_ids.h | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h
index b362d90eb9b0..e3bfea06b90e 100644
--- a/include/linux/pci_ids.h
+++ b/include/linux/pci_ids.h
@@ -172,6 +172,10 @@
#define PCI_DEVICE_ID_BERKOM_A4T 0xffa4
#define PCI_DEVICE_ID_BERKOM_SCITEL_QUADRO 0xffa8
+#define PCI_VENDOR_ID_CINTERION 0x1269
+
+#define PCI_VENDOR_ID_QUECTEL 0x1eac
+
#define PCI_VENDOR_ID_COMPAQ 0x0e11
#define PCI_DEVICE_ID_COMPAQ_TOKENRING 0x0508
#define PCI_DEVICE_ID_COMPAQ_TACHYON 0xa0fc
--
2.34.1
On Thu, Oct 27, 2022 at 07:51:23PM +0800, Slark Xiao wrote:
> In MHI driver, there are some companies product still
> do not have their own PCI vendor macro. So we add it
> here to make the code neat. Ref ID could be found in
> link https://pcisig.com/membership/member-companies.
Rewrap to fill 75 columns.
Post along with the patch that uses the definitions.
> Signed-off-by: Slark Xiao <[email protected]>
> ---
> include/linux/pci_ids.h | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h
> index b362d90eb9b0..e3bfea06b90e 100644
> --- a/include/linux/pci_ids.h
> +++ b/include/linux/pci_ids.h
> @@ -172,6 +172,10 @@
> #define PCI_DEVICE_ID_BERKOM_A4T 0xffa4
> #define PCI_DEVICE_ID_BERKOM_SCITEL_QUADRO 0xffa8
>
> +#define PCI_VENDOR_ID_CINTERION 0x1269
The URL above says 0x1269 belongs to Thales. Maybe add a comment to
that effect, similar to the one for PCI_VENDOR_ID_CB.
> +#define PCI_VENDOR_ID_QUECTEL 0x1eac
Both new definitions look like they're dropped in a random location.
Please move them so the vendor IDs in the file remain sorted by
numeric value.
> #define PCI_VENDOR_ID_COMPAQ 0x0e11
> #define PCI_DEVICE_ID_COMPAQ_TOKENRING 0x0508
> #define PCI_DEVICE_ID_COMPAQ_TACHYON 0xa0fc
> --
> 2.34.1
>
At 2022-10-27 22:27:42, "Bjorn Helgaas" <[email protected]> wrote:
>On Thu, Oct 27, 2022 at 07:51:23PM +0800, Slark Xiao wrote:
>> In MHI driver, there are some companies product still
>> do not have their own PCI vendor macro. So we add it
>> here to make the code neat. Ref ID could be found in
>> link https://pcisig.com/membership/member-companies.
>
>Rewrap to fill 75 columns.
>
>Post along with the patch that uses the definitions.
>
>> Signed-off-by: Slark Xiao <[email protected]>
>> ---
>> include/linux/pci_ids.h | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h
>> index b362d90eb9b0..e3bfea06b90e 100644
>> --- a/include/linux/pci_ids.h
>> +++ b/include/linux/pci_ids.h
>> @@ -172,6 +172,10 @@
>> #define PCI_DEVICE_ID_BERKOM_A4T 0xffa4
>> #define PCI_DEVICE_ID_BERKOM_SCITEL_QUADRO 0xffa8
>>
>> +#define PCI_VENDOR_ID_CINTERION 0x1269
>
>The URL above says 0x1269 belongs to Thales. Maybe add a comment to
>that effect, similar to the one for PCI_VENDOR_ID_CB.
>
>> +#define PCI_VENDOR_ID_QUECTEL 0x1eac
>
>Both new definitions look like they're dropped in a random location.
>Please move them so the vendor IDs in the file remain sorted by
>numeric value.
>
Hi Bjorn,
V2 version is sent. I add explanation about the relationship between Thales
and CInterion. And I just find these VIDs are sorted by numeric, so I fix it in v2.
Thanks.
>> #define PCI_VENDOR_ID_COMPAQ 0x0e11
>> #define PCI_DEVICE_ID_COMPAQ_TOKENRING 0x0508
>> #define PCI_DEVICE_ID_COMPAQ_TACHYON 0xa0fc
>> --
>> 2.34.1
>>