Attention is currently required from: Mary Ruthven, Vadim Bendebury, Joel Kitching, Furquan Shaikh, Christian Walter, Julius Werner.
Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51164 )
Change subject: security/tpm/tss/vendor/cr50: Introduce vendor sub-command to reset EC
......................................................................
Patch Set 3:
(1 comment)
File src/security/tpm/tss/vendor/cr50/cr50.h:
https://review.coreboot.org/c/coreboot/+/51164/comment/48229bd9_008ee9e3
PS1, Line 104: Issue a halt() after triggering EC reset because cr50 leaves 50 ms after receiving
: * the command to reset the EC.
I agree that it does not make sense to return in case of success.
Done
--
To view, visit
https://review.coreboot.org/c/coreboot/+/51164
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I46063678511d27fea5eabbd12fc3af0b1df68143
Gerrit-Change-Number: 51164
Gerrit-PatchSet: 3
Gerrit-Owner: Karthik Ramasubramanian
kramasub@google.com
Gerrit-Reviewer: Christian Walter
christian.walter@9elements.com
Gerrit-Reviewer: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: Joel Kitching
kitching@google.com
Gerrit-Reviewer: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Mary Ruthven
mruthven@google.com
Gerrit-Reviewer: Vadim Bendebury
vbendeb@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Vadim Bendebury
vbendeb@chromium.org
Gerrit-Attention: Mary Ruthven
mruthven@google.com
Gerrit-Attention: Vadim Bendebury
vbendeb@chromium.org
Gerrit-Attention: Joel Kitching
kitching@google.com
Gerrit-Attention: Furquan Shaikh
furquan@google.com
Gerrit-Attention: Vadim Bendebury
vbendeb@google.com
Gerrit-Attention: Christian Walter
christian.walter@9elements.com
Gerrit-Attention: Julius Werner
jwerner@chromium.org
Gerrit-Comment-Date: Tue, 02 Mar 2021 20:14:25 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Vadim Bendebury
vbendeb@chromium.org
Comment-In-Reply-To: Furquan Shaikh
furquan@google.com
Comment-In-Reply-To: Karthik Ramasubramanian
kramasub@google.com
Gerrit-MessageType: comment