<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Thu, Nov 8, 2018 at 12:56 PM Thadeu Lima de Souza Cascardo <<a href="mailto:cascardo@canonical.com">cascardo@canonical.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">That's not the most clear way of doing this. You are still calling the quirk<br>
for all devices on bus 0, and getting out if the option was off. I would<br>
not call an operation done for any/all devices a quirk, so would<br>
definitively do it as a new function.<br></blockquote><div><br></div><div>No worries, I agree and I get it.  In fact, Guilherme had a different patch</div><div>that would do something similar to what you say, however, that is somewhat</div><div>different from what was posted upstream, and I was asked to post the</div><div>upstream-based version (for that's what has been reviewed previously)</div><div>while Guilherme is out on vacation; so, that's why I have just moved</div><div>things around, which makes the quirk still to be called, but it bails out.<br></div><div></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
But I may be accused of bikeshedding or nitpicking, so I'll give my ACK.<br></blockquote><div><br></div><div>Understand :)  Thanks.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Acked-by: Thadeu Lima de Souza Cascardo <<a href="mailto:cascardo@canonical.com" target="_blank">cascardo@canonical.com</a>><br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><span><div><div dir="ltr"><div><div dir="ltr"><div>Mauricio Faria de Oliveira</div></div></div></div></div></span></div></div></div>