From 7c7294fa27c1508742f6ea7cf98a959d468478ba Mon Sep 17 00:00:00 2001 From: Zheng Bao Date: Wed, 4 Jan 2023 16:38:28 +0800 Subject: [PATCH] amdfwtool: Report the address of EFS header and body The address mode is an internal mode which AMD FWs use. Regular developers don't have to know that. Just report the relative address every time. For the cases head and body are split, the address of body is also reported. Change-Id: I77d9aac0b3d996363341c1d2dae049ec344b39aa Signed-off-by: Zheng Bao Reviewed-on: https://review.coreboot.org/c/coreboot/+/71651 Tested-by: build bot (Jenkins) Reviewed-by: Fred Reitberger --- util/amdfwtool/amdfwtool.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/util/amdfwtool/amdfwtool.c b/util/amdfwtool/amdfwtool.c index 1703cf7138..9b250b52b7 100644 --- a/util/amdfwtool/amdfwtool.c +++ b/util/amdfwtool/amdfwtool.c @@ -2545,9 +2545,12 @@ int main(int argc, char **argv) ctx.address_mode = AMD_ADDR_REL_BIOS; else ctx.address_mode = AMD_ADDR_PHYSICAL; - printf(" AMDFWTOOL Using firmware directory location of %s address: 0x%08x\n", - ctx.address_mode == AMD_ADDR_PHYSICAL ? "absolute" : "relative", - RUN_CURRENT(ctx)); + printf(" AMDFWTOOL Using firmware directory location of address: 0x%08x", + efs_location); + if (body_location != efs_location) + printf(" with a split body at: 0x%08x\n", body_location); + else + printf("\n"); integrate_firmwares(&ctx, amd_romsig, amd_fw_table);