Bug 216768

Summary:	Asus Tuf A15 Ryzen 6800H - Lost ACPI events after resume from suspend
Product:	Power Management	Reporter:	Paul (paul)
Component:	Hibernation/Suspend	Assignee:	Mario Limonciello (AMD) (mario.limonciello)
Status:	RESOLVED CODE_FIX
Severity:	normal	CC:	bilkow, fkrueger, mario.limonciello
Priority:	P1
Hardware:	AMD
OS:	Linux
Kernel Version:	6.1.0-rc7	Subsystem:
Regression:	No	Bisected commit-id:
Attachments:	ACPI dump dmidecode dump dmesg dump PATCH V1 dmidecode for FA507RE

Description Paul 2022-12-03 16:32:31 UTC

Comment 1 Paul 2022-12-03 16:35:55 UTC

Created attachment 303347 [details]
ACPI dump

Comment 2 Paul 2022-12-03 16:36:35 UTC

Created attachment 303348 [details]
dmidecode dump

Comment 3 Paul 2022-12-03 16:37:46 UTC

Created attachment 303349 [details]
dmesg dump

Comment 4 Paul 2022-12-03 16:46:19 UTC

This is a follow up from:

https://bugzilla.kernel.org/show_bug.cgi?id=216101

Tested on latest BIOS revision available and on 6.1.0-rc7.

The issue persists with the addition that the display brightness fn keys do not work, even before the laptop has been in suspend mode. I hope the logs can help.

(apologies for the edits, a field was highlighted in the form and I pressed enter, which submitted the report)

Comment 5 Mario Limonciello (AMD) 2022-12-05 21:19:02 UTC

Can you please have a try with the kernel command line option 'acpi.prefer_microsoft_dsm_guid=1'?

If that helps, then please try the attached patch (with no parameter).

Comment 6 Mario Limonciello (AMD) 2022-12-05 21:27:34 UTC

Created attachment 303364 [details]
PATCH V1

Comment 7 bilkow 2022-12-09 16:34:12 UTC

Hi, I also have an Asus Tuf A15 (2022) Ryzen 6800H, but my version is FA507RE.

Trying 'acpi.prefer_microsoft_dsm_guid=1' on Linux 6.1.0-RC8 works for me. I haven't tried the patch as it's only for FA507RM.

One thing to note is that if you match as you did for the A17 https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git/commit/?h=bleeding-edge&id=d0f61e89f08dd46a090da50f5d747204673f70ea, it may also patch the 2021 version, which has either a Ryzen 9 5900HX or Ryzen 5 4600H and may not have that same problem.

Comment 8 bilkow 2022-12-09 16:35:27 UTC

Also, I forgot to mention that without the parameter I only had problems after suspend. Before suspend my brightness keys always worked.

Comment 9 bilkow 2022-12-09 16:36:58 UTC

Created attachment 303389 [details]
dmidecode for FA507RE

Comment 10 Mario Limonciello (AMD) 2022-12-15 19:46:06 UTC

I just sent out this series https://lore.kernel.org/linux-acpi/20221215191617.1438-1-mario.limonciello@amd.com/T/#t for review that drops the Rembrandt behavior and parameter.  Here's a new branch https://gitlab.freedesktop.org/superm1/linux/-/commits/mlimonci/gitlab-2292-v3 with the same patches.

I expect it should help both of you guys.

Comment 11 bilkow 2022-12-17 02:42:20 UTC

Thanks a lot!

Tried your branch and resuming works as expected.

Comment 12 Mario Limonciello (AMD) 2022-12-31 05:57:42 UTC

Patches are merged into Linus' master tree.  6.2-rc2 or later have them merged, and they are CC to 6.1 stable so should eventually be there too.

Comment 13 Mario Limonciello (AMD) 2023-01-04 18:51:18 UTC

It's merged for 6.1.3.