acpi/ec: Add device to acpi_ec_no_wakeup qurik

[PATCH] acpi/ec: Add device to acpi_ec_no_wakeup qurik

Posted by Werner Sembach 7 months, 1 week ago

Add the TUXEDO InfinityBook Pro AMD Gen9 to the acpi_ec_no_wakeup quirk
list to prevent spurious wakeups.

Signed-off-by: Werner Sembach <wse@tuxedocomputers.com>
---
 drivers/acpi/ec.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/drivers/acpi/ec.c b/drivers/acpi/ec.c
index 8db09d81918fb..de45a5b59effd 100644
--- a/drivers/acpi/ec.c
+++ b/drivers/acpi/ec.c
@@ -2301,6 +2301,12 @@ static const struct dmi_system_id acpi_ec_no_wakeup[] = {
 			DMI_MATCH(DMI_PRODUCT_FAMILY, "103C_5336AN HP ZHAN 66 Pro"),
 		},
 	},
+	{
+		// TUXEDO InfinityBook Pro AMD Gen9
+		.matches = {
+			DMI_MATCH(DMI_BOARD_NAME, "GXxHRXx"),
+		},
+	},
 	{ },
 };
 
-- 
2.43.0

Re: [PATCH] acpi/ec: Add device to acpi_ec_no_wakeup qurik

Posted by Rafael J. Wysocki 7 months, 1 week ago

On Thu, May 8, 2025 at 1:16 PM Werner Sembach <wse@tuxedocomputers.com> wrote:
>
> Add the TUXEDO InfinityBook Pro AMD Gen9 to the acpi_ec_no_wakeup quirk
> list to prevent spurious wakeups.
>
> Signed-off-by: Werner Sembach <wse@tuxedocomputers.com>
> ---
>  drivers/acpi/ec.c | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/drivers/acpi/ec.c b/drivers/acpi/ec.c
> index 8db09d81918fb..de45a5b59effd 100644
> --- a/drivers/acpi/ec.c
> +++ b/drivers/acpi/ec.c
> @@ -2301,6 +2301,12 @@ static const struct dmi_system_id acpi_ec_no_wakeup[] = {
>                         DMI_MATCH(DMI_PRODUCT_FAMILY, "103C_5336AN HP ZHAN 66 Pro"),
>                 },
>         },
> +       {
> +               // TUXEDO InfinityBook Pro AMD Gen9
> +               .matches = {
> +                       DMI_MATCH(DMI_BOARD_NAME, "GXxHRXx"),
> +               },
> +       },
>         { },
>  };
>
> --

Applied as 6.16 material, thanks!

Re: [PATCH] acpi/ec: Add device to acpi_ec_no_wakeup qurik

Posted by Werner Sembach 7 months, 1 week ago

Am 09.05.25 um 21:21 schrieb Rafael J. Wysocki:
> On Thu, May 8, 2025 at 1:16 PM Werner Sembach <wse@tuxedocomputers.com> wrote:
>> Add the TUXEDO InfinityBook Pro AMD Gen9 to the acpi_ec_no_wakeup quirk
>> list to prevent spurious wakeups.
>>
>> Signed-off-by: Werner Sembach <wse@tuxedocomputers.com>
>> ---
>>   drivers/acpi/ec.c | 6 ++++++
>>   1 file changed, 6 insertions(+)
>>
>> diff --git a/drivers/acpi/ec.c b/drivers/acpi/ec.c
>> index 8db09d81918fb..de45a5b59effd 100644
>> --- a/drivers/acpi/ec.c
>> +++ b/drivers/acpi/ec.c
>> @@ -2301,6 +2301,12 @@ static const struct dmi_system_id acpi_ec_no_wakeup[] = {
>>                          DMI_MATCH(DMI_PRODUCT_FAMILY, "103C_5336AN HP ZHAN 66 Pro"),
>>                  },
>>          },
>> +       {
>> +               // TUXEDO InfinityBook Pro AMD Gen9
>> +               .matches = {
>> +                       DMI_MATCH(DMI_BOARD_NAME, "GXxHRXx"),
>> +               },
>> +       },
>>          { },
>>   };
>>
>> --
> Applied as 6.16 material, thanks!
Forgot to add cc stable but i guess i will just wait till it reaches mainline 
and then send it manually?

