* [PATCH] ACPI: resource: Do IRQ override on TongFang GMxXGxx
@ 2023-09-29 13:40 Werner Sembach
2023-10-03 18:39 ` Rafael J. Wysocki
0 siblings, 1 reply; 5+ messages in thread
From: Werner Sembach @ 2023-09-29 13:40 UTC (permalink / raw)
To: Rafael J. Wysocki, Len Brown; +Cc: Werner Sembach, linux-acpi, linux-kernel
The TongFang GMxXGxx/TUXEDO Stellaris/Pollaris Gen5 needs IRQ overriding
for the keyboard to work. Adding an entry for this laptop to the
override_table makes the internal keyboard functional again.
Signed-off-by: Werner Sembach <wse@tuxedocomputers.com>
Cc: <stable@vger.kernel.org>
---
drivers/acpi/resource.c | 16 ++++++++++++++--
1 file changed, 14 insertions(+), 2 deletions(-)
diff --git a/drivers/acpi/resource.c b/drivers/acpi/resource.c
index 32cfa3f4efd3d..3aff81820e321 100644
--- a/drivers/acpi/resource.c
+++ b/drivers/acpi/resource.c
@@ -470,13 +470,25 @@ static const struct dmi_system_id asus_laptop[] = {
{ }
};
-static const struct dmi_system_id tongfang_gm_rg[] = {
+static const struct dmi_system_id tongfang_gm[] = {
{
.ident = "TongFang GMxRGxx/XMG CORE 15 (M22)/TUXEDO Stellaris 15 Gen4 AMD",
.matches = {
DMI_MATCH(DMI_BOARD_NAME, "GMxRGxx"),
},
},
+ {
+ .ident = "TongFang GMxXGxx/TUXEDO Polaris 15 Gen5 AMD",
+ .matches = {
+ DMI_MATCH(DMI_BOARD_NAME, "GMxXGxx"),
+ },
+ },
+ {
+ .ident = "TongFang GM6XGxX/TUXEDO Stellaris 16 Gen5 AMD",
+ .matches = {
+ DMI_MATCH(DMI_BOARD_NAME, "GM6XGxX"),
+ },
+ },
{ }
};
@@ -536,7 +548,7 @@ struct irq_override_cmp {
static const struct irq_override_cmp override_table[] = {
{ medion_laptop, 1, ACPI_LEVEL_SENSITIVE, ACPI_ACTIVE_LOW, 0, false },
{ asus_laptop, 1, ACPI_LEVEL_SENSITIVE, ACPI_ACTIVE_LOW, 0, false },
- { tongfang_gm_rg, 1, ACPI_EDGE_SENSITIVE, ACPI_ACTIVE_LOW, 1, true },
+ { tongfang_gm, 1, ACPI_EDGE_SENSITIVE, ACPI_ACTIVE_LOW, 1, true },
{ maingear_laptop, 1, ACPI_EDGE_SENSITIVE, ACPI_ACTIVE_LOW, 1, true },
{ pcspecialist_laptop, 1, ACPI_EDGE_SENSITIVE, ACPI_ACTIVE_LOW, 1, true },
{ lg_laptop, 1, ACPI_LEVEL_SENSITIVE, ACPI_ACTIVE_LOW, 0, false },
--
2.34.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] ACPI: resource: Do IRQ override on TongFang GMxXGxx
2023-09-29 13:40 [PATCH] ACPI: resource: Do IRQ override on TongFang GMxXGxx Werner Sembach
@ 2023-10-03 18:39 ` Rafael J. Wysocki
2023-10-04 9:38 ` Werner Sembach
0 siblings, 1 reply; 5+ messages in thread
From: Rafael J. Wysocki @ 2023-10-03 18:39 UTC (permalink / raw)
To: Werner Sembach; +Cc: Rafael J. Wysocki, Len Brown, linux-acpi, linux-kernel
On Fri, Sep 29, 2023 at 3:40 PM Werner Sembach <wse@tuxedocomputers.com> wrote:
>
> The TongFang GMxXGxx/TUXEDO Stellaris/Pollaris Gen5 needs IRQ overriding
> for the keyboard to work. Adding an entry for this laptop to the
> override_table makes the internal keyboard functional again.
You said "again", so it used to work. Do you know which commit broke it?
> Signed-off-by: Werner Sembach <wse@tuxedocomputers.com>
> Cc: <stable@vger.kernel.org>
What's the oldest kernel version you want this to be applied to?
> ---
> drivers/acpi/resource.c | 16 ++++++++++++++--
> 1 file changed, 14 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/acpi/resource.c b/drivers/acpi/resource.c
> index 32cfa3f4efd3d..3aff81820e321 100644
> --- a/drivers/acpi/resource.c
> +++ b/drivers/acpi/resource.c
> @@ -470,13 +470,25 @@ static const struct dmi_system_id asus_laptop[] = {
> { }
> };
>
> -static const struct dmi_system_id tongfang_gm_rg[] = {
> +static const struct dmi_system_id tongfang_gm[] = {
> {
> .ident = "TongFang GMxRGxx/XMG CORE 15 (M22)/TUXEDO Stellaris 15 Gen4 AMD",
> .matches = {
> DMI_MATCH(DMI_BOARD_NAME, "GMxRGxx"),
> },
> },
> + {
> + .ident = "TongFang GMxXGxx/TUXEDO Polaris 15 Gen5 AMD",
> + .matches = {
> + DMI_MATCH(DMI_BOARD_NAME, "GMxXGxx"),
> + },
> + },
> + {
> + .ident = "TongFang GM6XGxX/TUXEDO Stellaris 16 Gen5 AMD",
> + .matches = {
> + DMI_MATCH(DMI_BOARD_NAME, "GM6XGxX"),
> + },
> + },
> { }
> };
>
> @@ -536,7 +548,7 @@ struct irq_override_cmp {
> static const struct irq_override_cmp override_table[] = {
> { medion_laptop, 1, ACPI_LEVEL_SENSITIVE, ACPI_ACTIVE_LOW, 0, false },
> { asus_laptop, 1, ACPI_LEVEL_SENSITIVE, ACPI_ACTIVE_LOW, 0, false },
> - { tongfang_gm_rg, 1, ACPI_EDGE_SENSITIVE, ACPI_ACTIVE_LOW, 1, true },
> + { tongfang_gm, 1, ACPI_EDGE_SENSITIVE, ACPI_ACTIVE_LOW, 1, true },
> { maingear_laptop, 1, ACPI_EDGE_SENSITIVE, ACPI_ACTIVE_LOW, 1, true },
> { pcspecialist_laptop, 1, ACPI_EDGE_SENSITIVE, ACPI_ACTIVE_LOW, 1, true },
> { lg_laptop, 1, ACPI_LEVEL_SENSITIVE, ACPI_ACTIVE_LOW, 0, false },
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ACPI: resource: Do IRQ override on TongFang GMxXGxx
2023-10-03 18:39 ` Rafael J. Wysocki
@ 2023-10-04 9:38 ` Werner Sembach
2023-10-06 19:30 ` Rafael J. Wysocki
0 siblings, 1 reply; 5+ messages in thread
From: Werner Sembach @ 2023-10-04 9:38 UTC (permalink / raw)
To: Rafael J. Wysocki; +Cc: Len Brown, linux-acpi, linux-kernel
Hi,
Am 03.10.23 um 20:39 schrieb Rafael J. Wysocki:
> On Fri, Sep 29, 2023 at 3:40 PM Werner Sembach <wse@tuxedocomputers.com> wrote:
>> The TongFang GMxXGxx/TUXEDO Stellaris/Pollaris Gen5 needs IRQ overriding
>> for the keyboard to work. Adding an entry for this laptop to the
>> override_table makes the internal keyboard functional again.
> You said "again", so it used to work. Do you know which commit broke it?
Sorry that "again" is a bit misleading. I Copied it from here:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=7592b79ba4a9
I actually only tested the direct predecessor of this Notebook on older kernels:
https://bugzilla.kernel.org/show_bug.cgi?id=216698
I suspected that it's the same issue and since the same fix worked I now even
more believe so, but I actually never tested a kernel before the breaking change.
>
>> Signed-off-by: Werner Sembach <wse@tuxedocomputers.com>
>> Cc: <stable@vger.kernel.org>
> What's the oldest kernel version you want this to be applied to?
5.15 because Ubuntu 22.04
Kind regards,
Werner Sembach
>
>> ---
>> drivers/acpi/resource.c | 16 ++++++++++++++--
>> 1 file changed, 14 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/acpi/resource.c b/drivers/acpi/resource.c
>> index 32cfa3f4efd3d..3aff81820e321 100644
>> --- a/drivers/acpi/resource.c
>> +++ b/drivers/acpi/resource.c
>> @@ -470,13 +470,25 @@ static const struct dmi_system_id asus_laptop[] = {
>> { }
>> };
>>
>> -static const struct dmi_system_id tongfang_gm_rg[] = {
>> +static const struct dmi_system_id tongfang_gm[] = {
>> {
>> .ident = "TongFang GMxRGxx/XMG CORE 15 (M22)/TUXEDO Stellaris 15 Gen4 AMD",
>> .matches = {
>> DMI_MATCH(DMI_BOARD_NAME, "GMxRGxx"),
>> },
>> },
>> + {
>> + .ident = "TongFang GMxXGxx/TUXEDO Polaris 15 Gen5 AMD",
>> + .matches = {
>> + DMI_MATCH(DMI_BOARD_NAME, "GMxXGxx"),
>> + },
>> + },
>> + {
>> + .ident = "TongFang GM6XGxX/TUXEDO Stellaris 16 Gen5 AMD",
>> + .matches = {
>> + DMI_MATCH(DMI_BOARD_NAME, "GM6XGxX"),
>> + },
>> + },
>> { }
>> };
>>
>> @@ -536,7 +548,7 @@ struct irq_override_cmp {
>> static const struct irq_override_cmp override_table[] = {
>> { medion_laptop, 1, ACPI_LEVEL_SENSITIVE, ACPI_ACTIVE_LOW, 0, false },
>> { asus_laptop, 1, ACPI_LEVEL_SENSITIVE, ACPI_ACTIVE_LOW, 0, false },
>> - { tongfang_gm_rg, 1, ACPI_EDGE_SENSITIVE, ACPI_ACTIVE_LOW, 1, true },
>> + { tongfang_gm, 1, ACPI_EDGE_SENSITIVE, ACPI_ACTIVE_LOW, 1, true },
>> { maingear_laptop, 1, ACPI_EDGE_SENSITIVE, ACPI_ACTIVE_LOW, 1, true },
>> { pcspecialist_laptop, 1, ACPI_EDGE_SENSITIVE, ACPI_ACTIVE_LOW, 1, true },
>> { lg_laptop, 1, ACPI_LEVEL_SENSITIVE, ACPI_ACTIVE_LOW, 0, false },
>> --
>> 2.34.1
>>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ACPI: resource: Do IRQ override on TongFang GMxXGxx
2023-10-04 9:38 ` Werner Sembach
@ 2023-10-06 19:30 ` Rafael J. Wysocki
2023-10-09 18:03 ` Werner Sembach
0 siblings, 1 reply; 5+ messages in thread
From: Rafael J. Wysocki @ 2023-10-06 19:30 UTC (permalink / raw)
To: Werner Sembach; +Cc: Rafael J. Wysocki, Len Brown, linux-acpi, linux-kernel
On Wed, Oct 4, 2023 at 11:38 AM Werner Sembach <wse@tuxedocomputers.com> wrote:
>
> Hi,
>
> Am 03.10.23 um 20:39 schrieb Rafael J. Wysocki:
> > On Fri, Sep 29, 2023 at 3:40 PM Werner Sembach <wse@tuxedocomputers.com> wrote:
> >> The TongFang GMxXGxx/TUXEDO Stellaris/Pollaris Gen5 needs IRQ overriding
> >> for the keyboard to work. Adding an entry for this laptop to the
> >> override_table makes the internal keyboard functional again.
> > You said "again", so it used to work. Do you know which commit broke it?
>
> Sorry that "again" is a bit misleading. I Copied it from here:
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=7592b79ba4a9
>
> I actually only tested the direct predecessor of this Notebook on older kernels:
> https://bugzilla.kernel.org/show_bug.cgi?id=216698
>
> I suspected that it's the same issue and since the same fix worked I now even
> more believe so, but I actually never tested a kernel before the breaking change.
>
> >
> >> Signed-off-by: Werner Sembach <wse@tuxedocomputers.com>
> >> Cc: <stable@vger.kernel.org>
> > What's the oldest kernel version you want this to be applied to?
>
> 5.15 because Ubuntu 22.04
So first of all Hans posted this:
https://patchwork.kernel.org/project/linux-acpi/patch/20231006123304.32686-1-hdegoede@redhat.com/
and I'm wondering if it addresses the problem for you too.
If it doesn't, the patch will need to be rebased on top of the new
acpi-resource material in linux-next and retested.
Thanks!
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ACPI: resource: Do IRQ override on TongFang GMxXGxx
2023-10-06 19:30 ` Rafael J. Wysocki
@ 2023-10-09 18:03 ` Werner Sembach
0 siblings, 0 replies; 5+ messages in thread
From: Werner Sembach @ 2023-10-09 18:03 UTC (permalink / raw)
To: Rafael J. Wysocki; +Cc: Len Brown, linux-acpi, linux-kernel
Hi
Am 06.10.23 um 21:30 schrieb Rafael J. Wysocki:
> On Wed, Oct 4, 2023 at 11:38 AM Werner Sembach <wse@tuxedocomputers.com> wrote:
>> Hi,
>>
>> Am 03.10.23 um 20:39 schrieb Rafael J. Wysocki:
>>> On Fri, Sep 29, 2023 at 3:40 PM Werner Sembach <wse@tuxedocomputers.com> wrote:
>>>> The TongFang GMxXGxx/TUXEDO Stellaris/Pollaris Gen5 needs IRQ overriding
>>>> for the keyboard to work. Adding an entry for this laptop to the
>>>> override_table makes the internal keyboard functional again.
>>> You said "again", so it used to work. Do you know which commit broke it?
>> Sorry that "again" is a bit misleading. I Copied it from here:
>> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=7592b79ba4a9
>>
>> I actually only tested the direct predecessor of this Notebook on older kernels:
>> https://bugzilla.kernel.org/show_bug.cgi?id=216698
>>
>> I suspected that it's the same issue and since the same fix worked I now even
>> more believe so, but I actually never tested a kernel before the breaking change.
>>
>>>> Signed-off-by: Werner Sembach <wse@tuxedocomputers.com>
>>>> Cc: <stable@vger.kernel.org>
>>> What's the oldest kernel version you want this to be applied to?
>> 5.15 because Ubuntu 22.04
> So first of all Hans posted this:
>
> https://patchwork.kernel.org/project/linux-acpi/patch/20231006123304.32686-1-hdegoede@redhat.com/
>
> and I'm wondering if it addresses the problem for you too.
Thanks for the link. That patch seems to do the same, but for a different board
name so it wont work for the GMxXGxx.
>
> If it doesn't, the patch will need to be rebased on top of the new
> acpi-resource material in linux-next and retested.
Sure, no problem, v2 incoming soon.
Kind regards,
Werner
>
> Thanks!
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-10-09 18:03 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-29 13:40 [PATCH] ACPI: resource: Do IRQ override on TongFang GMxXGxx Werner Sembach
2023-10-03 18:39 ` Rafael J. Wysocki
2023-10-04 9:38 ` Werner Sembach
2023-10-06 19:30 ` Rafael J. Wysocki
2023-10-09 18:03 ` Werner Sembach
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).