Shelley Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31100 )
Change subject: mb/google/hatch: Set up Wake GPIO from EC
......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/#/c/31100/1//COMMIT_MSG
Commit Message:
https://review.coreboot.org/#/c/31100/1//COMMIT_MSG@13
PS1, Line 13: Make sure device wakes up
Did lidopen, keyboard press also wake up the AP?
No, but I couldn't see any hostevents set for either of those, so I thought that it was because the EC wasn't actually setting PCH_WAKE_L (I didn't see it set when I did a gpioget PCH_WAKE_L). So, I tested it by gpiosetting PCH_WAKE_L to 0 and the device did indeed wake up as a result.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/31100
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I53d5291a6b9ab9a21e89ccd21f172180ce473bd5
Gerrit-Change-Number: 31100
Gerrit-PatchSet: 2
Gerrit-Owner: Shelley Chen
shchen@google.com
Gerrit-Reviewer: Aamir Bohra
aamir.bohra@intel.com
Gerrit-Reviewer: Aaron Durbin
adurbin@chromium.org
Gerrit-Reviewer: Duncan Laurie
dlaurie@chromium.org
Gerrit-Reviewer: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: Scott Collyer
scollyer@google.com
Gerrit-Reviewer: Shelley Chen
shchen@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Comment-Date: Fri, 25 Jan 2019 22:59:24 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Furquan Shaikh
furquan@google.com
Gerrit-MessageType: comment