diff mbox series

ACPI: PM: Quirk ASUS ROG M16 to default to S3 sleep

Message ID 20240908053607.4213-1-luke@ljones.dev
State Accepted
Commit 6924e9b2ac60817335a27f126274d8e88d5d818f
Headers show
Series ACPI: PM: Quirk ASUS ROG M16 to default to S3 sleep | expand

Commit Message

Luke D. Jones Sept. 8, 2024, 5:36 a.m. UTC
The 2023 ASUS ROG Zephyrus M16 can suffer from quite a variety of events
causing wakeup from s2idle sleep. The events may come from the EC being
noisey, from the MMC reader, from the AniMe matrix display on some models
or from AC events.

Defaulting to S3 sleep prevents all these wakeup issues.

Signed-off-by: Luke D. Jones <luke@ljones.dev>
---
 drivers/acpi/sleep.c | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

Comments

Rafael J. Wysocki Sept. 10, 2024, 6:21 p.m. UTC | #1
On Sun, Sep 8, 2024 at 7:36 AM Luke D. Jones <luke@ljones.dev> wrote:
>
> The 2023 ASUS ROG Zephyrus M16 can suffer from quite a variety of events
> causing wakeup from s2idle sleep. The events may come from the EC being
> noisey, from the MMC reader, from the AniMe matrix display on some models
> or from AC events.
>
> Defaulting to S3 sleep prevents all these wakeup issues.
>
> Signed-off-by: Luke D. Jones <luke@ljones.dev>
> ---
>  drivers/acpi/sleep.c | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
>
> diff --git a/drivers/acpi/sleep.c b/drivers/acpi/sleep.c
> index 889f1c1a1fa9..c8ee8e42b0f6 100644
> --- a/drivers/acpi/sleep.c
> +++ b/drivers/acpi/sleep.c
> @@ -351,6 +351,20 @@ static const struct dmi_system_id acpisleep_dmi_table[] __initconst = {
>                 DMI_MATCH(DMI_PRODUCT_NAME, "1025C"),
>                 },
>         },
> +       /*
> +        * The ASUS ROG M16 from 2023 has many events which wake it from s2idle
> +        * resulting in excessive battery drain and risk of laptop overheating,
> +        * these events can be caused by the MMC or  y AniMe display if installed.
> +        * The match is valid for all of the GU604V<x> range.
> +        */
> +       {
> +       .callback = init_default_s3,
> +       .ident = "ASUS ROG Zephyrus M16 (2023)",
> +       .matches = {
> +               DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
> +               DMI_MATCH(DMI_PRODUCT_NAME, "ROG Zephyrus M16 GU604V"),
> +               },
> +       },
>         /*
>          * https://bugzilla.kernel.org/show_bug.cgi?id=189431
>          * Lenovo G50-45 is a platform later than 2012, but needs nvs memory
> --

Applied as 6.12 material, thanks!
diff mbox series

Patch

diff --git a/drivers/acpi/sleep.c b/drivers/acpi/sleep.c
index 889f1c1a1fa9..c8ee8e42b0f6 100644
--- a/drivers/acpi/sleep.c
+++ b/drivers/acpi/sleep.c
@@ -351,6 +351,20 @@  static const struct dmi_system_id acpisleep_dmi_table[] __initconst = {
 		DMI_MATCH(DMI_PRODUCT_NAME, "1025C"),
 		},
 	},
+	/*
+	 * The ASUS ROG M16 from 2023 has many events which wake it from s2idle
+	 * resulting in excessive battery drain and risk of laptop overheating,
+	 * these events can be caused by the MMC or  y AniMe display if installed.
+	 * The match is valid for all of the GU604V<x> range.
+	 */
+	{
+	.callback = init_default_s3,
+	.ident = "ASUS ROG Zephyrus M16 (2023)",
+	.matches = {
+		DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
+		DMI_MATCH(DMI_PRODUCT_NAME, "ROG Zephyrus M16 GU604V"),
+		},
+	},
 	/*
 	 * https://bugzilla.kernel.org/show_bug.cgi?id=189431
 	 * Lenovo G50-45 is a platform later than 2012, but needs nvs memory