Re: [PATCH] acpi/ec: Add device to acpi_ec_no_wakeup qurik

Posted by Rafael J. Wysocki 7 months, 1 week ago

On Fri, May 9, 2025 at 9:49 PM Werner Sembach <wse@tuxedocomputers.com> wrote:
>
>
> Am 09.05.25 um 21:21 schrieb Rafael J. Wysocki:
> > On Thu, May 8, 2025 at 1:16 PM Werner Sembach <wse@tuxedocomputers.com> wrote:
> >> Add the TUXEDO InfinityBook Pro AMD Gen9 to the acpi_ec_no_wakeup quirk
> >> list to prevent spurious wakeups.
> >>
> >> Signed-off-by: Werner Sembach <wse@tuxedocomputers.com>
> >> ---
> >>   drivers/acpi/ec.c | 6 ++++++
> >>   1 file changed, 6 insertions(+)
> >>
> >> diff --git a/drivers/acpi/ec.c b/drivers/acpi/ec.c
> >> index 8db09d81918fb..de45a5b59effd 100644
> >> --- a/drivers/acpi/ec.c
> >> +++ b/drivers/acpi/ec.c
> >> @@ -2301,6 +2301,12 @@ static const struct dmi_system_id acpi_ec_no_wakeup[] = {
> >>                          DMI_MATCH(DMI_PRODUCT_FAMILY, "103C_5336AN HP ZHAN 66 Pro"),
> >>                  },
> >>          },
> >> +       {
> >> +               // TUXEDO InfinityBook Pro AMD Gen9
> >> +               .matches = {
> >> +                       DMI_MATCH(DMI_BOARD_NAME, "GXxHRXx"),
> >> +               },
> >> +       },
> >>          { },
> >>   };
> >>
> >> --
> > Applied as 6.16 material, thanks!
> Forgot to add cc stable but i guess i will just wait till it reaches mainline
> and then send it manually?

I can add a "stable" tag to it, but it won't apply to older kernels
anyway because the table has grown recently.

It's better to send manually when it gets merged I think.

Re: [PATCH] acpi/ec: Add device to acpi_ec_no_wakeup qurik

Posted by Werner Sembach 7 months, 1 week ago

Am 09.05.25 um 22:23 schrieb Rafael J. Wysocki:
> On Fri, May 9, 2025 at 9:49 PM Werner Sembach <wse@tuxedocomputers.com> wrote:
>>
>> Am 09.05.25 um 21:21 schrieb Rafael J. Wysocki:
>>> On Thu, May 8, 2025 at 1:16 PM Werner Sembach <wse@tuxedocomputers.com> wrote:
>>>> Add the TUXEDO InfinityBook Pro AMD Gen9 to the acpi_ec_no_wakeup quirk
>>>> list to prevent spurious wakeups.
>>>>
>>>> Signed-off-by: Werner Sembach <wse@tuxedocomputers.com>
>>>> ---
>>>>    drivers/acpi/ec.c | 6 ++++++
>>>>    1 file changed, 6 insertions(+)
>>>>
>>>> diff --git a/drivers/acpi/ec.c b/drivers/acpi/ec.c
>>>> index 8db09d81918fb..de45a5b59effd 100644
>>>> --- a/drivers/acpi/ec.c
>>>> +++ b/drivers/acpi/ec.c
>>>> @@ -2301,6 +2301,12 @@ static const struct dmi_system_id acpi_ec_no_wakeup[] = {
>>>>                           DMI_MATCH(DMI_PRODUCT_FAMILY, "103C_5336AN HP ZHAN 66 Pro"),
>>>>                   },
>>>>           },
>>>> +       {
>>>> +               // TUXEDO InfinityBook Pro AMD Gen9
>>>> +               .matches = {
>>>> +                       DMI_MATCH(DMI_BOARD_NAME, "GXxHRXx"),
>>>> +               },
>>>> +       },
>>>>           { },
>>>>    };
>>>>
>>>> --
>>> Applied as 6.16 material, thanks!
>> Forgot to add cc stable but i guess i will just wait till it reaches mainline
>> and then send it manually?
> I can add a "stable" tag to it, but it won't apply to older kernels
> anyway because the table has grown recently.
>
> It's better to send manually when it gets merged I think.
ok