diff options
| author | Zheng Bao <fishbaozi@gmail.com> | 2023-02-11 15:17:22 +0800 |
|---|---|---|
| committer | Martin L Roth <gaumless@gmail.com> | 2023-09-26 16:20:45 +0000 |
| commit | 1a0c99f55d57f0aeec69d1cd2ffa134a7a9ff255 (patch) | |
| tree | 015f35f802aee97788c499f2ed6894cad6627dab /util/amdfwtool | |
| parent | 42f8b59c119309c49b8153dcedec19674ef7f435 (diff) | |
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>
Diffstat (limited to 'util/amdfwtool')
| -rw-r--r-- | util/amdfwtool/amdfwtool.c | 18 |
1 files changed, 12 insertions, 6 deletions
diff --git a/util/amdfwtool/amdfwtool.c b/util/amdfwtool/amdfwtool.c index bd17415ae5..6822b2ad3e 100644 --- a/util/amdfwtool/amdfwtool.c +++ b/util/amdfwtool/amdfwtool.c @@ -88,6 +88,8 @@ #define AMD_ROMSIG_OFFSET 0x20000 #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)) @@ -1932,7 +1934,6 @@ int main(int argc, char **argv) uint32_t efs_location = 0; bool any_location = 0; uint32_t romsig_offset; - uint32_t rom_base_address; uint8_t efs_spi_readmode = 0xff; uint8_t efs_spi_speed = 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); - rom_base_address = 0xFFFFFFFF - ctx.rom_size + 1; + if (ctx.rom_size <= MAX_MAPPED_WINDOW) { + uint32_t rom_base_address; - if (efs_location & 0xFF000000) - efs_location = efs_location - rom_base_address; - if (body_location & 0xFF000000) - body_location = body_location - rom_base_address; + rom_base_address = 0xFFFFFFFF - ctx.rom_size + 1; + if (efs_location & ~MAX_MAPPED_WINDOW_MASK) + efs_location = efs_location - rom_base_address; + if (body_location & ~MAX_MAPPED_WINDOW_MASK) + 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) { fprintf(stderr, "Error: EFS/Directory location outside of ROM.\n\n"); return 1; |