GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

amdfwtool: Support firmware offsets of larger than 16MiB 

The mapped windows is up to 16M. Even if the flash size is 32MB, it is
not mapped at 0xFE000000.

So using "0xFFFFFFFF - rom_size + 1" to get the "rom_base_address" can
only explain well when rom_size is less or equal to 16MB. For larger
size, it is not physically correct (Even though it can get expected
result).

If the flash size is larger than 16M, we assume the given addresses
are already relative ones. So we don't need the physical base address
any more.

This commit is part of a series of patches to support 32/64M flash.
BUG=b:255374782

Change-Id: I9eea45f0be45a959c4150030e7e213923510ad68
Signed-off-by: Zheng Bao <fishbaozi@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/72959
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin L Roth <gaumless@gmail.com>
This commit is contained in:
Zheng Bao 2023-02-11 15:17:22 +08:00 committed by Martin L Roth
parent 42f8b59c11
commit 1a0c99f55d
1 changed files with 12 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
util/amdfwtool/amdfwtool.c
View File

@ -88,6 +88,8 @@
#define AMD_ROMSIG_OFFSET 0x20000 #define AMD_ROMSIG_OFFSET 0x20000
#define MIN_ROM_KB 256 #define MIN_ROM_KB 256
#define MAX_MAPPED_WINDOW (16 * MiB)
#define MAX_MAPPED_WINDOW_MASK (MAX_MAPPED_WINDOW - 1)
#define _MAX(A, B) (((A) > (B)) ? (A) : (B)) #define _MAX(A, B) (((A) > (B)) ? (A) : (B))
@ -1932,7 +1934,6 @@ int main(int argc, char **argv)
uint32_t efs_location = 0; uint32_t efs_location = 0;
bool any_location = 0; bool any_location = 0;
uint32_t romsig_offset; uint32_t romsig_offset;
uint32_t rom_base_address;
uint8_t efs_spi_readmode = 0xff; uint8_t efs_spi_readmode = 0xff;
uint8_t efs_spi_speed = 0xff; uint8_t efs_spi_speed = 0xff;
uint8_t efs_spi_micron_flag = 0xff; uint8_t efs_spi_micron_flag = 0xff;
@ -2234,13 +2235,18 @@ int main(int argc, char **argv)
printf(" AMDFWTOOL Using ROM size of %dKB\n", ctx.rom_size / 1024); printf(" AMDFWTOOL Using ROM size of %dKB\n", ctx.rom_size / 1024);
if (ctx.rom_size <= MAX_MAPPED_WINDOW) {
uint32_t rom_base_address;
rom_base_address = 0xFFFFFFFF - ctx.rom_size + 1; rom_base_address = 0xFFFFFFFF - ctx.rom_size + 1;
if (efs_location & ~MAX_MAPPED_WINDOW_MASK)
if (efs_location & 0xFF000000)
efs_location = efs_location - rom_base_address; efs_location = efs_location - rom_base_address;
if (body_location & 0xFF000000) if (body_location & ~MAX_MAPPED_WINDOW_MASK)
body_location = body_location - rom_base_address; body_location = body_location - rom_base_address;
}
/* If the flash size is larger than 16M, we assume the given
addresses are already relative ones. Otherwise we print error.*/
if (efs_location && efs_location > ctx.rom_size) { if (efs_location && efs_location > ctx.rom_size) {
fprintf(stderr, "Error: EFS/Directory location outside of ROM.\n\n"); fprintf(stderr, "Error: EFS/Directory location outside of ROM.\n\n");
return 1; return 1;