Attention is currently required from: Raul Rangel, Martin Roth, Mathew King, Karthik Ramasubramanian. Furquan Shaikh 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/a32487d9_b86795dc PS1, Line 13: is added to SSDT.
I am trying to wake on WiFi. Unfortunately the device is not waking up. […]
Can you please try configuring the pad to `PAD_SCI` instead of `PAD_NF_SCI`. I had asked this question on https://review.coreboot.org/c/coreboot/+/51756/comment/21b7f1ba_4d7691e6/. But, it was conveniently ignored.
Reference from zork: b/162605108#comment6
If changing to PAD_SCI does not help, you can check a few things: 1. Is the mapping of GPIO_2 to GEVENT_8 looks correct in the OS? 2. Is the GPE_EN bit set when going into suspend? (probably will have to use S3 for this so that you can dump the state just before suspending). 3. Is GPE_STS bit set on resume? 4. Is the behavior same in case of S3 and S0i3?