diff mbox series

[v2] ACPI: resource: Add TongFang GM6BGEQ, GM6BG5Q and GM6BG0Q to irq1_edge_low_force_override[]

Message ID 20231006123304.32686-1-hdegoede@redhat.com (mailing list archive)
State Superseded, archived
Delegated to: Rafael Wysocki
Headers show
Series [v2] ACPI: resource: Add TongFang GM6BGEQ, GM6BG5Q and GM6BG0Q to irq1_edge_low_force_override[] | expand

Commit Message

Hans de Goede Oct. 6, 2023, 12:33 p.m. UTC
The TongFang GM6BGEQ, GM6BG5Q and GM6BG0Q are 3 GPU variants of a TongFang
barebone design which is sold under various brand names.

The ACPI IRQ override for the keyboard IRQ must be used on these AMD Zen
laptops in order for the IRQ to work.

Adjust the irq1_edge_low_force_override[] DMI match table for this:

1. Drop the sys-vendor match from the existing PCSpecialist Elimina Pro 16
   entry for the GM6BGEQ (RTX3050 GPU) model so that it will also match
   the laptop when sold by other vendors such as hyperbook.pl.

2. Add board-name matches for the GM6BG5Q (RTX4050) and GM6B0Q (RTX4060)
   models.

Suggested-by: August Wikerfors <git@augustwikerfors.se>
Reported-by: Francesco <f.littarru@outlook.com>
Closes: https://bugzilla.kernel.org/show_bug.cgi?id=217394
Link: https://laptopparts4less.frl/index.php?route=product/search&filter_name=GM6BG
Link: https://hyperbook.pl/en/content/14-hyperbook-drivers
Link: https://linux-hardware.org/?probe=bfa70344e3
Link: https://bbs.archlinuxcn.org/viewtopic.php?id=13313
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
Changes in v2:
- Add missing reported and suggested by tags
---
 drivers/acpi/resource.c | 20 +++++++++++++-------
 1 file changed, 13 insertions(+), 7 deletions(-)

Comments

August Wikerfors Oct. 6, 2023, 2:18 p.m. UTC | #1
Hi Hans,

On 2023-10-06 14:33, Hans de Goede wrote:
> The TongFang GM6BGEQ, GM6BG5Q and GM6BG0Q are 3 GPU variants of a TongFang
> barebone design which is sold under various brand names.
> 
> The ACPI IRQ override for the keyboard IRQ must be used on these AMD Zen
> laptops in order for the IRQ to work.
> 
> Adjust the irq1_edge_low_force_override[] DMI match table for this:
> 
> 1. Drop the sys-vendor match from the existing PCSpecialist Elimina Pro 16
>     entry for the GM6BGEQ (RTX3050 GPU) model so that it will also match
>     the laptop when sold by other vendors such as hyperbook.pl.
> 
> 2. Add board-name matches for the GM6BG5Q (RTX4050) and GM6B0Q (RTX4060)
>     models.
> 
> Suggested-by: August Wikerfors <git@augustwikerfors.se>
> Reported-by: Francesco <f.littarru@outlook.com>
> Closes: https://bugzilla.kernel.org/show_bug.cgi?id=217394
> Link: https://laptopparts4less.frl/index.php?route=product/search&filter_name=GM6BG
> Link: https://hyperbook.pl/en/content/14-hyperbook-drivers
> Link: https://linux-hardware.org/?probe=bfa70344e3
> Link: https://bbs.archlinuxcn.org/viewtopic.php?id=13313
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Since this is a regression fix for 453b014e2c29 ("ACPI: resource: Fix 
IRQ override quirk for PCSpecialist Elimina Pro 16 M") (for PCSpecialist 
systems like Francesco's with product name "Elimina Pro 16 M" but not 
board name "GM6BGEQ") and 2d331a6ac481 ("ACPI: resource: revert "Remove 
"Zen" specific match and quirks"") (for other vendors using the same 
TongFang design), it should have a "Fixes:" tag for at least one of those.

