Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/56641 )
Change subject: commonlib: Add timestamp strings for TS_FSP_MULTI_PHASE_SI_INIT_x ......................................................................
commonlib: Add timestamp strings for TS_FSP_MULTI_PHASE_SI_INIT_x
TEST=Refer to `cbmem -t` output below: Without this code changes, timestamp ids 962 and 963 are listed as `unknown`.
962:<unknown> 1,704,863 (157) 963:<unknown> 1,704,878 (14)
With this code changes, lists the timestamps along with the timestamp strings.
962:calling FspMultiPhaseSiInit 1,704,863 (157) 963:returning from FspMultiPhaseSiInit 1,704,878 (14)
Change-Id: I528567e4cc630e15dffffd1c7684798fcdde4535 Signed-off-by: Subrata Banik subrata.banik@intel.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/56641 Reviewed-by: Furquan Shaikh furquan@google.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/commonlib/include/commonlib/timestamp_serialized.h 1 file changed, 2 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Furquan Shaikh: Looks good to me, approved
diff --git a/src/commonlib/include/commonlib/timestamp_serialized.h b/src/commonlib/include/commonlib/timestamp_serialized.h index 75bccca..6330b75 100644 --- a/src/commonlib/include/commonlib/timestamp_serialized.h +++ b/src/commonlib/include/commonlib/timestamp_serialized.h @@ -275,6 +275,8 @@ { TS_FSP_TEMP_RAM_EXIT_END, "returning from FspTempRamExit" }, { TS_FSP_SILICON_INIT_START, "calling FspSiliconInit" }, { TS_FSP_SILICON_INIT_END, "returning from FspSiliconInit" }, + { TS_FSP_MULTI_PHASE_SI_INIT_START, "calling FspMultiPhaseSiInit" }, + { TS_FSP_MULTI_PHASE_SI_INIT_END, "returning from FspMultiPhaseSiInit" }, { TS_FSP_BEFORE_ENUMERATE, "calling FspNotify(AfterPciEnumeration)" }, { TS_FSP_AFTER_ENUMERATE, "returning from FspNotify(AfterPciEnumeration)" },