On Thu, 12 Jan 2012 23:42:35 +0100 Vincent Vanackere <vincent.vanackere at gmail.com> wrote: > On Thu, Jan 12, 2012 at 23:24, Jesse Barnes <jbarnes at virtuousgeek.org>wrote: > > > On Thu, 12 Jan 2012 23:21:10 +0100 > > Vincent Vanackere <vincent.vanackere at gmail.com> 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 Barnes, Intel Open Source Technology Center -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20120112/6843dcbd/attachment-0001.pgp>