[PATCH] PCI: pci_ids: Add Cixtech P1 Platforms vendor and device ID

[PATCH] PCI: pci_ids: Add Cixtech P1 Platforms vendor and device ID

[hans.zhang]

From: Hans Zhang <hans.zhang@cixtech.com>

Add Cixtech P1 (internal name sky1) as a vendor and device ID for PCI
devices so we can use the macro for future drivers.

Signed-off-by: Hans Zhang <hans.zhang@cixtech.com>
Reviewed-by: Peter Chen <peter.chen@cixtech.com>
---
 include/linux/pci_ids.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h
index 1a2594a38199..531b34c1181a 100644
--- a/include/linux/pci_ids.h
+++ b/include/linux/pci_ids.h
@@ -200,6 +200,9 @@
 #define PCI_DEVICE_ID_COMPAQ_THUNDER	0xf130
 #define PCI_DEVICE_ID_COMPAQ_NETFLEX3B	0xf150
 
+#define PCI_VENDOR_ID_CIX               0x1f6c
+#define PCI_DEVICE_ID_CIX_SKY1          0x0001
+
 #define PCI_VENDOR_ID_NCR		0x1000
 #define PCI_VENDOR_ID_LSI_LOGIC		0x1000
 #define PCI_DEVICE_ID_NCR_53C810	0x0001

base-commit: 99fa936e8e4f117d62f229003c9799686f74cebc
-- 
2.47.1

Re: [PATCH] PCI: pci_ids: Add Cixtech P1 Platforms vendor and device ID

[Bjorn Helgaas]

On Wed, Mar 05, 2025 at 02:30:18PM +0800, hans.zhang@cixtech.com wrote:
> From: Hans Zhang <hans.zhang@cixtech.com>
> 
> Add Cixtech P1 (internal name sky1) as a vendor and device ID for PCI
> devices so we can use the macro for future drivers.

See note at top of file:

   *      Please keep sorted by numeric Vendor ID and Device ID.
   *
   *      Do not add new entries to this file unless the definitions
   *      are shared between multiple drivers.

Include this patch in a series that shows multiple drivers using it
and mention those drivers in the commit log.

Update the subject line to match the existing style (use "git log
--oneline include/linux/pci_ids.h"):

  PCI: Add Cix Technology Vendor and Device ID

> Signed-off-by: Hans Zhang <hans.zhang@cixtech.com>
> Reviewed-by: Peter Chen <peter.chen@cixtech.com>
> ---
>  include/linux/pci_ids.h | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h
> index 1a2594a38199..531b34c1181a 100644
> --- a/include/linux/pci_ids.h
> +++ b/include/linux/pci_ids.h
> @@ -200,6 +200,9 @@
>  #define PCI_DEVICE_ID_COMPAQ_THUNDER	0xf130
>  #define PCI_DEVICE_ID_COMPAQ_NETFLEX3B	0xf150
>  
> +#define PCI_VENDOR_ID_CIX               0x1f6c
> +#define PCI_DEVICE_ID_CIX_SKY1          0x0001

This is not sorted by Vendor ID.

>  #define PCI_VENDOR_ID_NCR		0x1000
>  #define PCI_VENDOR_ID_LSI_LOGIC		0x1000
>  #define PCI_DEVICE_ID_NCR_53C810	0x0001
> 
> base-commit: 99fa936e8e4f117d62f229003c9799686f74cebc
> -- 
> 2.47.1
> 

Re: [PATCH] PCI: pci_ids: Add Cixtech P1 Platforms vendor and device ID

[hans.zhang]

On 2025/3/6 01:49, Bjorn Helgaas wrote:
> EXTERNAL EMAIL
> 
> On Wed, Mar 05, 2025 at 02:30:18PM +0800, hans.zhang@cixtech.com wrote:
>> From: Hans Zhang <hans.zhang@cixtech.com>
>>
>> Add Cixtech P1 (internal name sky1) as a vendor and device ID for PCI
>> devices so we can use the macro for future drivers.
> 
> See note at top of file:
> 
>     *      Please keep sorted by numeric Vendor ID and Device ID.
>     *
>     *      Do not add new entries to this file unless the definitions
>     *      are shared between multiple drivers.
> 
> Include this patch in a series that shows multiple drivers using it
> and mention those drivers in the commit log.

OK. I will submit it together and use it in the future when our PCIe 
controller drives upsteam.

> 
> Update the subject line to match the existing style (use "git log
> --oneline include/linux/pci_ids.h"):
> 
>    PCI: Add Cix Technology Vendor and Device ID

Will change.

> 
>> Signed-off-by: Hans Zhang <hans.zhang@cixtech.com>
>> Reviewed-by: Peter Chen <peter.chen@cixtech.com>
>> ---
>>   include/linux/pci_ids.h | 3 +++
>>   1 file changed, 3 insertions(+)
>>
>> diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h
>> index 1a2594a38199..531b34c1181a 100644
>> --- a/include/linux/pci_ids.h
>> +++ b/include/linux/pci_ids.h
>> @@ -200,6 +200,9 @@
>>   #define PCI_DEVICE_ID_COMPAQ_THUNDER 0xf130
>>   #define PCI_DEVICE_ID_COMPAQ_NETFLEX3B       0xf150
>>
>> +#define PCI_VENDOR_ID_CIX               0x1f6c
>> +#define PCI_DEVICE_ID_CIX_SKY1          0x0001
> 
> This is not sorted by Vendor ID.

Will change.

> 
>>   #define PCI_VENDOR_ID_NCR            0x1000
>>   #define PCI_VENDOR_ID_LSI_LOGIC              0x1000
>>   #define PCI_DEVICE_ID_NCR_53C810     0x0001
>>
>> base-commit: 99fa936e8e4f117d62f229003c9799686f74cebc
>> --
>> 2.47.1
>>