Both of those commits are in 6.5 (and 6.6-rc) so this should go into 
6.6-rc and be backported to 6.5, but the patch seems to depend on 
424009ab2030 ("ACPI: resource: Drop .ident values from dmi_system_id 
tables") and maybe also d37273af0e42 ("ACPI: resource: Consolidate IRQ 
trigger-type override DMI tables") to apply cleanly, which seem to only 
be queued for linux-next/6.7? I'm not familiar with what the correct 
process is for such cases.

> ---
> Changes in v2:
> - Add missing reported and suggested by tags
> ---
>   drivers/acpi/resource.c | 20 +++++++++++++-------
>   1 file changed, 13 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/acpi/resource.c b/drivers/acpi/resource.c
> index 014a3911381b..18f6353c142e 100644
> --- a/drivers/acpi/resource.c
> +++ b/drivers/acpi/resource.c
> @@ -512,17 +512,23 @@ static const struct dmi_system_id irq1_edge_low_force_override[] = {
>   		},
>   	},
>   	{
> -		/*
> -		 * PCSpecialist Elimina Pro 16 M
> -		 *
> -		 * Some models have product-name "Elimina Pro 16 M",
> -		 * others "GM6BGEQ". Match on board-name to match both.
> -		 */
> +		/* TongFang GM6BGEQ / PCSpecialist Elimina Pro 16 M, RTX 3050 */
>   		.matches = {
> -			DMI_MATCH(DMI_SYS_VENDOR, "PCSpecialist"),
>   			DMI_MATCH(DMI_BOARD_NAME, "GM6BGEQ"),
>   		},
>   	},
> +	{
> +		/* TongFang GM6BG5Q, RTX 4050 */
FWIW the PCSpecialist Elimina Pro 16 M is sold with all three GPU 
options: https://www.pcspecialist.ie/notebooks/elimina-pro-16-M/
Presumably the RTX 4050 option has board name "GM6BG5Q" but maybe it's 
better to leave it out if it's not confirmed.

> +		.matches = {
> +			DMI_MATCH(DMI_BOARD_NAME, "GM6BG5Q"),
> +		},
> +	},
> +	{
> +		/* TongFang GM6BG0Q / PCSpecialist Elimina Pro 16 M, RTX 4060 */
> +		.matches = {
> +			DMI_MATCH(DMI_BOARD_NAME, "GM6BG0Q"),
> +		},
> +	},
>   	{ }
>   };
>   

Regards,
August Wikerfors
Hans de Goede Oct. 6, 2023, 3:17 p.m. UTC | #2
Hi August,

