[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.
> Have about refactoring fsp_print_header_info() to get version string via vsp_get_version(). […]

The usage of fsp_get_version() is to print the version format to a buffer that can be added into coreboot table, while fsp_print_header_info() is meant to print it out on console
I think it's fine to keep fsp_print_header_info() As-is, and CPX can print its own version format in soc_validate_fsp_version().

I make fsp_get_version weak function in CB:47603 and override it in CB:47560.