<p dir="ltr"><br>
</p>
<p dir="ltr"></p>
<p dir="ltr">On Mon, Dec 15, 2014, 10:04 AM Greg KH &lt;<a href="mailto:greg@kroah.com">greg@kroah.com</a>&gt; wrote:</p>
<blockquote><p dir="ltr">On Mon, Dec 15, 2014 at 04:23:18AM +0000, karthik nayak wrote:<br>
&gt;<br>
&gt; On Mon, Dec 15, 2014, 7:31 AM Greg KH &lt;<a href="mailto:greg@kroah.com">greg@kroah.com</a>&gt; wrote:<br>
&gt;<br>
&gt;     On Sun, Dec 14, 2014 at 07:36:23PM -0600, Jonathan Jin wrote:<br>
&gt;     &gt; This patch cleans up the following coding style issues that are detected<br>
&gt;     by<br>
&gt;     &gt; checkpatch:<br>
&gt;     &gt;         ERROR: trailing whitespace<br>
&gt;     &gt;         ERROR: open brace &#39;{&#39; following struct go on the same line<br>
&gt;<br>
&gt;     Don&#39;t do multiple things in the same patch, make this at least 2<br>
&gt;     patches.<br>
&gt;<br>
&gt;     greg k-h<br>
&gt;<br>
&gt; (Sorry, replied on my phone, default was not set to reply all.)<br>
&gt;<br>
&gt; Wouldn&#39;t this count as one checkpatch fix patch? I mean, being the same file.</p>
<p dir="ltr">Nope, you are doing two different _types_ of checkpatch fixes.  So it<br>
needs to be more than one patch.</p>
<p dir="ltr">Remember, you need to make it as easy as possible for a maintainer to<br>
read your patch, doing more than one thing in a patch makes it much<br>
harder, and it gives them an easy reason to reject to it.</p>
<p dir="ltr">greg k-h<br>
</p>
</blockquote>
<p dir="ltr">Thank you for that </p>
<p dir="ltr"><br>
</p>