[U-Boot-Users] [PATCH] Add option -p to mkimage to align an image to word boundary
jean-paul.saman at nxp.com
Fri Mar 2 17:52:20 CET 2007
wd at denx.de wrote on 02-03-2007 16:10:06:
> In message <OFD9437230.22326F6B-ONC1257292.00465732-C1257292.
> 0047638F at philips.com> you wrote:
> > CHANGELOG:
> > * Add option -p to mkimage for always alligning an image to word
> Images are already aligned. They have a 64 byte header prepended.
I was not talking about the header, but about the resulting image. Maybe
aligning is the wrong english word for this.
> > This patch adds a feature to tools/mkimage.c by making it possible to
> > always align a filesystem image to word boundary. Images created with
> > option are always aligned on word boundary and will always correctly
> > loaded with the ARM RVDS when using raw mode (Raw data one word per
> I don't get what you're trying to tell.
Maybe my english is not correct (I am not a native english speaker). Let
me try again:
Sometimes an ramdisk image created with mkimage has a size not a multiple
of 32 bit words. Loading such an image with the ARM RVDS in "raw word
mode" directly into the main memory, later on results in a CRC checksum
failure detected by u-boot (eg: using bootm ). The cause is that the ARM
RVDS doesn't upload the last bytes if they are not a multiple of 32 bit
words. The error is in ARM RVDS and not in mkimage, hence I named it a new
> Looking at your patch, it seems you're confusing padding to a
> multiuple of 32 bit words with alignment - note these are completely
> different things.
I mean padding to a multiple of 32 bit words.
> > of memory). Images created without this option will not boot with
> > and abort with "Data CRC error". This situation has been reported
> > times on this maillinglist.
> Has it? I have never seen it. Do you have a reference?
Here is a report from a guy, probably using ARM RVDS (however not clear
from his description).
I remember there were a few more at the end of last year but fail to find
them. Here is another reference from another mailinglist:
> > It is tested against todays git repository.
> You patch is not acceptable; the description is wrong both in the log
> and in the help message, it has coding style violations (indentation
> not by TAB etc.) and spelling errors.
The indentation and spelling can be fixed.
> Also, I'm not sure if this is really a mkimage problem.
I never said it was a mkimage problem.
> Best regards,
> Wolfgang Denk
NXP Semiconductors CTO/RTG DesignIP
Philips HighTech Campus, building WDA 3.29
Professor van den Holstlaan 4
5655 AA Eindhoven
tel: +31 (0)40 27 45131
More information about the U-Boot