Attention is currently required from: Jason Glenesk, Raul Rangel, Marshall Dawson, Paul Menzel, Name of user not set #1003801, Andrey Petrov, Patrick Rudolph. Hello build bot (Jenkins), Jason Glenesk, Raul Rangel, Marshall Dawson, Andrey Petrov, Julian Schroeder, Patrick Rudolph, Felix Held,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/61281
to look at the new patch set (#12).
Change subject: soc/amd/common/fsp: check fsp image revision ......................................................................
soc/amd/common/fsp: check fsp image revision
Check if FSP binary and coreboot FSP structures (fspmupd.h) match sufficiently.
A change in minor number denotes less critical changes or additions to the FSP API that still allow for the boot process to proceed. A change of the AMD image revision major number will halt boot. The Fspmupd.h header now defines IMAGE_REVISION_ macros for AMD Picasso, Cezanne and Sabrina APUs.
BUG=b:184650244 TEST=build, boot and check fsp image revision info. Example:
FSP major = 1 FSP minor = 0 FSP revision = 5 FSP build = 0
Signed-off-by: Julian Schroeder julianmarcusschroeder@gmail.com Change-Id: I0fbf9413b0cf3e6093ee9c61ff692ff78ebefebc --- M src/soc/amd/common/fsp/fsp_validate.c M src/vendorcode/amd/fsp/cezanne/FspmUpd.h M src/vendorcode/amd/fsp/picasso/FspmUpd.h M src/vendorcode/amd/fsp/sabrina/FspmUpd.h 4 files changed, 55 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/81/61281/12