<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, May 7, 2018 at 4:05 PM, Scott D Phillips <span dir="ltr"><<a href="mailto:scott.d.phillips@intel.com" target="_blank">scott.d.phillips@intel.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="">Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> writes:<br>
<br>
> If I'm honest, I don't really like the way this patch worked out.  It has<br>
> the virtue of being fairly simple and a nice incremental change.  However,<br>
> it seems to me like we should be able to do better.  That said, I don't<br>
> really know how off-hand and this looks correct and gets us going.<br>
<br>
</span>I think the real opportunities for cleanups from this will only come if<br>
we were to fork the driver and split out the code for the<br>
pre-softpin-capable platforms. I think it's just going to be a bit of a<br>
platypus as long as it's handling both methods.<br>
</blockquote></div><br></div><div class="gmail_extra">Agreed.  Hence me not really objecting. :-)<br></div></div>