summaryrefslogtreecommitdiff
path: root/util/amdfwtool/signed_psp.c
diff options
context:
space:
mode:
authorKarthikeyan Ramasubramanian <kramasub@google.com>2023-07-13 11:11:04 -0600
committerMartin L Roth <gaumless@gmail.com>2023-08-04 20:49:55 +0000
commit24b522709132d542386eb71e684ed13f42bf1fdc (patch)
tree21f90a0e2af4f4efc31eb3d60ea7d3c5098c9c51 /util/amdfwtool/signed_psp.c
parent71d60ce506941cec2965e609627759fce43f5fe9 (diff)
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> Reviewed-on: https://review.coreboot.org/c/coreboot/+/76585 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Martin L Roth <gaumless@gmail.com>
Diffstat (limited to 'util/amdfwtool/signed_psp.c')
-rw-r--r--util/amdfwtool/signed_psp.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/util/amdfwtool/signed_psp.c b/util/amdfwtool/signed_psp.c
index 2e19f72002..738c588d10 100644
--- a/util/amdfwtool/signed_psp.c
+++ b/util/amdfwtool/signed_psp.c
@@ -384,7 +384,6 @@ void process_signed_psp_firmwares(const char *signed_rom,
exit(-1);
/* File is successfully processed and is part of signed PSP binaries set. */
- fw_table[i].fw_id = get_psp_fw_type(soc_id, &header);
fw_table[i].addr_signed = signed_start_addr;
fw_table[i].file_size = (uint32_t)fd_stat.st_size;
hash_files[fw_table[i].hash_tbl_id].present = true;