[coreboot] [PATCH] flashrom: Add --programmer to help text and man page

Uwe Hermann uwe at hermann-uwe.de
Sat May 9 01:47:20 CEST 2009


On Sat, May 09, 2009 at 01:37:00AM +0200, Carl-Daniel Hailfinger wrote:
> Add --programmer to help text and man page.
> 
> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>

With the changes below:
Acked-by: Uwe Hermann <uwe at hermann-uwe.de>


> Index: flashrom-programmer_help/flashrom.8
> ===================================================================
> --- flashrom-programmer_help/flashrom.8	(Revision 480)
> +++ flashrom-programmer_help/flashrom.8	(Arbeitskopie)
> @@ -123,6 +123,11 @@
>  Please let us know if you can verify other boards to work or not work out
>  of the box.
>  .TP
> +.B "\-p, \-\-programmer <name>"
> +Specify the programmer device. Currently supported are:
> +.sp

> +  internal (default, for onboard flash)

.RB " -" " internal" " (default, for in-system flashing in the mainboard)"


> +.TP
>  .B "\-h, \-\-help"
>  Show a help text and exit.
>  .TP
> Index: flashrom-programmer_help/flashrom.c
> ===================================================================
> --- flashrom-programmer_help/flashrom.c	(Revision 480)
> +++ flashrom-programmer_help/flashrom.c	(Arbeitskopie)
> @@ -313,6 +313,7 @@
>  	     "   -l | --layout <file.layout>:      read rom layout from file\n"
>  	     "   -i | --image <name>:              only flash image name from flash layout\n"
>  	     "   -L | --list-supported:            print supported devices\n"
> +	     "   -p | --programmer <name>:         specify the programmer device or internal\n"

I'd drop the " or internal". A list is / will be in the manpage, and
there's no need to mention "internal" here, it's the default anyway.


Uwe.
-- 
http://www.hermann-uwe.de  | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org




More information about the coreboot mailing list