Attention is currently required from: Jason Glenesk, Raul Rangel, Subrata Banik, Marshall Dawson, Name of user not set #1003801, Andrey Petrov, Julian Schroeder, Patrick Rudolph. Hello build bot (Jenkins), Jason Glenesk, Raul Rangel, Subrata Banik, Marshall Dawson, Tim Wawrzynczak, Andrey Petrov, Patrick Rudolph, Julian Schroeder, Felix Held,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/58869
to look at the new patch set (#9).
Change subject: drivers/intel/fsp2_0/include/fsp: fix fsp_header ......................................................................
drivers/intel/fsp2_0/include/fsp: fix fsp_header
This patch aligns fsp_header with the Intel specification 2.0 and 2.3. The main impetus for this change is to make the fsp_info_header fully accessible in soc/vendor code. Here items such as image_revision can be checked.
TEST=verify image revision output in the coreboot serial log. compare to FSP version shown in serial debug output. verify Google Guybrush machine boots into OS.
Signed-off-by: Julian Schroeder julianmarcusschroeder@gmail.com Change-Id: Ibf50f16b5e9793d946a95970fcdabc4c07289646 --- M src/drivers/intel/fsp2_0/header_display.c M src/drivers/intel/fsp2_0/include/fsp/info_header.h M src/drivers/intel/fsp2_0/memory_init.c M src/drivers/intel/fsp2_0/silicon_init.c M src/drivers/intel/fsp2_0/temp_ram_exit.c M src/drivers/intel/fsp2_0/util.c 6 files changed, 52 insertions(+), 62 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/58869/9