From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dirk Behme Date: Fri, 4 Jan 2013 07:20:16 +0100 Subject: [U-Boot] [PATCH v2] tools: imximage: Load a size that is multiple of 512 In-Reply-To: <1357237473-1401-1-git-send-email-fabio.estevam@freescale.com> References: <1357237473-1401-1-git-send-email-fabio.estevam@freescale.com> Message-ID: <50E674A0.1070600@de.bosch.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > Signed-off-by: Fabio Estevam > --- > 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