mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* Make some help comments less confusing
@ 2010-03-17 13:04 Juergen Beisert
  2010-03-18  7:11 ` Sascha Hauer
  0 siblings, 1 reply; 2+ messages in thread
From: Juergen Beisert @ 2010-03-17 13:04 UTC (permalink / raw)
  To: barebox

Signed-off-by: Juergen Beisert <jbe@pengutronix.de>

---
 commands/nand.c         |    2 +-
 drivers/nand/nand_imx.c |    6 ++++--
 2 files changed, 5 insertions(+), 3 deletions(-)

Index: barebox-2010.03.0/commands/nand.c
===================================================================
--- barebox-2010.03.0.orig/commands/nand.c
+++ barebox-2010.03.0/commands/nand.c
@@ -363,6 +363,6 @@ static const __maybe_unused char cmd_nan
 
 BAREBOX_CMD_START(nand)
 	.cmd		= do_nand,
-	.usage		= "",
+	.usage		= "NAND specific handling",
 	BAREBOX_CMD_HELP(cmd_nand_help)
 BAREBOX_CMD_END
Index: barebox-2010.03.0/drivers/nand/nand_imx.c
===================================================================
--- barebox-2010.03.0.orig/drivers/nand/nand_imx.c
+++ barebox-2010.03.0/drivers/nand/nand_imx.c
@@ -1166,11 +1166,13 @@ static int do_nand_boot_test(struct comm
 }
 
 static const __maybe_unused char cmd_nand_boot_test_help[] =
-"Usage: nand_boot_test <dest> <size>\n";
+"Usage: nand_boot_test <dest> <size>\n"
+"This command loads the booloader from the NAND memory like the reset\n"
+"routine does. Its intended for development tests only";
 
 BAREBOX_CMD_START(nand_boot_test)
 	.cmd		= do_nand_boot_test,
-	.usage		= "list a file or directory",
+	.usage		= "load bootloader from NAND",
 	BAREBOX_CMD_HELP(cmd_nand_boot_test_help)
 BAREBOX_CMD_END
 #endif

-- 
Pengutronix e.K.                              | Juergen Beisert             |
Linux Solutions for Science and Industry      | Phone: +49-8766-939 228     |
Vertretung Sued/Muenchen, Germany             | Fax:   +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686              | http://www.pengutronix.de/  |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Make some help comments less confusing
  2010-03-17 13:04 Make some help comments less confusing Juergen Beisert
@ 2010-03-18  7:11 ` Sascha Hauer
  0 siblings, 0 replies; 2+ messages in thread
From: Sascha Hauer @ 2010-03-18  7:11 UTC (permalink / raw)
  To: Juergen Beisert; +Cc: barebox


applied to -master.

Sascha

On Wed, Mar 17, 2010 at 02:04:45PM +0100, Juergen Beisert wrote:
> Signed-off-by: Juergen Beisert <jbe@pengutronix.de>
> 
> ---
>  commands/nand.c         |    2 +-
>  drivers/nand/nand_imx.c |    6 ++++--
>  2 files changed, 5 insertions(+), 3 deletions(-)
> 
> Index: barebox-2010.03.0/commands/nand.c
> ===================================================================
> --- barebox-2010.03.0.orig/commands/nand.c
> +++ barebox-2010.03.0/commands/nand.c
> @@ -363,6 +363,6 @@ static const __maybe_unused char cmd_nan
>  
>  BAREBOX_CMD_START(nand)
>  	.cmd		= do_nand,
> -	.usage		= "",
> +	.usage		= "NAND specific handling",
>  	BAREBOX_CMD_HELP(cmd_nand_help)
>  BAREBOX_CMD_END
> Index: barebox-2010.03.0/drivers/nand/nand_imx.c
> ===================================================================
> --- barebox-2010.03.0.orig/drivers/nand/nand_imx.c
> +++ barebox-2010.03.0/drivers/nand/nand_imx.c
> @@ -1166,11 +1166,13 @@ static int do_nand_boot_test(struct comm
>  }
>  
>  static const __maybe_unused char cmd_nand_boot_test_help[] =
> -"Usage: nand_boot_test <dest> <size>\n";
> +"Usage: nand_boot_test <dest> <size>\n"
> +"This command loads the booloader from the NAND memory like the reset\n"
> +"routine does. Its intended for development tests only";
>  
>  BAREBOX_CMD_START(nand_boot_test)
>  	.cmd		= do_nand_boot_test,
> -	.usage		= "list a file or directory",
> +	.usage		= "load bootloader from NAND",
>  	BAREBOX_CMD_HELP(cmd_nand_boot_test_help)
>  BAREBOX_CMD_END
>  #endif
> 
> -- 
> Pengutronix e.K.                              | Juergen Beisert             |
> Linux Solutions for Science and Industry      | Phone: +49-8766-939 228     |
> Vertretung Sued/Muenchen, Germany             | Fax:   +49-5121-206917-5555 |
> Amtsgericht Hildesheim, HRA 2686              | http://www.pengutronix.de/  |
> 
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
> 

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2010-03-18  7:12 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-03-17 13:04 Make some help comments less confusing Juergen Beisert
2010-03-18  7:11 ` Sascha Hauer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox