On 07.10.2010 02:56, Carl-Daniel Hailfinger wrote: > On 06.10.2010 23:57, Uwe Hermann wrote: > >> On Wed, Oct 06, 2010 at 01:19:47AM +0200, Carl-Daniel Hailfinger wrote: >> >> >>> Update the man page to have correct info for the upcoming release. >>> >>> >> Looks good. >> >> Acked-by: Uwe Hermann <[email protected]> >> >> > > Thanks, committed in r1197. > > > >>> Should we mention all contributors with more than one patch? Should we >>> restrict the author list to those who have substantial amounts of code >>> in flashrom? Do we have to ask people if adding their email address to >>> the manpage is OK? >>> >>> >> I'd rather remove the whole list to be honest, it's tedious to keep >> up-to-date and the manpage is the wrong place anyway IMHO (yes, I know >> I'm the one who added it there in the first place). >> >> A short sentence like "Thanks to all contributors" is sufficient IMHO, >> the details are in (C) lines in the source or in >> >> svn log | grep -i Signed-off-by >> >> or similar. >> >> > > Mh. The early authors are missing in that list. You do have a point, > though. Did you mean something like this? >
New version. Additional doc updates and fixes. Signed-off-by: Carl-Daniel Hailfinger <[email protected]> Index: flashrom-authors_programmer_doc/pcidev.c =================================================================== --- flashrom-authors_programmer_doc/pcidev.c (Revision 1199) +++ flashrom-authors_programmer_doc/pcidev.c (Arbeitskopie) @@ -134,7 +134,7 @@ msg_pinfo("PCI devices:\n"); for (i = 0; devs[i].vendor_name != NULL; i++) { - msg_pinfo("%s %s [%02x:%02x]%s\n", devs[i].vendor_name, + msg_pinfo("%s %s [%04x:%04x]%s\n", devs[i].vendor_name, devs[i].device_name, devs[i].vendor_id, devs[i].device_id, (devs[i].status == NT) ? " (untested)" : ""); Index: flashrom-authors_programmer_doc/flashrom.8 =================================================================== --- flashrom-authors_programmer_doc/flashrom.8 (Revision 1199) +++ flashrom-authors_programmer_doc/flashrom.8 (Arbeitskopie) @@ -476,7 +476,10 @@ .B " http://www.flashrom.org/trac/flashrom/newticket"; .sp or on the flashrom mailing list at +.B "<[email protected]>" .sp +We recommend to subscribe first at +.sp .B " http://www.flashrom.org/mailman/listinfo/flashrom"; .sp Using flashrom on laptops is dangerous and may easily make your hardware @@ -492,73 +495,75 @@ .br Please see the individual files. .SH AUTHORS -Andrew Morgan <[email protected]> +Andrew Morgan .br Carl-Daniel Hailfinger .br -Claus Gindhart <[email protected]> +Claus Gindhart .br -David Borg <[email protected]> +David Borg .br -David Hendricks <[email protected]> +David Hendricks .br -Dominik Geyer <[email protected]> +Dominik Geyer .br Eric Biederman .br -Giampiero Giancipoli <[email protected]> +Giampiero Giancipoli .br -Helge Wagner <[email protected]> +Helge Wagner .br -Idwer Vollering <[email protected]> +Idwer Vollering .br -Joe Bao <[email protected]> +Joe Bao .br -Jörg Fischer <[email protected]> +Jörg Fischer .br -Joshua Roys <[email protected]> +Joshua Roys .br -Luc Verhaegen <[email protected]> +Luc Verhaegen .br Li-Ta Lo .br -Markus Boas <[email protected]> +Markus Boas .br -Mattias Mattsson <[email protected]> +Mattias Mattsson .br -Michael Karcher <[email protected]> +Michael Karcher .br -Nikolay Petukhov <[email protected]> +Nikolay Petukhov .br -Patrick Georgi <[email protected]> +Patrick Georgi .br -Peter Lemenkov <[email protected]> +Peter Lemenkov .br -Peter Stuge <[email protected]> +Peter Stuge .br -Reinder E.N. de Haan <[email protected]> +Reinder E.N. de Haan .br -Ronald G. Minnich <[email protected]> +Ronald G. Minnich .br -Ronald Hoogenboom <[email protected]> +Ronald Hoogenboom .br -Sean Nelson <[email protected]> +Sean Nelson .br -Stefan Reinauer <[email protected]> +Stefan Reinauer .br -Stefan Wildemann <[email protected]> +Stefan Wildemann .br -Stephan Guilloux <[email protected]> +Stephan Guilloux .br -Steven James <[email protected]> +Steven James .br -Uwe Hermann <[email protected]> +Uwe Hermann .br Wang Qingpei .br Yinghai Lu .br -some others +some others, please see the flashrom svn changelog for details. +.br +All authors can be reached via e-mail at <[email protected]>. .PP This manual page was written by Uwe Hermann <[email protected]> and Carl-Daniel Hailfinger. Index: flashrom-authors_programmer_doc/print.c =================================================================== --- flashrom-authors_programmer_doc/print.c (Revision 1199) +++ flashrom-authors_programmer_doc/print.c (Arbeitskopie) @@ -238,6 +238,7 @@ printf("\nSupported devices for the %s programmer:\n", programmer_table[PROGRAMMER_DUMMY].name); /* FIXME */ + printf("Dummy device, does nothing and logs all accesses\n"); #endif #if CONFIG_NIC3COM == 1 printf("\nSupported devices for the %s programmer:\n", @@ -286,21 +287,25 @@ printf("\nSupported devices for the %s programmer:\n", programmer_table[PROGRAMMER_SERPROG].name); /* FIXME */ + printf("All programmer devices speaking the serprog protocol\n"); #endif #if CONFIG_BUSPIRATE_SPI == 1 printf("\nSupported devices for the %s programmer:\n", programmer_table[PROGRAMMER_BUSPIRATE_SPI].name); /* FIXME */ + printf("Dangerous Prototypes Bus Pirate\n"); #endif #if CONFIG_DEDIPROG == 1 printf("\nSupported devices for the %s programmer:\n", programmer_table[PROGRAMMER_DEDIPROG].name); /* FIXME */ + printf("Dediprog SF100\n"); #endif #if CONFIG_RAYER_SPI == 1 printf("\nSupported devices for the %s programmer:\n", programmer_table[PROGRAMMER_RAYER_SPI].name); /* FIXME */ + printf("RayeR parallel port programmer\n"); #endif #if CONFIG_NICINTEL_SPI == 1 printf("\nSupported devices for the %s programmer:\n", -- http://www.hailfinger.org/ _______________________________________________ flashrom mailing list [email protected] http://www.flashrom.org/mailman/listinfo/flashrom
