Attention is currently required from: Bora Guvendik, Selma Bensaid, Subrata Banik, Patrick Rudolph. Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58993 )
Change subject: soc/intel/common/cse: Add support to get CSME timestamps ......................................................................
Patch Set 3:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/58993/comment/f747e847_313ccc37 PS3, Line 10: while `during`
File src/soc/intel/common/block/cse/cse_lite.c:
https://review.coreboot.org/c/coreboot/+/58993/comment/3383347b_29c1436e PS3, Line 126: nit: extra line
File src/soc/intel/common/block/include/intelblocks/cse.h:
https://review.coreboot.org/c/coreboot/+/58993/comment/da47504d_47084486 PS3, Line 153: /* Early boot performance data */ : /* 0: CSME ROM start execution */ : /* 1: EC Boot Load Done (CSME ROM starts main execution) */ : /* 2: CSME ROM completed execution / CSME RBE started */ : /* 3: CSME got ESE Init Done indication from ESE */ : /* 4: CSME RBE start PMC patch/es loading */ : /* 5: CSME RBE completed PMC patch/es loading */ : /* 6: CSME RBE set "Boot Stall Done" indication to PMC */ : /* 7: CSME start poll for PMC PPS register */ : /* 8: PMC set PPS */ : /* 9: CSME BUP start running */ : /* 10: CSME set "Host Boot Prep Done" indication to PMC */ : /* 11: CSME starts PHYs loading */ : /* 12: CSME completed PHYs loading */ : /* 13: PMC indicated CSME that xxPWRGOOD was asserted */ : /* 14: PMC indicated CSME that SYS_PWROK was asserted */ : /* 15: PMC sent "CPU_BOOT_CONFIG" start message to CSME */ : /* 16: CSME sent "CPU_BOOT_CONFIG" done message to PMC */ : /* 17: PMC indicated CSME that xxPLTRST was de-asserted */ : /* 18: PMC indicated CSME that TCO_S0 was asserted */ : /* 19: PMC sent "Core Reset Done Ack - Sent" message to CSME */ : /* 20: ACM Active indication - ACM started its execution */ : /* 21: ACM Done indication - ACM completed execution */ : /* 22: BIOS sent DRAM Init Done message */ : /* 23: CSME sent DRAM Init Done message back to BIOS */ : /* 24: CSME completed loading TCSS */ : /* 25: CSME started loading ISH Bringup module */ : /* 26: CSME completed loading ISH Bringup module */ : /* 27: CSME started loading ISH Main module */ : /* 28: CSME completed loading Main module */ : /* 29: BIOS sent "End Of Post" message to CSME */ : /* 30: CSME sent "End Of Post" ack message back to BIOS */ : /* 31: BIOS sent "Core BIOS Done" message to CSME */ : /* 32: CSME sent "Core BIOS Done" ack message back to BIOS */ : /* 33: CSME reached Firmware Init Done */ : /* 34 - 63 Reserved */ Should we make these an enum ?
-
Tim Wawrzynczak (Code Review)