On Jan 12, 2012 8:45 PM, "Jesse Barnes" <<a href="mailto:jbarnes@virtuousgeek.org">jbarnes@virtuousgeek.org</a>> wrote: > > On Thu, 12 Jan 2012 23:42:35 +0100 > Vincent Vanackere <<a href="mailto:vincent.vanackere@gmail.com">vincent.vanackere@gmail.com</a>> wrote: > > > On Thu, Jan 12, 2012 at 23:24, Jesse Barnes <<a href="mailto:jbarnes@virtuousgeek.org">jbarnes@virtuousgeek.org</a>>wrote: > > > > > On Thu, 12 Jan 2012 23:21:10 +0100 > > > Vincent Vanackere <<a href="mailto:vincent.vanackere@gmail.com">vincent.vanackere@gmail.com</a>> wrote: > > > > I'm happy to report that this patch indeed fixes the issue for me (tested > > > > both on linus' master and drm-intel-next). > > > > Note that I had to apply it by hand (the patch was mangled by your mailer > > > > ?), therefore I'm resending your fix as an attachment in case other > > > people > > > > are interested. > > > > > > > > Many thanks for the quick fix ! > > > > > > Great, thanks for testing. Your fixed up patch was missing one line > > > though, can you re-test with it added? > > > > > > After the: > > > temp &= ~PORT_TRANS_SEL_MASK; > > > you need to also add a: > > > temp |= PORT_TRANS_SEL_CPT(pipe); > > > > > > in case you end up running the LVDS off of transcoder B. > > > > > > > > Tested with the (attached, hopefully correct) patch and it still works. :-) > > Yay, thanks a lot! I'll send it upstream now. > Jesse, if you want, you can include my r-b into it as well!