Re: [PATCH] ACPI: resource: Skip IRQ override on Asus Vivobook Go E1404GAB

[Thorsten Leemhuis]

[CCing Hans]

On 05.09.24 12:45, Thorsten Leemhuis wrote:
> On 05.09.24 11:51, Thorsten Leemhuis wrote:
>> On 03.09.24 03:43, Tamim Khan wrote:
>>> Like other Asus Vivobooks, the Asus Vivobook Go E1404GAB has a DSDT
>>> that describes IRQ 1 as ActiveLow, while the kernel overrides to Edge_High.
>>> This override prevents the internal keyboard from working. This patch fixes
>>> this problem by adding this laptop to the table that prevents the kernel from
>>> overriding the IRQ.
>>>
>>> Link: https://bugzilla.kernel.org/show_bug.cgi?id=219212
>>
>> Thx for that. FWIW, I by chance noticed another such report about the
>> E1404GA: https://bugzilla.kernel.org/show_bug.cgi?id=219224
> 
> TWIMC, shortly after sending this mail I noticed there is another request
> for a quirk that was send to the list, bug afaics fall through the
> cracks. See here:
> https://lore.kernel.org/all/1226760b-4699-4529-bf57-6423938157a3@xxxxxxxxxx/
> 
> It afaics add a X1704VAP:
> 
> 	{
> 		/* Asus Vivobook X1704V */
> 		.matches = {
> 			DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
> 			DMI_MATCH(DMI_BOARD_NAME, "X1704VAP"),
> 		},
> 	},

FWIW, noticed another such report, this time about the B2502CVA:
https://bugzilla.kernel.org/show_bug.cgi?id=217760#c12

"""
> adding this section to file drivers/acpi/resource.c helped; if someone could do an official patch it would be great :-|
> 
>         {
>                 /* Asus ExpertBook B2502CVA */
>                 .matches = {
>                         DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
>                         DMI_MATCH(DMI_BOARD_NAME, "B2502CVA"),
>                 },
"""

This is not a regression, but given how frequently I notice these issues
please as a bystander allow me to ask: is there maybe some way to
improve the situation so that we do not have to add all these and likely
many other Asus Laptops to that file?

Ciao, Thorsten