On Wed, Mar 02, 2011 at 06:10:56PM +0300, Dan Carpenter wrote:
> On Wed, Mar 02, 2011 at 02:34:40PM +0000, Philip Worrall wrote:
> > -/*+
> > +/*
> >   *
> 
> The blank line isn't needed here.
> 
> >   * Routine Description:
> >   * Enable hw power saving functions
> > @@ -68,7 +68,7 @@ static int msglevel = MSG_LEVEL_INFO;
> >   * Return Value:
> >   *    None.
> >   *
> 
> Or here.
> 

Actually, it would be even better to write these in kernel-doc format.
See Documentation/kernel-doc-nano-HOWTO.txt

regards,
dan carpenter

_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to