Hi Robert, On Mon, Nov 01, 2010 at 09:33:39AM +0100, Robert Schwebel wrote: > Signed-off-by: Robert Schwebel <r.schwebel@xxxxxxxxxxxxxx> > --- > Documentation/commands.dox | 1 + > commands/dfu.c | 28 ++++++++++++++++++---------- > 2 files changed, 19 insertions(+), 10 deletions(-) > [snip] > -static const __maybe_unused char cmd_dfu_help[] = > -"Usage: dfu [OPTION]... description\n" > -"start dfu firmware update\n" > -" -m <str> Manufacturer string (barebox)\n" > -" -p <str> product string (" CONFIG_BOARDINFO ")\n" > -" -V <id> vendor id\n" > -" -P <id> product id\n" > -"description has the form\n" > -"device1(name1)[sr],device2(name2)[sr]\n" > -"where s is for save mode and r for read back of firmware\n"; > +BAREBOX_CMD_HELP_START(dfu) > +BAREBOX_CMD_HELP_USAGE("dfu [OPTIONS] <description>\n") > +BAREBOX_CMD_HELP_SHORT("Start firmware update with the Device Firmware Update (DFU) protocol.") > +BAREBOX_CMD_HELP_OPT ("-m <str>", "Manufacturer string (barebox)\n") > +BAREBOX_CMD_HELP_OPT ("-p <str>", "product string (" CONFIG_BOARDINFO ")\n") > +BAREBOX_CMD_HELP_OPT ("-V <id>", "vendor id\n") > +BAREBOX_CMD_HELP_OPT ("-P <id>", "product id\n") > +BAREBOX_CMD_HELP_END > + > +/** > + * @page dfu_command > +\<description> has the following form: > +device1(name1)[sr],device2)[sr] Should probably be: device1(name1)[sr],device2(name2)[sr] baruch > +'s' means 'safe mode' (download the complete image before flashing) and > +'r' that readback of the firmware is allowed. > + > +\todo Add example, how to use dfu from a Linux or Windows host. > + */ > > BAREBOX_CMD_START(dfu) > .cmd = do_dfu, -- ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - baruch@xxxxxxxxxx - tel: +972.2.679.5364, http://www.tkos.co.il - _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox