<div dir="ltr">ah... got your point...<div style>I just split later because Ben wanted the frequency patch as the first one so I decided to let split at last patch to be really optional...</div><div style>so, you suggestion is to revert the order of this two latest patches or the 3?</div>
<div style>I guess frequency one was already queued right?</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Mar 26, 2013 at 1:30 PM, Daniel Vetter <span dir="ltr"><<a href="mailto:daniel@ffwll.ch" target="_blank">daniel@ffwll.ch</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Tue, Mar 26, 2013 at 5:25 PM, Rodrigo Vivi <<a href="mailto:rodrigo.vivi@gmail.com">rodrigo.vivi@gmail.com</a>> wrote:<br>

> I just checked the code and this patch looks right for me.<br>
> it doesn't add any if block... just remove them.<br>
> What am I missing?<br>
<br>
</div>You've added it right in the previous patch ;-)<br>
<br>
Which means if someone tries to understand the history of a given<br>
piece of code with git blame, they now have to jump through these 2<br>
patches which change nothing and are right following each another. But<br>
in the usual recursive git blame mode you don't see that (or at least<br>
I don't check for that by default), so you end up reading both patches<br>
to make sure you still see where the code is moving around.<br>
<br>
So if you want to split (and I agree that it starts to make sense),<br>
pls split first, then apply hsw changes to the hsw rps code only.<br>
<span class="HOEnZb"><font color="#888888">-Daniel<br>
</font></span><div class="HOEnZb"><div class="h5">--<br>
Daniel Vetter<br>
Software Engineer, Intel Corporation<br>
<a href="tel:%2B41%20%280%29%2079%20365%2057%2048" value="+41793655748">+41 (0) 79 365 57 48</a> - <a href="http://blog.ffwll.ch" target="_blank">http://blog.ffwll.ch</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Rodrigo Vivi</div><div>Blog: <a href="http://blog.vivi.eng.br" target="_blank">http://blog.vivi.eng.br</a></div><div> </div>
</div>