Attention is currently required from: Sridhar Siricilla.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59668 )
Change subject: commonlib: Add new TS for CSE firmware Sync
......................................................................
Patch Set 2:
(2 comments)
File src/commonlib/include/commonlib/timestamp_serialized.h:
https://review.coreboot.org/c/coreboot/+/59668/comment/33f47796_e86794b8
PS2, Line 210: cse_fw_sync
can you please explicitly call it "CSE firmware sync" ?
File src/commonlib/include/commonlib/timestamp_serialized.h:
https://review.coreboot.org/c/coreboot/+/59668/comment/e618bc33_7d764df5
PS1, Line 209: { TS_START_CSE_FW_SYNC, "Before cse_fw_sync"},
: { TS_END_CSE_FW_SYNC, "After cse_fw_sync"},
Before cse_fw_sync => starting CSE firmware sync […]
Ack
--
To view, visit
https://review.coreboot.org/c/coreboot/+/59668
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I9ed82c5358eb94b5e7c91b9fd783c5e09189b77a
Gerrit-Change-Number: 59668
Gerrit-PatchSet: 2
Gerrit-Owner: Sridhar Siricilla
sridhar.siricilla@intel.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-CC: Sridhar Siricilla
sridhar.siricilla@intel.corp-partner.google.com
Gerrit-CC: Subrata Banik
subrata.banik@intel.com
Gerrit-Attention: Sridhar Siricilla
sridhar.siricilla@intel.com
Gerrit-Comment-Date: Fri, 26 Nov 2021 05:36:18 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Subrata Banik
subrata.banik@intel.com
Gerrit-MessageType: comment