Re: [PATCH 1/2] platform/x86: amd-pmc: Add alternative acpi id for PMC controller

From: Hans de Goede
Date: Mon Oct 11 2021 - 09:46:45 EST


Hi,

On 10/2/21 6:18 AM, Sachi King wrote:
> The Surface Laptop 4 AMD has used the AMD0005 to identify this
> controller instead of using the appropriate ACPI ID AMDI0005. Include
> AMD0005 in the acpi id list.
>
> Cc: <stable@xxxxxxxxxxxxxxx> # 5.14+
> Signed-off-by: Sachi King <nakato@xxxxxxxxx>

Thank you for your patch, I've applied this patch to my review-hans
branch:
https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=review-hans

Once I've run some tests on this branch the patches there will be
added to the platform-drivers-x86/for-next branch and eventually
will be included in the pdx86 pull-request to Linus for the next
merge-window.

I will also include this in my upcoming pdx86-fixes pull-req for 5.15 .

Regards,

Hans

> ---
> drivers/platform/x86/amd-pmc.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/platform/x86/amd-pmc.c b/drivers/platform/x86/amd-pmc.c
> index d6a7c896ac86..fc95620101e8 100644
> --- a/drivers/platform/x86/amd-pmc.c
> +++ b/drivers/platform/x86/amd-pmc.c
> @@ -476,6 +476,7 @@ static const struct acpi_device_id amd_pmc_acpi_ids[] = {
> {"AMDI0006", 0},
> {"AMDI0007", 0},
> {"AMD0004", 0},
> + {"AMD0005", 0},
> { }
> };
> MODULE_DEVICE_TABLE(acpi, amd_pmc_acpi_ids);
>