[coreboot-gerrit] [L] Change in coreboot[master]: Recently published Intel CedarIslandFSP binary contains PE images in ...

[Ed Sharma (Code Review)]

Attention is currently required from: Arthur Heymans, Furquan Shaikh, Subrata Banik, Johnny Lin, Ed Sharma, Tim Wawrzynczak, Julian Schroeder, Nathaniel L Desimone, Felix Held, Isaac W Oram.

Hello Arthur Heymans, build bot (Jenkins), Furquan Shaikh, Subrata Banik, Jonathan Zhang, Johnny Lin, Tim Wawrzynczak, Julian Schroeder, Nathaniel L Desimone, Felix Held, Isaac W Oram,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/66819

to look at the new patch set (#2).

Change subject: Recently published Intel CedarIslandFSP binary contains PE images in FSP-M and FSP-S. This causes coreboot boot hang on DeltaLake servers.
......................................................................

Recently published Intel CedarIslandFSP binary contains PE images in
FSP-M and FSP-S. This causes coreboot boot hang on DeltaLake servers.

The change required to add support for pe_relocate in fsp-relocate.c
I had to move a few functions to top of file as they need to be used
by pe_relocate but were placed below the te_relocate function. I chose
to place pe_relocate function next to te_relocate.
The code supports PE32 format, not PE64, at this time.

TEST=
This code is tested with FSP version 33A for DeltaLake boot.

Change-Id: I01e2c123d74f735a647b994aa66419c9796f193e
Signed-off-by: Eddie Sharma 
---
M src/commonlib/fsp_relocate.c
1 file changed, 214 insertions(+), 37 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/19/66819/2