Change in coreboot[master]: src/drivers/intel/fsp2_0: allow larger FSP 2.0 header

[Furquan Shaikh (Code Review)]

Attention is currently required from: Martin Roth, Michał Żygowski, Marshall Dawson, Subrata Banik, Nikolai Vyssotski, Andrey Petrov, Patrick Rudolph, Nathaniel L Desimone.
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56190 )

Change subject: src/drivers/intel/fsp2_0: allow larger FSP 2.0 header
......................................................................

Patch Set 4: -Code-Review

(4 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/56190/comment/b428f8b7_cae846b1 
PS4, Line 12:  This field is not used in coreboot yet.
This is not really correct. It is used by some Intel platforms.

https://review.coreboot.org/c/coreboot/+/56190/comment/e47759db_52e68659 
PS4, Line 13: Allow headers of both lengths to be accepted.
Can you please add the detail that the check for header length in `looks_like_fsp_header()` is updated to look for a minimum length based on version and the reason for that?

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

https://review.coreboot.org/c/coreboot/+/56190/comment/7bff07cc_56376d58 
PS4, Line 17:      if (CONFIG(PLATFORM_USES_FSP2_2))
> please, no spaces at the start of a line

Please fix this and the comments below.

https://review.coreboot.org/c/coreboot/+/56190/comment/7c5e1f36_a230f335 
PS4, Line 36: fsp_header_length < fsp_hdr_get_expected_min_length()
Can you please add a comment here explaining why header length is compared against a minimum value for the version?