[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: Kangheui Won, 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 6:

(5 comments)

File src/drivers/intel/fsp2_0/Kconfig:

https://review.coreboot.org/c/coreboot/+/62942/comment/fb6a622f_e81e23f5 
PS6, Line 362: P
> TIMESTAMPS?

Ack

https://review.coreboot.org/c/coreboot/+/62942/comment/eec655f1_61f9375b 
PS6, Line 363: bool
> Shouldn't this be shown in menuconfig?

Ack

https://review.coreboot.org/c/coreboot/+/62942/comment/9e1d95a2_6b0a05ba 
PS6, Line 365: timestamp
> timestamps?

Ack

https://review.coreboot.org/c/coreboot/+/62942/comment/697e4015_1d7a0978 
PS6, Line 368: SoC users requires to compile and generate FSP binary with `PcdFspPerformanceEnable`
             : 	  set to `TRUE`.
> Mabye `To be able to use this, FSP has to be compiled with .... […]

Ack

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

https://review.coreboot.org/c/coreboot/+/62942/comment/22cf1a79_af7182db 
PS6, Line 82: |P
> add a space, please

Ack