[coreboot-gerrit] Change in coreboot[master]: intel/fsp2_0: Add soc_validate_fsp_version for FSP version check

[Johnny Lin (Code Review)]

Johnny Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47559 )

Change subject: intel/fsp2_0: Add soc_validate_fsp_version for FSP version check
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/c/coreboot/+/47559/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/47559/1//COMMIT_MSG@10 
PS1, Line 10: the console is usually ready.
> In addition, we need to have weak function of getting FSP version. […]

There's already a fsp_get_version in util.c, that can be the weak function that implements the general version display format. CPX can override it and have it's own format, but should other functions like fsp_print_header_info do the same? The display format only CPX is different.