Martin Roth has submitted this change. ( https://review.coreboot.org/c/coreboot/+/73653 )
Change subject: amdfwtool: Increase MAX_PSP_ENTRIES ......................................................................
amdfwtool: Increase MAX_PSP_ENTRIES
The MAX_PSP_ENTRIES constant reserves space for the psp directory table entries. This table is aligned to 4K and the next binary is also aligned to 4K. The number of psp directory entries on Birman exceeds the previous limit, so increase it to the maximum that will fit in a 4K block.
TEST=timeless builds for Birman unchanged
Signed-off-by: Fred Reitberger reitbergerfred@gmail.com Change-Id: I297edc9cccffde0ad1ce7461b375542f9f2f7c23 Reviewed-on: https://review.coreboot.org/c/coreboot/+/73653 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Bao Zheng fishbaozi@gmail.com Reviewed-by: Nikolai Vyssotski nikolai.vyssotski@amd.corp-partner.google.com Reviewed-by: Martin Roth martin.roth@amd.corp-partner.google.com --- M util/amdfwtool/amdfwtool.h 1 file changed, 24 insertions(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Bao Zheng: Looks good to me, but someone else must approve Nikolai Vyssotski: Looks good to me, but someone else must approve Martin Roth: Looks good to me, approved
diff --git a/util/amdfwtool/amdfwtool.h b/util/amdfwtool/amdfwtool.h index 03cc056..455c90e 100644 --- a/util/amdfwtool/amdfwtool.h +++ b/util/amdfwtool/amdfwtool.h @@ -219,7 +219,7 @@ psp_directory_entry entries[]; } __attribute__((packed, aligned(16))) psp_directory_table;
-#define MAX_PSP_ENTRIES 0x2f +#define MAX_PSP_ENTRIES 0xff
typedef struct _psp_combo_header { uint32_t cookie;