Attention is currently required from: Kapil Porwal, Karthik Ramasubramanian, Subrata Banik.
Dinesh Gehlot has posted comments on this change by Dinesh Gehlot. ( https://review.coreboot.org/c/coreboot/+/85217?usp=email )
Change subject: commonlib: Add ELOG_TYPE_FW_CSE_SYNC eventLog type ......................................................................
Patch Set 2:
(4 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/85217/comment/9b2aba69_21fa4d8c?usp... : PS1, Line 10: notify the user about the
Nit: Since event logs are purely for debugging purposes and not to notify the user, I would rephrase […]
Acknowledged
File src/commonlib/bsd/include/commonlib/bsd/elog.h:
https://review.coreboot.org/c/coreboot/+/85217/comment/fb47db1a_62e86450?usp... : PS1, Line 394: ELOG_FW_ROMSTAGE_CSE_SYNC
ELOG_FW_EARLY_CSE_SYNC
Acknowledged
https://review.coreboot.org/c/coreboot/+/85217/comment/8d1ae533_1b1ededb?usp... : PS1, Line 395: ELOG_FW_RAMSTAGE_CSE_SYNC
ELOG_FW_LATE_CSE_SYNC
Acknowledged
https://review.coreboot.org/c/coreboot/+/85217/comment/23289538_70794366?usp... : PS1, Line 396: ELOG_FW_PAYLOAD_CSE_SYNC
ELOG_FW_CSE_SYNC_AT_PAYLOAD
Acknowledged