Subrata Banik has submitted this change. ( https://review.coreboot.org/c/coreboot/+/85219?usp=email )
Change subject: src/soc/intel/cmn/blk/cse: Log cse sync information ......................................................................
src/soc/intel/cmn/blk/cse: Log cse sync information
This patch adds an event log entry for successful CSE synchronization, along with the boot stage where the synchronization occurred, either early or late.
BUG=b:379585294 TEST=elog verified on rex0 and rex64
Rex0: ``` rex-rev3 ~ # /media/usb/elogtool list 3 | 2024-01-01 22:25:59-0800 | Firmware CSE sync | Late CSE Sync ``` Rex64: ``` rex64-rev3 ~ # /media/usb/elogtool list 8 | 2024-01-01 22:51:00-0800 | Firmware CSE sync | CSE Sync at Payload ```
Change-Id: Idece841c2b069d7688afc258470667ed2851a282 Signed-off-by: Dinesh Gehlot digehlot@google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/85219 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Kapil Porwal kapilporwal@google.com Reviewed-by: Karthik Ramasubramanian kramasub@google.com Reviewed-by: Subrata Banik subratabanik@google.com --- M src/soc/intel/common/block/cse/cse_lite.c 1 file changed, 2 insertions(+), 0 deletions(-)
Approvals: Karthik Ramasubramanian: Looks good to me, approved Subrata Banik: Looks good to me, approved build bot (Jenkins): Verified Kapil Porwal: Looks good to me, approved
diff --git a/src/soc/intel/common/block/cse/cse_lite.c b/src/soc/intel/common/block/cse/cse_lite.c index 07233a9..00b5de6 100644 --- a/src/soc/intel/common/block/cse/cse_lite.c +++ b/src/soc/intel/common/block/cse/cse_lite.c @@ -806,6 +806,8 @@ return CB_ERR;
printk(BIOS_INFO, "cse_lite: CSE RW Update Successful\n"); + elog_add_event_byte(ELOG_TYPE_FW_CSE_SYNC, ENV_RAMSTAGE ? ELOG_FW_LATE_CSE_SYNC : + ELOG_FW_EARLY_CSE_SYNC); return CB_SUCCESS; }