PCI: Add vendor ID for Quectel and Cinterion

Message ID 20221027115123.5326-1-slark_xiao@163.com
State New
Headers
Series PCI: Add vendor ID for Quectel and Cinterion |

Commit Message

Slark Xiao Oct. 27, 2022, 11:51 a.m. UTC
  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 <slark_xiao@163.com>
---
 include/linux/pci_ids.h | 4 ++++
 1 file changed, 4 insertions(+)
  

Comments

Bjorn Helgaas Oct. 27, 2022, 2:27 p.m. UTC | #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 <slark_xiao@163.com>
> ---
>  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
>
  
Slark Xiao Oct. 28, 2022, 2:41 a.m. UTC | #2
At 2022-10-27 22:27:42, "Bjorn Helgaas" <helgaas@kernel.org> 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 <slark_xiao@163.com>
>> ---
>>  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
>>
  

Patch

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