On Jan 12, 2012 8:45 PM, "Jesse Barnes" <jbarnes at virtuousgeek.org> wrote: > > 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, if you want, you can include my r-b into it as well! -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20120112/63344943/attachment.htm>
