Re: [PATCH u-boot-marvell v2 00/20] tools: kwbimage: Load address fixes

Stefan Roese
2022-01-14T15:40:32Z

On 1/12/22 18:20, Pali Rohár wrote:
> This patch series fixes generating images in kwbimage format, main fix
> is setting correct load address of U-Boot SPL. Also it adds support for
> generating kwbimage config file from existing kwbimage file via
> dumpimage tool.
>
> Changes in v2:
> * Fix base address for Sheeva CPUs (A370, AXP), it is 0x40004000
> * Fix information about mapped area of load address (it is L2, not CESA)
> * Add new kwbimage config option CPU
>
> Pali Rohár (20):
> tools: kwbimage: Mark all local functions as static
> tools: kwbimage: Deduplicate v1 regtype header finishing
> tools: kwbimage: Fix generating image with multiple DATA_DELAY
> commands
> tools: kwbimage: Preserve order of BINARY, DATA and DATA_DELAY
> commands
> arm: mvebu: Generate kwbimage.cfg with $(call cmd, ...)
> tools: kwbimage: Add support for specifying CPU core
> tools: kwbimage: Add support for specifying LOAD_ADDRESS for BINARY
> command
> tools: kwbimage: Check the return value of image_headersz_v1()
> tools: kwbimage: Check for maximal kwbimage header size
> arm: mvebu: Set CPU for U-Boot SPL binary in kwbimage
> arm: mvebu: Correctly set LOAD_ADDRESS for U-Boot SPL binary in
> kwbimage
> tools: kwbimage: Enforce 128-bit boundary alignment only for Sheeva
> CPU
> arm: mvebu: Enable BootROM output on A38x
> tools: kwbimage: Add missing check for maximal value for DATA_DELAY
> tools: kwbimage: Show binary image offset in mkimage -l, in addition
> to size
> tools: kwbimage: Dump kwbimage config file on '-p -1' option
> tools: kwbimage: Do not cast const pointers to non-const pointers
> tools: kwbimage/kwboot: Check ext field for non-zero value
> tools: kwbimage: Extract main data image without -p arg for dumpimage
> tools: kwbimage: Fix mkimage/dumpimage -l argument
>
> arch/arm/mach-mvebu/Makefile | 25 +-
> arch/arm/mach-mvebu/kwbimage.cfg.in | 10 +-
> tools/kwbimage.c | 549 +++++++++++++++++++++++-----
> tools/kwbimage.h | 10 +-
> tools/kwboot.c | 4 +-
> 5 files changed, 486 insertions(+), 112 deletions(-)
>

Applied to u-boot-marvell/master

Thanks,
Stefan