On Wed, Jan 2, 2013 at 1:42 PM, Troy Kisky <troy.ki...@boundarydevices.com> wrote: > On 1/2/2013 11:33 AM, Fabio Estevam wrote: >> >> Thanks for bisecting, Robert. Troy/Stefano, Any suggestions about this? It >> would be really nice if we could keep mx53loco functional when using gcc >> 4.6.2. Regards, Fabio Estevam > > Could you see if this patch makes any difference? It still needs cleaned up > some before mainline > [PATCH V4 01/11] imximage: mx53 needs transfer length a multiple of 512 > > diff --git a/tools/imximage.c b/tools/imximage.c > index 63f88b6..7e54e97 100644 > --- a/tools/imximage.c > +++ b/tools/imximage.c > @@ -494,6 +494,8 @@ static void imximage_print_header(const void *ptr) > } > } > +#define ALIGN(a, b) (((a) + (b) - 1) & ~((b) - 1)) > + > static void imximage_set_header(void *ptr, struct stat *sbuf, int ifd, > struct mkimage_params *params) > { > @@ -515,7 +517,13 @@ 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. > + * The remaining fraction of a block bytes would > + * not be loaded. > + */ > + *header_size_ptr = ALIGN(sbuf->st_size + imxhdr->flash_offset, 512); > } > int imximage_check_params(struct mkimage_params *params) > -- 1.7.9.5
Sweet! Thanks Troy, That fixes it on my Dialog based mx53loco with linaro's 4.7.3 gcc toolchain... Regards, -- Robert Nelson http://www.rcn-ee.com/ _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot