[PATCH] platform/x86: intel-hid: add Rocket Lake ACPI device ID

[PATCH] platform/x86: intel-hid: add Rocket Lake ACPI device ID

[Alex Hung]

Rocket Lake has a new ACPI ID for Intel HID event filter device.

Signed-off-by: Alex Hung <alex.hung@canonical.com>
---
 drivers/platform/x86/intel-hid.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/platform/x86/intel-hid.c b/drivers/platform/x86/intel-hid.c
index 8626197..a65c48c 100644
--- a/drivers/platform/x86/intel-hid.c
+++ b/drivers/platform/x86/intel-hid.c
@@ -21,6 +21,7 @@ MODULE_AUTHOR("Alex Hung");
 static const struct acpi_device_id intel_hid_ids[] = {
 	{"INT33D5", 0},
 	{"INTC1051", 0},
+	{"INTC1054", 0},
 	{"", 0},
 };
 MODULE_DEVICE_TABLE(acpi, intel_hid_ids);
-- 
2.7.4

Re: [PATCH] platform/x86: intel-hid: add Rocket Lake ACPI device ID

[Alex Hung]

On 2020-12-07 11:27 p.m., Limonciello, Mario wrote:
>> Rocket Lake has a new ACPI ID for Intel HID event filter device.
> 
> Isn't RKL a "desktop" CPU?  IIRC all the existing events were mobile
> oriented.  Are there new events that come through for desktop?

Resent with plain text

The ID was for an AIO system so it is a desktop CPU indeed. I don't know
whether there are new events but the ID is needed to handle existing
events, such as power button events, that were handled in INT33D5
previously.

Re: [PATCH] platform/x86: intel-hid: add Rocket Lake ACPI device ID

[Hans de Goede]

Hi,

On 12/8/20 3:06 AM, Alex Hung wrote:
> Rocket Lake has a new ACPI ID for Intel HID event filter device.
> 
> Signed-off-by: Alex Hung <alex.hung@canonical.com>

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

Note it will show up in my review-hans branch once I've pushed my
local branch there, which might take a while.

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.

Regards,

Hans


> ---
>  drivers/platform/x86/intel-hid.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/platform/x86/intel-hid.c b/drivers/platform/x86/intel-hid.c
> index 8626197..a65c48c 100644
> --- a/drivers/platform/x86/intel-hid.c
> +++ b/drivers/platform/x86/intel-hid.c
> @@ -21,6 +21,7 @@ MODULE_AUTHOR("Alex Hung");
>  static const struct acpi_device_id intel_hid_ids[] = {
>  	{"INT33D5", 0},
>  	{"INTC1051", 0},
> +	{"INTC1054", 0},
>  	{"", 0},
>  };
>  MODULE_DEVICE_TABLE(acpi, intel_hid_ids);
>