[coreboot-gerrit] Change in coreboot[master]: drivers/intel/fsp2_0: Add provision to extract FSP Performance Data

[Subrata Banik (Code Review)]

Attention is currently required from: Maulik V Vaghela, Tim Wawrzynczak, Paul Menzel, Nick Vaccaro, Michael Niewöhner, Andrey Petrov.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62942 )

Change subject: drivers/intel/fsp2_0: Add provision to extract FSP Performance Data
......................................................................

Patch Set 7:

(1 comment)

File src/drivers/intel/fsp2_0/silicon_init.c:

https://review.coreboot.org/c/coreboot/+/62942/comment/ed5fb974_70b99f19 
PS7, Line 248: 	if (CONFIG(DISPLAY_FSP_TIMESTAMPS))
             : 		fsp_display_timestamp();
             :
> Ok, then let's keep it after whole silicon init. Is multi-phase init surely finished after do_silicon_init, or does it continue asynchronously?

yes, when you see "FspSiliconInitApi() - [Status: 0x00000000] - End" prints, both FSP_S and MultiPhaseSiInit are done.