[U-Boot] [PATCH v2] tools: imximage: Load a size that is multiple of 512

Dirk Behme dirk.behme at de.bosch.com
Fri Jan 4 07:20:16 CET 2013


On 03.01.2013 19:24, Fabio Estevam wrote:
> In order to mx53 ROM to properly load the U-boot image, its header size should 
> be multiple of 512 bytes.
> 
...
> Signed-off-by: Troy Kisky <troy.kisky at boundarydevices.com>
> Signed-off-by: Fabio Estevam <fabio.estevam at freescale.com>
> ---
> Changes since v1:
> - Improvec commit log
>  include/image.h  |    3 +++
>  tools/imximage.c |    9 ++++++++-
>  2 files changed, 11 insertions(+), 1 deletion(-)
> 
> diff --git a/include/image.h b/include/image.h
> index f54d983..e1e83b4 100644
> --- a/include/image.h
> +++ b/include/image.h
> @@ -179,6 +179,9 @@
>  #define IH_MAGIC	0x27051956	/* Image Magic Number		*/
>  #define IH_NMLEN		32	/* Image Name Length		*/
>  
> +/* Reused from common.h */
> +#define ROUND(a, b)		(((a) + (b) - 1) & ~((b) - 1))
> +
>  /*
>   * Legacy format image header,
>   * all data in network byte order (aka natural aka bigendian).
> diff --git a/tools/imximage.c b/tools/imximage.c
> index 63f88b6..a93d7eb 100644
> --- a/tools/imximage.c
> +++ b/tools/imximage.c
> @@ -515,7 +515,14 @@ static void imximage_set_header(void *ptr, struct stat *sbuf, int ifd,
>  
>  	/* Set the imx header */
>  	(*set_imx_hdr)(imxhdr, dcd_len, params->ep, imxhdr->flash_offset);
> -	*header_size_ptr = sbuf->st_size + imxhdr->flash_offset;
> +
> +	/*
> +	 * ROM bug alert
> +	 * mx53 only loads 512 byte multiples.

Is this i.MX53 specific or is this valid for i.MX6, too?

Best regards

Dirk


More information about the U-Boot mailing list