Attention is currently required from: Frank Chu, Isaac Lee, Tim Wawrzynczak, Zhuohao Lee.
Ken Lu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56302 )
Change subject: mb/google/volteer/variants/collis: Redefine GPIO_EC_IN_RW to GPP_F17
......................................................................
Patch Set 4:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/56302/comment/b090bf79_db04444f
PS1, Line 11: none
Create a bug for this. I think the FAFT test should catch this problem.
Done
https://review.coreboot.org/c/coreboot/+/56302/comment/caad08da_c22cc6d4
PS1, Line 12: none
firmware-volteer-13672. […]
Done
https://review.coreboot.org/c/coreboot/+/56302/comment/adda27f9_cd7d7ea2
PS1, Line 13: none
It shouldn't be none. You need to make sure the pin is working correctly. […]
Done
--
To view, visit
https://review.coreboot.org/c/coreboot/+/56302
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I24f4803dc99ef3fc78852241f3a9e86ec70293d9
Gerrit-Change-Number: 56302
Gerrit-PatchSet: 4
Gerrit-Owner: Frank Chu
frank_chu@pegatron.corp-partner.google.com
Gerrit-Reviewer: Isaac Lee
isaaclee@google.com
Gerrit-Reviewer: Tim Wawrzynczak
twawrzynczak@chromium.org
Gerrit-Reviewer: Zhuohao Lee
zhuohao@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Anson Tseng
ansontseng@google.com
Gerrit-CC: Hank Lin
hank2_lin@pegatron.corp-partner.google.com
Gerrit-CC: Ken Lu
ken_lu@pegatron.corp-partner.google.com
Gerrit-Attention: Frank Chu
frank_chu@pegatron.corp-partner.google.com
Gerrit-Attention: Isaac Lee
isaaclee@google.com
Gerrit-Attention: Tim Wawrzynczak
twawrzynczak@chromium.org
Gerrit-Attention: Zhuohao Lee
zhuohao@google.com
Gerrit-Comment-Date: Thu, 15 Jul 2021 02:51:16 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Zhuohao Lee
zhuohao@google.com
Gerrit-MessageType: comment