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

[Tim Wawrzynczak (Code Review)]

Attention is currently required from: Subrata Banik, Maulik V Vaghela, Paul Menzel, Nick Vaccaro, Eric Lai, Andrey Petrov.
Tim Wawrzynczak 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 4:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/62942/comment/e6f15c41_12547bc2 
PS4, Line 111: 
             : BOOT_STATE_INIT_ENTRY(BS_PAYLOAD_LOAD, BS_ON_EXIT, fsp_get_timestamp, NULL);
             : BOOT_STATE_INIT_ENTRY(BS_OS_RESUME, BS_ON_ENTRY, fsp_get_timestamp, NULL);
Instead of boot state entries, can we just add call fsp_print_timestamps() from somewhere in the flow of the fsp2_0 driver, e.g. end of do_silicon_init() ? I realize that you will not capture any timestamp emitted by Notify APIs that way, but we are also moving towards eliminating the need for those too...