Attention is currently required from: Ricardo Quesada, Jack Rosenthal.
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56883 )
Change subject: elogtool: add "clear" command
......................................................................
Patch Set 13: Code-Review+2
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/56883/comment/fab7cdb9_3cd09307
PS13, Line 10: using flashrom
flashrom or file write if an input file is provided.
https://review.coreboot.org/c/coreboot/+/56883/comment/fadf1aab_1a4b47ea
PS13, Line 22:
TEST=?
Did you try `elogtool list` after `elogtool clear` to see if it correctly clears the elog region and adds ELOG_TYPE_LOG_CLEAR?
--
To view, visit
https://review.coreboot.org/c/coreboot/+/56883
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ia28a6eb34c82103ab078a0841b022e2e5e430585
Gerrit-Change-Number: 56883
Gerrit-PatchSet: 13
Gerrit-Owner: Ricardo Quesada
ricardoq@google.com
Gerrit-Reviewer: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: Jack Rosenthal
jrosenth@chromium.org
Gerrit-Reviewer: Ricardo Quesada
ricardoq@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-Attention: Ricardo Quesada
ricardoq@google.com
Gerrit-Attention: Jack Rosenthal
jrosenth@chromium.org
Gerrit-Comment-Date: Wed, 01 Sep 2021 04:56:11 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment