Attention is currently required from: Raul Rangel, Furquan Shaikh, Martin Roth, Mathew King.
Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52908 )
Change subject: mb/google/guybrush: Configure wake resource for WiFi
......................................................................
Patch Set 1:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/52908/comment/fd7ed953_3bc6fc31
PS1, Line 13: is added to SSDT.
It means that the OSPM is not really setting it correctly. What does `cat /proc/acpi/wakeup` show?
cat /proc/acpi/wakeup lists WF00 device
Interesting. It means that the GPE enable bit is somehow not getting set correctly. Do you see the GPE STS bit set correctly in your experiment now?
Yes. I also see the concerned bit set in SMI:event_status register in the resume path.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/52908
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic238d9606aea20c058e9b47093693f10b14e6288
Gerrit-Change-Number: 52908
Gerrit-PatchSet: 1
Gerrit-Owner: Karthik Ramasubramanian
kramasub@google.com
Gerrit-Reviewer: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: Martin Roth
martinroth@google.com
Gerrit-Reviewer: Mathew King
mathewk@chromium.org
Gerrit-Reviewer: Raul Rangel
rrangel@chromium.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Attention: Raul Rangel
rrangel@chromium.org
Gerrit-Attention: Furquan Shaikh
furquan@google.com
Gerrit-Attention: Martin Roth
martinroth@google.com
Gerrit-Attention: Mathew King
mathewk@chromium.org
Gerrit-Comment-Date: Mon, 10 May 2021 23:58:08 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Furquan Shaikh
furquan@google.com
Comment-In-Reply-To: Karthik Ramasubramanian
kramasub@google.com
Gerrit-MessageType: comment