Attention is currently required from: Bao Zheng, Felix Held, Karthik Ramasubramanian, Martin Roth, Matt DeVillier, Raul Rangel, Tim Van Patten.
Hello Bao Zheng, Felix Held, Martin Roth, Matt DeVillier, Raul Rangel, Tim Van Patten, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/76585?usp=email
to look at the new patch set (#7).
The following approvals got outdated and were removed: Code-Review+2 by Martin Roth, Verified+1 by build bot (Jenkins)
Change subject: util/amdfwtool: Support multiple firmware identifier types ......................................................................
util/amdfwtool: Support multiple firmware identifier types
Currently this tool generates a hash table to verify signed binaries, with a 2 byte FWID as the only kind of identifier. Going forward some binaries are going to adopt 16 byte UUID identifiers and more binaries will follow in the future SoCs. Hence add support for handling multiple firmware identifier types. While at this remove the unused fwid from the PSP FW table.
BUG=b:277292697 TEST=Build BIOS image and boot to OS in Myst & Skyrim.
Change-Id: I5180dc0fe812b174b1d40fea9f00a85d6ef00f2f Signed-off-by: Karthikeyan Ramasubramanian kramasub@google.com --- M util/amdfwtool/amdfwtool.h M util/amdfwtool/data_parse.c M util/amdfwtool/signed_psp.c 3 files changed, 32 insertions(+), 9 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/85/76585/7