Re: [PATCH] platform/x86/amd/pmc: Declare quirk_spurious_8042 for MECHREVO Wujie 14XA (GX4HRXL)

[Mario Limonciello <mario.limonciello@amd.com>]

On 5/6/2025 6:25 AM, Runhua He wrote:
> MECHREVO Wujie 14XA (GX4HRXL) wakes up immediately after s2idle entry.
> This happens regardless of whether the laptop is plugged into AC power,
> or whether any peripheral is plugged into the laptop.
> 
> Similar to commit a55bdad5dfd1 ("platform/x86/amd/pmc: Disable keyboard
> wakeup on AMD Framework 13"), the MECHREVO Wujie 14XA wakes up almost
> instantly after s2idle suspend entry (IRQ1 is the keyboard):
> 
> 2025-04-18 17:23:57,588 DEBUG:  PM: Triggering wakeup from IRQ 9
> 2025-04-18 17:23:57,588 DEBUG:  PM: Triggering wakeup from IRQ 1
> 
> Add this model to the spurious_8042 quirk to workaround this.
> 
> This patch does not affect the wake-up function of the built-in keyboard.
> Because the firmware of this machine adds an insurance for keyboard
> wake-up events, as it always triggers an additional IRQ 9 to wake up the
> system.
> 
> I have only matched the motherboard model, as the same chassis and
> motherboard (GX4HRXL) combination may be used under different product
> names.
> 
> Suggested-by: Mingcong Bai <jeffbai@aosc.io>
> Suggested-by: Xinhui Yang <cyan@cyano.uk>
> Suggested-by: Rong Zhang <i@rong.moe>
> Fixes: a55bdad5dfd1 ("platform/x86/amd/pmc: Disable keyboard wakeup on AMD Framework 13")
> Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/4166
> Cc: Mario Limonciello <mario.limonciello@amd.com>
> Link: https://zhuanldan.zhihu.com/p/730538041
> Tested-by: Yemu Lu <prcups@krgm.moe>
> Signed-off-by: Runhua He <hua@aosc.io>
> ---
>   drivers/platform/x86/amd/pmc/pmc-quirks.c | 7 +++++++
>   1 file changed, 7 insertions(+)
> 
> diff --git a/drivers/platform/x86/amd/pmc/pmc-quirks.c b/drivers/platform/x86/amd/pmc/pmc-quirks.c
> index b4f49720c87f..e01012d5ecd0 100644
> --- a/drivers/platform/x86/amd/pmc/pmc-quirks.c
> +++ b/drivers/platform/x86/amd/pmc/pmc-quirks.c
> @@ -217,6 +217,13 @@ static const struct dmi_system_id fwbug_list[] = {
>   			DMI_MATCH(DMI_BIOS_VERSION, "03.05"),
>   		}
>   	},
> +	{
> +		.ident = "MECHREVO Wujie 14 Series (GX4HRXL)",
> +		.driver_data = &quirk_spurious_8042,
> +		.matches = {
> +			DMI_MATCH(DMI_BOARD_NAME, "GX4HRXL"),

This feels like it might be too wide.  Could we match a system vendor as 
well?