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 devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel