Re: [PATCH] hwmon: (k10temp): Add PCI IDs for 1ah/80h

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] The new PCI device ID `PCI_DEVICE_ID_AMD_1AH_M80H_DF_F3` is inappropriately added to the global `include/linux/pci_ids.h` instead of being defined locally in the driver.
--

commit 7a5a1abca982d7be5902a75b129f16bd3cb32d69
Author: Mario Limonciello <mario.limonciello@amd.com>

hwmon: (k10temp): Add PCI IDs for 1ah/80h

This commit adds a new PCI ID to support AMD Family 1Ah model 80h processors
in the k10temp hardware monitoring driver.

[ ... ]

> diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h
> index 63774ae2bf71d..57a4ad9984798 100644
> --- a/include/linux/pci_ids.h
> +++ b/include/linux/pci_ids.h
> @@ -585,6 +585,7 @@
>  #define PCI_DEVICE_ID_AMD_1AH_M20H_DF_F3 0x16fb
>  #define PCI_DEVICE_ID_AMD_1AH_M60H_DF_F3 0x124b
>  #define PCI_DEVICE_ID_AMD_1AH_M70H_DF_F3 0x12bb
> +#define PCI_DEVICE_ID_AMD_1AH_M80H_DF_F3 0x1243

[Severity: Low]
Should this new device ID be defined locally in drivers/hwmon/k10temp.c
instead of the global include/linux/pci_ids.h header?

There is an existing comment in drivers/hwmon/k10temp.c that establishes
the practice for these specific devices:

/*
 * PCI Device IDs for AMD's Family 1Ah-based SOCs.
 * Defining locally as IDs are not shared.
 */

Since this ID does not appear to be shared with other drivers, would it
be more appropriate to define it directly in the k10temp driver to avoid
cluttering the global PCI IDs header?

>  #define PCI_DEVICE_ID_AMD_MI200_DF_F3	0x14d3
>  #define PCI_DEVICE_ID_AMD_MI300_DF_F3	0x152b
>  #define PCI_DEVICE_ID_AMD_VANGOGH_USB	0x163a

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260911134304.226286-1-mario.limonciello@amd.com?part=1