diff options
| author | Pali Rohár | 2023-03-23 20:57:51 +0100 |
|---|---|---|
| committer | Stefan Roese | 2023-03-24 13:13:14 +0100 |
| commit | 12e79fbfffa2323679059898414e76caeffa3364 (patch) | |
| tree | ec999e5b38fcbd5821fa492caa2a2988d3906227 /tools/kwbimage.c | |
| parent | e6900565b58d7d7ceac6364baf6e001a65761598 (diff) | |
tools: kwbimage: Fix invalid UART kwbimage v1 headersz
Armada 385 BootROM ignores low 7 bits of headersz when parsing kwbimage
header of UART type, which effectively means that headersz is rounded down
to multiply of 128 bytes. For all other image types BootROM reads and use
all bits of headersz. Therefore fill into UART type of kwbimage v1 headersz
aligned to 128 bytes.
Fixes: 2b0980c24027 ("tools: kwbimage: Fill the real header size into the main header")
Signed-off-by: Pali Rohár <pali@kernel.org>
Reviewed-by: Stefan Roese <sr@denx.de>
Diffstat (limited to 'tools/kwbimage.c')
| -rw-r--r-- | tools/kwbimage.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/tools/kwbimage.c b/tools/kwbimage.c index 309657a5637..177084adf82 100644 --- a/tools/kwbimage.c +++ b/tools/kwbimage.c @@ -1231,6 +1231,16 @@ static size_t image_headersz_v1(int *hasext) if (count > 0) headersz += sizeof(struct register_set_hdr_v1) + 8 * count + 4; + /* + * For all images except UART, headersz stored in header itself should + * contains header size without padding. For UART image BootROM rounds + * down headersz to multiply of 128 bytes. Therefore align UART headersz + * to multiply of 128 bytes to ensure that remaining UART header bytes + * are not ignored by BootROM. + */ + if (image_get_bootfrom() == IBR_HDR_UART_ID) + headersz = ALIGN(headersz, 128); + return headersz; } |