Change in coreboot[master]: soc/amd/common/fsp/fsp_validate.c: print warning instead of error

[Name of user not set (Code Review)]

Attention is currently required from: Marshall Dawson, Paul Menzel, Fred Reitberger, Rob Barnes, Julian Schroeder.
Name of user not set #1003801 has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62349 )

Change subject: soc/amd/common/fsp/fsp_validate.c: print warning instead of error
......................................................................

Patch Set 2:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/62349/comment/627118c9_a300a05e 
PS2, Line 10: warning instead of an error.
> Why? Is that for CI testing, or is it expected to have such FSP binaries?

The FSP spec, IIRC, doesn't require valid image revision info. The older AMD FSPs attempt to provide one but fail and instead put in '1'. I fixed that since but older FSP binaries will still be around for a while.

Patchset:

PS1:
> > No Signed-off-by line in commit message

done