On 10/6/23 16:18, August Wikerfors wrote:
> Hi Hans,
> 
> On 2023-10-06 14:33, Hans de Goede wrote:
>> The TongFang GM6BGEQ, GM6BG5Q and GM6BG0Q are 3 GPU variants of a TongFang
>> barebone design which is sold under various brand names.
>>
>> The ACPI IRQ override for the keyboard IRQ must be used on these AMD Zen
>> laptops in order for the IRQ to work.
>>
>> Adjust the irq1_edge_low_force_override[] DMI match table for this:
>>
>> 1. Drop the sys-vendor match from the existing PCSpecialist Elimina Pro 16
>>     entry for the GM6BGEQ (RTX3050 GPU) model so that it will also match
>>     the laptop when sold by other vendors such as hyperbook.pl.
>>
>> 2. Add board-name matches for the GM6BG5Q (RTX4050) and GM6B0Q (RTX4060)
>>     models.
>>
>> Suggested-by: August Wikerfors <git@augustwikerfors.se>
>> Reported-by: Francesco <f.littarru@outlook.com>
>> Closes: https://bugzilla.kernel.org/show_bug.cgi?id=217394
>> Link: https://laptopparts4less.frl/index.php?route=product/search&filter_name=GM6BG
>> Link: https://hyperbook.pl/en/content/14-hyperbook-drivers
>> Link: https://linux-hardware.org/?probe=bfa70344e3
>> Link: https://bbs.archlinuxcn.org/viewtopic.php?id=13313
>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> Since this is a regression fix for 453b014e2c29 ("ACPI: resource: Fix IRQ override quirk for PCSpecialist Elimina Pro 16 M") (for PCSpecialist systems like Francesco's with product name "Elimina Pro 16 M" but not board name "GM6BGEQ") and 2d331a6ac481 ("ACPI: resource: revert "Remove "Zen" specific match and quirks"") (for other vendors using the same TongFang design), it should have a "Fixes:" tag for at least one of those.
> 
> Both of those commits are in 6.5 (and 6.6-rc) so this should go into 6.6-rc and be backported to 6.5, but the patch seems to depend on 424009ab2030 ("ACPI: resource: Drop .ident values from dmi_system_id tables") and maybe also d37273af0e42 ("ACPI: resource: Consolidate IRQ trigger-type override DMI tables") to apply cleanly, which seem to only be queued for linux-next/6.7? I'm not familiar with what the correct process is for such cases.

You are right:

Fixes: 453b014e2c29 ("ACPI: resource: Fix IRQ override quirk for PCSpecialist Elimina Pro 16 M")

Note likewise this commit should really also be send as a fix
for 6.6 and backported to the stable kernels:

https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git/commit/drivers/acpi/resource.c?h=bleeding-edge&id=c1ed72171ed580fbf159e703b77685aa4b0d0df5

Regards,

Hans



> 
>> ---
>> Changes in v2:
>> - Add missing reported and suggested by tags
>> ---
>>   drivers/acpi/resource.c | 20 +++++++++++++-------
>>   1 file changed, 13 insertions(+), 7 deletions(-)
>>
>> diff --git a/drivers/acpi/resource.c b/drivers/acpi/resource.c
>> index 014a3911381b..18f6353c142e 100644
>> --- a/drivers/acpi/resource.c
>> +++ b/drivers/acpi/resource.c
>> @@ -512,17 +512,23 @@ static const struct dmi_system_id irq1_edge_low_force_override[] = {
>>           },
>>       },
>>       {
>> -        /*
>> -         * PCSpecialist Elimina Pro 16 M
>> -         *
>> -         * Some models have product-name "Elimina Pro 16 M",
>> -         * others "GM6BGEQ". Match on board-name to match both.
>> -         */
>> +        /* TongFang GM6BGEQ / PCSpecialist Elimina Pro 16 M, RTX 3050 */
>>           .matches = {
>> -            DMI_MATCH(DMI_SYS_VENDOR, "PCSpecialist"),
>>               DMI_MATCH(DMI_BOARD_NAME, "GM6BGEQ"),
>>           },
>>       },
>> +    {
>> +        /* TongFang GM6BG5Q, RTX 4050 */
> FWIW the PCSpecialist Elimina Pro 16 M is sold with all three GPU options: https://www.pcspecialist.ie/notebooks/elimina-pro-16-M/
> Presumably the RTX 4050 option has board name "GM6BG5Q" but maybe it's better to leave it out if it's not confirmed.
> 
>> +        .matches = {
>> +            DMI_MATCH(DMI_BOARD_NAME, "GM6BG5Q"),
>> +        },
>> +    },
>> +    {
>> +        /* TongFang GM6BG0Q / PCSpecialist Elimina Pro 16 M, RTX 4060 */
>> +        .matches = {
>> +            DMI_MATCH(DMI_BOARD_NAME, "GM6BG0Q"),
>> +        },
>> +    },
>>       { }
>>   };
>>   
> 
> Regards,
> August Wikerfors
>
Rafael J. Wysocki Oct. 6, 2023, 7:20 p.m. UTC | #3
On Fri, Oct 6, 2023 at 5:17 PM Hans de Goede <hdegoede@redhat.com> wrote:
>
> Hi August,
>
> On 10/6/23 16:18, August Wikerfors wrote:
> > Hi Hans,
> >
> > On 2023-10-06 14:33, Hans de Goede wrote:
> >> The TongFang GM6BGEQ, GM6BG5Q and GM6BG0Q are 3 GPU variants of a TongFang
> >> barebone design which is sold under various brand names.
> >>
> >> The ACPI IRQ override for the keyboard IRQ must be used on these AMD Zen
> >> laptops in order for the IRQ to work.
> >>
> >> Adjust the irq1_edge_low_force_override[] DMI match table for this:
> >>
> >> 1. Drop the sys-vendor match from the existing PCSpecialist Elimina Pro 16
> >>     entry for the GM6BGEQ (RTX3050 GPU) model so that it will also match
> >>     the laptop when sold by other vendors such as hyperbook.pl.
> >>
> >> 2. Add board-name matches for the GM6BG5Q (RTX4050) and GM6B0Q (RTX4060)
> >>     models.
> >>
> >> Suggested-by: August Wikerfors <git@augustwikerfors.se>
> >> Reported-by: Francesco <f.littarru@outlook.com>
> >> Closes: https://bugzilla.kernel.org/show_bug.cgi?id=217394
> >> Link: https://laptopparts4less.frl/index.php?route=product/search&filter_name=GM6BG
> >> Link: https://hyperbook.pl/en/content/14-hyperbook-drivers
> >> Link: https://linux-hardware.org/?probe=bfa70344e3
> >> Link: https://bbs.archlinuxcn.org/viewtopic.php?id=13313
> >> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> > Since this is a regression fix for 453b014e2c29 ("ACPI: resource: Fix IRQ override quirk for PCSpecialist Elimina Pro 16 M") (for PCSpecialist systems like Francesco's with product name "Elimina Pro 16 M" but not board name "GM6BGEQ") and 2d331a6ac481 ("ACPI: resource: revert "Remove "Zen" specific match and quirks"") (for other vendors using the same TongFang design), it should have a "Fixes:" tag for at least one of those.
> >
> > Both of those commits are in 6.5 (and 6.6-rc) so this should go into 6.6-rc and be backported to 6.5, but the patch seems to depend on 424009ab2030 ("ACPI: resource: Drop .ident values from dmi_system_id tables") and maybe also d37273af0e42 ("ACPI: resource: Consolidate IRQ trigger-type override DMI tables") to apply cleanly, which seem to only be queued for linux-next/6.7? I'm not familiar with what the correct process is for such cases.
>
> You are right:
>
> Fixes: 453b014e2c29 ("ACPI: resource: Fix IRQ override quirk for PCSpecialist Elimina Pro 16 M")

OK

> Note likewise this commit should really also be send as a fix
> for 6.6 and backported to the stable kernels:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git/commit/drivers/acpi/resource.c?h=bleeding-edge&id=c1ed72171ed580fbf159e703b77685aa4b0d0df5

That's harder, because it does depend on commit 424009ab2030 ("ACPI:
resource: Drop .ident values from dmi_system_id tables") which is not
6.6-rc material IMV.

So I'm going to queue this up with the Fixes tag above and Cc: stable
pointing to commit 424009ab2030 as a dependency, but for 6.7.
Hans de Goede Oct. 7, 2023, 8:51 a.m. UTC | #4
Hi Rafael,

On 10/6/23 21:20, Rafael J. Wysocki wrote:
> On Fri, Oct 6, 2023 at 5:17 PM Hans de Goede <hdegoede@redhat.com> wrote:
>>
>> Hi August,
>>
>> On 10/6/23 16:18, August Wikerfors wrote:
>>> Hi Hans,
>>>
>>> On 2023-10-06 14:33, Hans de Goede wrote:
>>>> The TongFang GM6BGEQ, GM6BG5Q and GM6BG0Q are 3 GPU variants of a TongFang
>>>> barebone design which is sold under various brand names.
>>>>
>>>> The ACPI IRQ override for the keyboard IRQ must be used on these AMD Zen
>>>> laptops in order for the IRQ to work.
>>>>
>>>> Adjust the irq1_edge_low_force_override[] DMI match table for this:
>>>>
>>>> 1. Drop the sys-vendor match from the existing PCSpecialist Elimina Pro 16
>>>>     entry for the GM6BGEQ (RTX3050 GPU) model so that it will also match
>>>>     the laptop when sold by other vendors such as hyperbook.pl.
>>>>
>>>> 2. Add board-name matches for the GM6BG5Q (RTX4050) and GM6B0Q (RTX4060)
>>>>     models.
>>>>
>>>> Suggested-by: August Wikerfors <git@augustwikerfors.se>
>>>> Reported-by: Francesco <f.littarru@outlook.com>
>>>> Closes: https://bugzilla.kernel.org/show_bug.cgi?id=217394
>>>> Link: https://laptopparts4less.frl/index.php?route=product/search&filter_name=GM6BG
>>>> Link: https://hyperbook.pl/en/content/14-hyperbook-drivers
>>>> Link: https://linux-hardware.org/?probe=bfa70344e3
>>>> Link: https://bbs.archlinuxcn.org/viewtopic.php?id=13313
>>>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
>>> Since this is a regression fix for 453b014e2c29 ("ACPI: resource: Fix IRQ override quirk for PCSpecialist Elimina Pro 16 M") (for PCSpecialist systems like Francesco's with product name "Elimina Pro 16 M" but not board name "GM6BGEQ") and 2d331a6ac481 ("ACPI: resource: revert "Remove "Zen" specific match and quirks"") (for other vendors using the same TongFang design), it should have a "Fixes:" tag for at least one of those.
>>>
>>> Both of those commits are in 6.5 (and 6.6-rc) so this should go into 6.6-rc and be backported to 6.5, but the patch seems to depend on 424009ab2030 ("ACPI: resource: Drop .ident values from dmi_system_id tables") and maybe also d37273af0e42 ("ACPI: resource: Consolidate IRQ trigger-type override DMI tables") to apply cleanly, which seem to only be queued for linux-next/6.7? I'm not familiar with what the correct process is for such cases.
>>
>> You are right:
>>
>> Fixes: 453b014e2c29 ("ACPI: resource: Fix IRQ override quirk for PCSpecialist Elimina Pro 16 M")
> 
> OK
> 
>> Note likewise this commit should really also be send as a fix
>> for 6.6 and backported to the stable kernels:
>>
>> https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git/commit/drivers/acpi/resource.c?h=bleeding-edge&id=c1ed72171ed580fbf159e703b77685aa4b0d0df5
> 
> That's harder, because it does depend on commit 424009ab2030 ("ACPI:
> resource: Drop .ident values from dmi_system_id tables") which is not
> 6.6-rc material IMV.
> 
> So I'm going to queue this up with the Fixes tag above and Cc: stable
> pointing to commit 424009ab2030 as a dependency, but for 6.7.

I realize that I'm the architect of this whole mess, but delaying
these new quirks to 6.7 does not seem like a good idea to me.

This fixes a serious problem (kbd not working pretty much makes
the whole laptop unusable under Linux at least on the road). With
a very small chance of regressions since this is DMI match base.

Maybe we can just take all the resource.c DMI quirk changes
as fixes to 6.6 ?  I admit the:

"ACPI: resource: Consolidate IRQ trigger-type override DMI tables"
"ACPI: resource: Drop .ident values from dmi_system_id tables"

patches are not really bugfixes. But they too have a very low
chance of causing regressions.

Regards,

Hans


p.s.

Note that "ACPI: resource: Skip IRQ override on ASUS ExpertBook B1402CBA"
should probably go as a fix to 6.6 regardless of the discussion since
that is a pure bugfix which applies cleanly on top of 6.6

And the same goes for this (unrelated) drivers/acpi/ec.c patch:

"ACPI: EC: Add quirk for the HP Pavilion Gaming 15-dk1xxx"

That too is a pure bugfix which IMHO should go to 6.6 as such.
Rafael J. Wysocki Oct. 8, 2023, 12:16 p.m. UTC | #5
On Sat, Oct 7, 2023 at 10:51 AM Hans de Goede <hdegoede@redhat.com> wrote:
>
> Hi Rafael,
>
> On 10/6/23 21:20, Rafael J. Wysocki wrote:
> > On Fri, Oct 6, 2023 at 5:17 PM Hans de Goede <hdegoede@redhat.com> wrote:
> >>
> >> Hi August,
> >>
> >> On 10/6/23 16:18, August Wikerfors wrote:
> >>> Hi Hans,
> >>>
> >>> On 2023-10-06 14:33, Hans de Goede wrote:
> >>>> The TongFang GM6BGEQ, GM6BG5Q and GM6BG0Q are 3 GPU variants of a TongFang
> >>>> barebone design which is sold under various brand names.
> >>>>
> >>>> The ACPI IRQ override for the keyboard IRQ must be used on these AMD Zen
> >>>> laptops in order for the IRQ to work.
> >>>>
> >>>> Adjust the irq1_edge_low_force_override[] DMI match table for this:
> >>>>
> >>>> 1. Drop the sys-vendor match from the existing PCSpecialist Elimina Pro 16
> >>>>     entry for the GM6BGEQ (RTX3050 GPU) model so that it will also match
> >>>>     the laptop when sold by other vendors such as hyperbook.pl.
> >>>>
> >>>> 2. Add board-name matches for the GM6BG5Q (RTX4050) and GM6B0Q (RTX4060)
> >>>>     models.
> >>>>
> >>>> Suggested-by: August Wikerfors <git@augustwikerfors.se>
> >>>> Reported-by: Francesco <f.littarru@outlook.com>
> >>>> Closes: https://bugzilla.kernel.org/show_bug.cgi?id=217394
> >>>> Link: https://laptopparts4less.frl/index.php?route=product/search&filter_name=GM6BG
> >>>> Link: https://hyperbook.pl/en/content/14-hyperbook-drivers
> >>>> Link: https://linux-hardware.org/?probe=bfa70344e3
> >>>> Link: https://bbs.archlinuxcn.org/viewtopic.php?id=13313
> >>>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> >>> Since this is a regression fix for 453b014e2c29 ("ACPI: resource: Fix IRQ override quirk for PCSpecialist Elimina Pro 16 M") (for PCSpecialist systems like Francesco's with product name "Elimina Pro 16 M" but not board name "GM6BGEQ") and 2d331a6ac481 ("ACPI: resource: revert "Remove "Zen" specific match and quirks"") (for other vendors using the same TongFang design), it should have a "Fixes:" tag for at least one of those.
> >>>
> >>> Both of those commits are in 6.5 (and 6.6-rc) so this should go into 6.6-rc and be backported to 6.5, but the patch seems to depend on 424009ab2030 ("ACPI: resource: Drop .ident values from dmi_system_id tables") and maybe also d37273af0e42 ("ACPI: resource: Consolidate IRQ trigger-type override DMI tables") to apply cleanly, which seem to only be queued for linux-next/6.7? I'm not familiar with what the correct process is for such cases.
> >>
> >> You are right:
> >>
> >> Fixes: 453b014e2c29 ("ACPI: resource: Fix IRQ override quirk for PCSpecialist Elimina Pro 16 M")
> >
> > OK
> >
> >> Note likewise this commit should really also be send as a fix
> >> for 6.6 and backported to the stable kernels:
> >>
> >> https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git/commit/drivers/acpi/resource.c?h=bleeding-edge&id=c1ed72171ed580fbf159e703b77685aa4b0d0df5
> >
> > That's harder, because it does depend on commit 424009ab2030 ("ACPI:
> > resource: Drop .ident values from dmi_system_id tables") which is not
> > 6.6-rc material IMV.
> >
> > So I'm going to queue this up with the Fixes tag above and Cc: stable
> > pointing to commit 424009ab2030 as a dependency, but for 6.7.
>
> I realize that I'm the architect of this whole mess, but delaying
> these new quirks to 6.7 does not seem like a good idea to me.
>
> This fixes a serious problem (kbd not working pretty much makes
> the whole laptop unusable under Linux at least on the road). With
> a very small chance of regressions since this is DMI match base.
>
> Maybe we can just take all the resource.c DMI quirk changes
> as fixes to 6.6 ?  I admit the:
>
> "ACPI: resource: Consolidate IRQ trigger-type override DMI tables"
> "ACPI: resource: Drop .ident values from dmi_system_id tables"
>
> patches are not really bugfixes. But they too have a very low
> chance of causing regressions.
>
> Regards,
>
> Hans
>
>
> p.s.
>
> Note that "ACPI: resource: Skip IRQ override on ASUS ExpertBook B1402CBA"
> should probably go as a fix to 6.6 regardless of the discussion since
> that is a pure bugfix which applies cleanly on top of 6.6

That can be done.

> And the same goes for this (unrelated) drivers/acpi/ec.c patch:
>
> "ACPI: EC: Add quirk for the HP Pavilion Gaming 15-dk1xxx"

That can be done too.

> That too is a pure bugfix which IMHO should go to 6.6 as such.

But if you want the $subject patch to go into 6.6-rc, please rebase it
on top of the latest 6.6-rc and resubmit.  It will not apply cleanly
on top of the current linux-next, but I can handle the merge conflict.
Hans de Goede Oct. 9, 2023, 12:10 p.m. UTC | #6
Hi,

On 10/8/23 14:16, Rafael J. Wysocki wrote:
> On Sat, Oct 7, 2023 at 10:51 AM Hans de Goede <hdegoede@redhat.com> wrote:

<snip>

>> I realize that I'm the architect of this whole mess, but delaying
>> these new quirks to 6.7 does not seem like a good idea to me.
>>
>> This fixes a serious problem (kbd not working pretty much makes
>> the whole laptop unusable under Linux at least on the road). With
>> a very small chance of regressions since this is DMI match base.
>>
>> Maybe we can just take all the resource.c DMI quirk changes
>> as fixes to 6.6 ?  I admit the:
>>
>> "ACPI: resource: Consolidate IRQ trigger-type override DMI tables"
>> "ACPI: resource: Drop .ident values from dmi_system_id tables"
>>
>> patches are not really bugfixes. But they too have a very low
>> chance of causing regressions.
>>
>> Regards,
>>
>> Hans
>>
>>
>> p.s.
>>
>> Note that "ACPI: resource: Skip IRQ override on ASUS ExpertBook B1402CBA"
>> should probably go as a fix to 6.6 regardless of the discussion since
>> that is a pure bugfix which applies cleanly on top of 6.6
> 
> That can be done.

Great thank you.

>> And the same goes for this (unrelated) drivers/acpi/ec.c patch:
>>
>> "ACPI: EC: Add quirk for the HP Pavilion Gaming 15-dk1xxx"
> 
> That can be done too.

Thank you again.

>> That too is a pure bugfix which IMHO should go to 6.6 as such.
> 
> But if you want the $subject patch to go into 6.6-rc, please rebase it
> on top of the latest 6.6-rc and resubmit.  It will not apply cleanly
> on top of the current linux-next, but I can handle the merge conflict.

That works for me, thank yuo. I've prepared a v3 which is based on
v6.6-rc1 so that it should apply cleanly for merging as fix for v6.6.

I'll send v3 out right after this email.

Regards,

Hans
diff mbox series

Patch

diff --git a/drivers/acpi/resource.c b/drivers/acpi/resource.c
index 014a3911381b..18f6353c142e 100644
--- a/drivers/acpi/resource.c
+++ b/drivers/acpi/resource.c
@@ -512,17 +512,23 @@  static const struct dmi_system_id irq1_edge_low_force_override[] = {
 		},
 	},
 	{
-		/*
-		 * PCSpecialist Elimina Pro 16 M
-		 *
-		 * Some models have product-name "Elimina Pro 16 M",
-		 * others "GM6BGEQ". Match on board-name to match both.
-		 */
+		/* TongFang GM6BGEQ / PCSpecialist Elimina Pro 16 M, RTX 3050 */
 		.matches = {
-			DMI_MATCH(DMI_SYS_VENDOR, "PCSpecialist"),
 			DMI_MATCH(DMI_BOARD_NAME, "GM6BGEQ"),
 		},
 	},
+	{
+		/* TongFang GM6BG5Q, RTX 4050 */
+		.matches = {
+			DMI_MATCH(DMI_BOARD_NAME, "GM6BG5Q"),
+		},
+	},
+	{
+		/* TongFang GM6BG0Q / PCSpecialist Elimina Pro 16 M, RTX 4060 */
+		.matches = {
+			DMI_MATCH(DMI_BOARD_NAME, "GM6BG0Q"),
+		},
+	},
 	{ }
 };