Dear Rob Herring, > From: Marek Vasut <marek.va...@gmail.com> > > This patch allows loading RAW ramdisk via bootz command. The raw ramdisk is > loaded only in case it's size is specified: > > bootz <kernel addr> <ramdisk addr>:<ramdisk size> <fdt addr> > > For example: > > bootz 0x42000000 0x43000000:0x12345 0x44000000 > > Signed-off-by: Marek Vasut <ma...@denx.de> > Signed-off-by: Rob Herring <rob.herr...@calxeda.com> > Cc: Tom Warren <twar...@nvidia.com> > Cc: albert.u.b...@aribaud.net > Cc: aflem...@gmail.com > Cc: Simon Glass <s...@chromium.org> > Cc: Stephen Warren <swar...@nvidia.com> > Cc: Nicolas Pitre <n...@fluxnic.net> > Cc: Wolfgang Denk <w...@denx.de> > Cc: Detlev Zundel <d...@denx.de>
Doesn't this still colide with CONFIG_FIT? Aka. in case of CONFIG_FIT enabled, you can't use raw ramdisk? btw. maybe we should use "@" instead of ":" and be done with it? > --- > V3: > - fix operation when CONFIG_FIT is enabled as FIT images use > <addr>[:<subimage>] > > README | 5 +++++ > common/cmd_bootm.c | 6 ++++-- > common/image.c | 15 ++++++++++++--- > 3 files changed, 21 insertions(+), 5 deletions(-) > > diff --git a/README b/README > index 5141751..068ec71 100644 > --- a/README > +++ b/README > @@ -4330,6 +4330,11 @@ On some platforms, it's possible to boot Linux > zImage. This is done using the "bootz" command. The syntax of "bootz" > command is the same as the syntax of "bootm" command. > > +Note, defining the CONFIG_SUPPORT_INITRD_RAW allows user to supply > +kernel with raw initrd images. The syntax is slightly different, the > +address of the initrd must be augmented by it's size, in the following > +format: "<initrd addres>:<initrd size>". > + > > Standalone HOWTO: > ================= > diff --git a/common/cmd_bootm.c b/common/cmd_bootm.c > index b49d4f7..2f9b214 100644 > --- a/common/cmd_bootm.c > +++ b/common/cmd_bootm.c > @@ -1634,9 +1634,11 @@ static int do_bootz(cmd_tbl_t *cmdtp, int flag, int > argc, char * const argv[]) U_BOOT_CMD( > bootz, CONFIG_SYS_MAXARGS, 1, do_bootz, > "boot Linux zImage image from memory", > - "[addr [initrd] [fdt]]\n - boot Linux zImage stored in memory\n" > + "[addr [initrd[:size]] [fdt]]\n" > + " - boot Linux zImage stored in memory\n" > "\tThe argument 'initrd' is optional and specifies the address\n" > - "\tof the initrd in memory.\n" > + "\tof the initrd in memory. The optional argument ':size' allows\n" > + "\tspecifying the size of RAW initrd.\n" > #if defined(CONFIG_OF_LIBFDT) > "\tWhen booting a Linux kernel which requires a flat device-tree\n" > "\ta third argument is required which is the address of the\n" > diff --git a/common/image.c b/common/image.c > index 77ca6e4..2a25f5f 100644 > --- a/common/image.c > +++ b/common/image.c > @@ -796,6 +796,7 @@ int boot_get_ramdisk(int argc, char * const argv[], > bootm_headers_t *images, ulong rd_addr, rd_load; > ulong rd_data, rd_len; > const image_header_t *rd_hdr; > + char *end; > #if defined(CONFIG_FIT) > void *fit_hdr; > const char *fit_uname_config = NULL; > @@ -989,9 +990,17 @@ int boot_get_ramdisk(int argc, char * const argv[], > bootm_headers_t *images, break; > #endif > default: > - puts("Wrong Ramdisk Image Format\n"); > - rd_data = rd_len = rd_load = 0; > - return 1; > +#ifdef CONFIG_SUPPORT_RAW_INITRD > + if (argc >= 3 && (end = strchr(argv[2], ':'))) { > + rd_len = simple_strtoul(++end, NULL, 16); > + rd_data = rd_addr; > + } else > +#endif > + { > + puts("Wrong Ramdisk Image Format\n"); > + rd_data = rd_len = rd_load = 0; > + return 1; > + } > } > } else if (images->legacy_hdr_valid && > image_check_type(&images->legacy_hdr_os_copy, Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot