On Thu, Jan 12, 2012 at 23:24, Jesse Barnes <span dir="ltr"><<a href="mailto:jbarnes@virtuousgeek.org">jbarnes@virtuousgeek.org</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">On Thu, 12 Jan 2012 23:21:10 +0100<br>
Vincent Vanackere <<a href="mailto:vincent.vanackere@gmail.com">vincent.vanackere@gmail.com</a>> wrote:<br>
> I'm happy to report that this patch indeed fixes the issue for me (tested<br>
> both on linus' master and drm-intel-next).<br>
> Note that I had to apply it by hand (the patch was mangled by your mailer<br>
> ?), therefore I'm resending your fix as an attachment in case other people<br>
> are interested.<br>
><br>
> Many thanks for the quick fix !<br>
<br>
</div>Great, thanks for testing.  Your fixed up patch was missing one line<br>
though, can you re-test with it added?<br>
<br>
After the:<br>
        temp &= ~PORT_TRANS_SEL_MASK;<br>
you need to also add a:<br>
        temp |= PORT_TRANS_SEL_CPT(pipe);<br>
<br>
in case you end up running the LVDS off of transcoder B.<br>
<br></blockquote></div><br>Tested with the (attached, hopefully correct) patch and it still works. :-)<br><br>Vincent<br><br>