[U-Boot] [PATCH] tools: kwboot: patch destaddr only for SoCs with header version 1

Stefan Roese sr at denx.de
Tue Aug 9 10:27:33 CEST 2016


Hi Simon,

On 08.08.2016 20:39, Simon Baatz wrote:
> Commit f4db6c976cf ("arm: mvebu: Add runtime detection of UART (xmodem)
> boot-mode") added a change to hdr->destaddr when dynamically patching an
> image for UART boot mode.  With this change, kwboot ceases to work on
> Kirkwood.
>
> Thus, let's change hdr->destaddr only when we are patching an image with
> header version 1 (Orion and Kirkwood use header version 0).
>
> Signed-off-by: Simon Baatz <gmbnomis at gmail.com>
> Fixes: f4db6c976cf ("arm: mvebu: Add runtime detection of UART (xmodem) boot-mode")
> Cc: Stefan Roese <sr at denx.de>
> Cc: Luka Perkov <luka.perkov at sartura.hr>
> Cc: Kevin Smith <kevin.smith at elecsyscorp.com>
> ---
>
> Hi,
>
> this patch makes kwboot work again on Kirkwood for me.  However, I could not
> verify whether Armada XP / 38x still works, since I don't have such
> hardware.

Thanks for the patch. I'll test it AXP / A38x next week. One comment
below.

> - Simon
>
>  tools/kwboot.c | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/tools/kwboot.c b/tools/kwboot.c
> index e00958a..4160e5b 100644
> --- a/tools/kwboot.c
> +++ b/tools/kwboot.c
> @@ -655,14 +655,6 @@ kwboot_img_patch_hdr(void *img, size_t size)
>
>  	hdr->blockid = IBR_HDR_UART_ID;
>
> -	/*
> -	 * Subtract mkimage header size from destination address
> -	 * as this header is not expected by the Marvell BootROM.
> -	 * This way, the execution address is identical to the
> -	 * one the image is compiled for (TEXT_BASE).
> -	 */
> -	hdr->destaddr = hdr->destaddr - sizeof(struct image_header);
> -
>  	if (image_ver == 0) {
>  		struct main_hdr_v0 *hdr_v0 = img;
>
> @@ -672,6 +664,14 @@ kwboot_img_patch_hdr(void *img, size_t size)
>  		hdr_v0->srcaddr = hdr_v0->ext
>  			? sizeof(struct kwb_header)
>  			: sizeof(*hdr_v0);
> +	} else {
> +	  /*
> +	   * Subtract mkimage header size from destination address
> +	   * as this header is not expected by the Marvell BootROM.
> +	   * This way, the execution address is identical to the
> +	   * one the image is compiled for (TEXT_BASE).
> +	   */
> +	  hdr->destaddr = hdr->destaddr - sizeof(struct image_header);

Indentation looks incorrect here. Please change and send v2. I'll
apply it next after I return from my short vacation.

Thanks,
Stefan


More information about the U-Boot mailing list