<div dir="ltr">Fixed and resent, thanks!<div><br></div><div>David</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Aug 27, 2016 at 11:28 PM, Bruno Jesus <span dir="ltr"><<a href="mailto:00cpxxx@gmail.com" target="_blank">00cpxxx@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sun, Aug 28, 2016 at 3:18 AM, David Lawrie <<a href="mailto:david.dljunk@gmail.com">david.dljunk@gmail.com</a>> wrote:<br>
> Adds device_disabled_registry, helper functions to shared joystick code<br>
<br>
</span>Hi, thanks for working on this. The problem with this series is that<br>
you are introducing dead code in patches 1, 2 and 3. As the code is<br>
not used anywhere until patch 5.<br>
Patch 4 also adds dead code that is only used in patch 5.<br>
<br>
IMO you should merge patches 1,2,3 into a single patch and also 4 and<br>
5 into second patch. This will make the dead code will last a single<br>
commit.<br>
<br>
Best wishes,<br>
Bruno<br>
</blockquote></div><br></div>