Hello Patrick Rudolph, Subrata Banik, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/35349
to look at the new patch set (#3).
Change subject: drivers/intel/fsp2_0: Allocate cfg_region_size for UPD ......................................................................
drivers/intel/fsp2_0: Allocate cfg_region_size for UPD
In FSP-S, the driver constructs its pointer to UPD using the offset in the header. Similarly, use the header's cfg_region_size for allocating memory and copying the default configuration.
Add sanity checks for cases where there are more UPD definitions than cfg_region_size, and for a region length of 0.
TEST=Verify OK on Mandolin, verify a mock error condition BUG=b:140648081
Change-Id: I20fad0e27a2ad537898b6d01e5241e1508da690c Signed-off-by: Marshall Dawson marshalldawson3rd@gmail.com --- M src/drivers/intel/fsp2_0/silicon_init.c 1 file changed, 12 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/49/35349/3