<br><br><div class="gmail_quote">On 2 October 2012 17:16, Scott Lovenberg <span dir="ltr">&lt;<a href="mailto:scott.lovenberg@gmail.com" target="_blank">scott.lovenberg@gmail.com</a>&gt;</span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Looks good to me.<br></blockquote><div><br></div><div>Maintainer didn&#39;t think so :(</div><div><br></div><div class="gmail_quote">On 2 October 2012 19:46, David Miller <span dir="ltr">&lt;<a href="mailto:davem@davemloft.net" target="_blank">davem@davemloft.net</a>&gt;</span> wrote:<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

That comment and that unconditional if() are documentation.<br><br>Don&#39;t be an automaton and blindly make changes based upon<br><a href="http://checkpatch.pl/" target="_blank">checkpatch.pl</a> output.</blockquote></div>

<div> </div><div>Perhaps I&#39;ll just clean up some of drivers/staging while I learn the process before I dive in to &quot;net&quot; again.</div><div><br></div><div>Matthew Walster</div></div>