From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?ISO-8859-1?Q?Andreas_Bie=DFmann?= Date: Mon, 19 Nov 2012 11:17:10 +0100 Subject: [U-Boot] [PATCH v3 3/3] SPL: Change description for spl command In-Reply-To: <1353316265-24430-3-git-send-email-sbabic@denx.de> References: <1352717979-6284-1-git-send-email-sbabic@denx.de> <1353316265-24430-1-git-send-email-sbabic@denx.de> <1353316265-24430-3-git-send-email-sbabic@denx.de> Message-ID: <50AA0726.3080809@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > --- > > 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 [kernel_addr] [initrd_addr] " > - "[fdt_addr if = fdt] - export a kernel parameter image\n" > - "\t initrd_img can be set to \"-\" if fdt_addr without initrd img is" > - "used"); > + "export [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