[U-Boot-Users] [PATCH] Add option -p to mkimage to align an image to word boundary

Jean-Paul Saman 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:
> >
> > * 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

Kind greetings,

Jean-Paul Saman

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 mailing list