Re: [PATCH v3] usb: xhci-pci: Add AMD 600 series to xhci-pci-prom21

[Stephen Horvath]

Hi Mirko.

On 4/10/26 10:27, Mirko Jechow wrote:
> From: Stephen Horvath <linux@xxxxxxxxxxxx>
> 
> It seems the xhci-pci-prom21 module only has IDs for the 800 series
> chipsets. However, the 600 series chipsets also have Promontory 21
> chips, so we should add their IDs to the list as well.
> 
> I have tested this on my X670 board featuring DID `0x43f7`.
> 
> `1022:43f7` also appears on the PCI ID database
> (https://pci-ids.ucw.cz/read/PC/1022/43f7).
> Device IDs `0x43f9` (B665) & `0x43fa` (A620) were suggested by Shyam.
> 
> Also list the new IDs in Documentation/hwmon/prom21-xhci.rst.
> 
> Cc: stable@xxxxxxxxxxxxxxx # 7.2.x
> Cc: Shyam Sundar S K <Shyam-sundar.S-k@xxxxxxx>
> Cc: Jihong Min <hurryman2212@xxxxxxxxx>
> Signed-off-by: Stephen Horvath <linux@xxxxxxxxxxxx>
> Tested-by: Mirko Jechow <mirko.jechow@xxxxxx>
> [mirko.jechow@xxxxxx: add new IDs to prom21-xhci.rst]
> Signed-off-by: Mirko Jechow <mirko.jechow@xxxxxx>
> ---
> Changes in v3:
> - List the new IDs on the PCI IDs line of Documentation/hwmon/prom21-xhci.rst
>   (requested by Jihong Min)
> - Picked up from Stephen; v2 had no follow-up since 2026-08-23

Thanks for picking this up! Sorry I've been busy and forgot about this.

> - Link to v2: https://patch.msgid.link/20260820-xhci-pci-prom21-v2-1-638e5958fbbf@xxxxxxxxxxxx
> ---
>  Documentation/hwmon/prom21-xhci.rst | 2 +-
>  drivers/usb/host/xhci-pci-prom21.c  | 3 +++
>  drivers/usb/host/xhci-pci.c         | 3 +++
>  drivers/usb/host/xhci-pci.h         | 3 +++
>  4 files changed, 10 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/hwmon/prom21-xhci.rst b/Documentation/hwmon/prom21-xhci.rst
> index 7984fb187bd8a..6c0b6e7368921 100644
> --- a/Documentation/hwmon/prom21-xhci.rst
> +++ b/Documentation/hwmon/prom21-xhci.rst
> @@ -9,7 +9,7 @@ Supported chips:
>  
>      Prefix: 'prom21_xhci'
>  
> -    PCI IDs: 1022:43fc, 1022:43fd
> +    PCI IDs: 1022:43f7, 1022:43f9, 1022:43fa, 1022:43fc, 1022:43fd
>  
>  Author:
>  

Looks good to me!

Thanks,
Steve