On 2 October 2012 17:16, Scott Lovenberg <scott.lovenberg@gmail.com> wrote:
Looks good to me.

Maintainer didn't think so :(

On 2 October 2012 19:46, David Miller <davem@davemloft.net> wrote:
That comment and that unconditional if() are documentation.

Don't be an automaton and blindly make changes based upon
checkpatch.pl output.
 
Perhaps I'll just clean up some of drivers/staging while I learn the process before I dive in to "net" again.

Matthew Walster