[U-Boot] [PATCH v3 3/3] SPL: Change description for spl command
Andreas Bießmann
andreas.devel at googlemail.com
Mon Nov 19 11:17:10 CET 2012
Dear Stefano Babic,
On 19.11.2012 10:11, Stefano Babic wrote:
> Add a more descriptive text to the help of the spl
> command.
>
> Signed-off-by: Stefano Babic <sbabic at denx.de>
> ---
>
> common/cmd_spl.c | 12 ++++++++----
> 1 file changed, 8 insertions(+), 4 deletions(-)
>
> diff --git a/common/cmd_spl.c b/common/cmd_spl.c
> index 9ec054a..b21d41d 100644
> --- a/common/cmd_spl.c
> +++ b/common/cmd_spl.c
> @@ -182,7 +182,11 @@ static int do_spl(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
>
> U_BOOT_CMD(
> spl, 6 , 1, do_spl, "SPL configuration",
> - "export <img=atags|fdt> [kernel_addr] [initrd_addr] "
> - "[fdt_addr if <img> = fdt] - export a kernel parameter image\n"
> - "\t initrd_img can be set to \"-\" if fdt_addr without initrd img is"
> - "used");
> + "export <img=atags|fdt> [kernel_addr] [initrd_addr] [fdt_addr ]\n"
---------------------------------------------------------------------^
unnecessary blank
> + "\timg\t\t\"atags\" or \"fdt\"\n"
> + "\tkernel_addr\taddress where a kernel image is stored.\n"
> + "\t\t\tkernel is loaded as part of the boot process, but it is not started.\n"
> + "\tinitrd_addr\tAddress of initial ramdisk\n"
------------------------^
Capitalization should be uniform. Either change all other places to
capital beginning or start here with small letters.
> + "\t\t\tcan be set to \"-\" if fdt_addr without initrd img is used\n"
--------------------------------------------------------------^
I think we should write here either 'initrd image'/'initial ramdisk
image' or 'initrd_addr'. I know img is a well known abbreviation for
image, but in a descriptive text we should IMHO not abbreviate.
Additionally the sentence should stop with a full stop like the others do.
> + "\tfdt_addr\tin case of fdt, the address of the device tree.\n"
> + );
>
Best regards
Andreas Bießmann
More information about the U-Boot
mailing list