-----Original Message----- From: Simon Kagstrom [mailto:simon.kagstrom@netinsight.net] Sent: Thursday, September 03, 2009 5:30 PM To: Prafulla Wadaskar Cc: u-boot@lists.denx.de; Ashish Karkare; Prabhanjan Sarnaik Subject: Re: [U-Boot] [PATCH v3 4/4] tools: mkimage: Add: Kirkwood Boot Image support (kwbimage)
On Mon, 24 Aug 2009 23:03:59 +0530 Prafulla Wadaskar prafulla@marvell.com wrote:
This patch adds type kwbimabe support for new mkimage core For more details refer docs/README.kwbimage
+Command syntax: +-------------- +./tools/mkimage -l <kwboot_file>
to list the kwb image file details+./tools/mkimage -n <board specific configuration file> \
-T kwbimage -a <start address> -e<execution address> \
-d <input_raw_binary> <output_kwboot_file>+for ex. +./tools/mkimage -n ./board/Marvell/openrd_base/kwbimage.cfg \
-T kwbimage -a 0x00600000 -e 0x00600000 \-d u-boot.bin u-boot.kwbI think it could also be useful to be able to produce just the boot header without the U-boot image. For example if you want to place U-boot at some other location on the flash or (as a developer) frequently re-flash U-boot, but don't want to write to the first block all the time (you only need to rewrite it when moving U-boot).
Hi Simon This is not possible. Boot Header is a part of Kirkwood boot image, Header contains u-boot imagesize so for this, first block need to be written each time. Secondly u-boot header is of 512bytes only whereas minimum sector size could be 4kbytes (typical), so rest part of first sector will be a waste. Also at the end of boot image checksum need to be calculated on entire image including header.
If one need just header, just use only initial 512 byte I don't see any good use case for this at this moment.
Regards.. Prafulla . .
Perhaps something like
+Typical example of kwimage.cfg file: +-----------------------------------
+# Boot Media configurations +BOOT_FROM nand +NAND_ECC_MODE default +NAND_PAGE_SIZE 0x0800
SOURCE_OFFSET 0x0 # Relative to the header or NAND start?
+static void kwbimage_set_header (void *ptr, struct stat
*sbuf, int ifd,
struct mkimage_params *params)+{
- struct kwb_header *hdr = (struct kwb_header *)ptr;
[...]
- mhdr->srcaddr = sizeof(struct kwb_header);
and this could then be changed to have this info.
// Simon