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 <[email protected]>
With the changes below: Acked-by: Uwe Hermann <[email protected]> > 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 -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
