Messages in this thread | | | Date | Mon, 28 Jan 2008 15:28:09 -0800 | From | Randy Dunlap <> | Subject | Re: [PATCH] ipwireless: driver for 3G PC Card |
| |
On Tue, 29 Jan 2008 00:18:03 +0100 Jiri Slaby wrote:
> On 01/28/2008 07:08 PM, Randy Dunlap wrote: > >> +/* > >> + * @return 1 if something has been received from hw > > > > What's with the '@'? > ... > >> + > >> +/*! > > > > and the '!' ? > > > >> + * @return true if the card is physically present. > >> + */ > > What exactly is wrong with @ and !? It's perfectly doxygen-styled.
What part of kernel documentation uses doxygen?
> >> diff --git a/drivers/char/pcmcia/ipwireless/main.c b/drivers/char/pcmcia/ipwireless/main.c > >> new file mode 100644 > >> index 0000000..cab5722 > >> --- /dev/null > >> +++ b/drivers/char/pcmcia/ipwireless/main.c > >> @@ -0,0 +1,496 @@ > [...] > >> +module_param(major, int, 0); > >> +module_param(ipwireless_debug, int, 0); > >> +module_param(ipwireless_loopback, int, 0); > >> +module_param(ipwireless_out_queue, int, 0); > >> +MODULE_PARM_DESC(major, "ttyIPWp major number [0]"); > >> +MODULE_PARM_DESC(ipwireless_debug, "switch on debug messages [0]"); > >> +MODULE_PARM_DESC(ipwireless_debug, "switch on loopback mode [0]"); > >> +MODULE_PARM_DESC(ipwireless_debug, "set size of outgoing queue [1]"); > > > > Will these parameters be documented anywhere? > > Anyway the descs are wrong. Those 3 are for the only one variable.
Ack, thanks.
--- ~Randy
| |