On Sun, 13 May 2012 16:08:32 +0200, Daniel Vetter <daniel at ffwll.ch> wrote: > On Sun, May 13, 2012 at 09:54:09AM +0100, Chris Wilson wrote: > > Hidden away within one chipset specific path was the necessary logic to > > turn on the PLL. This needs to be done everywhere in order for us to > > drive any display! As such as soon as we tested on a non-CougarPoint > > chipset, we failed to bring up any DisplayPorts and generated a nice set > > of assertion failures in the process. At least one part of our logic is > > working, the part that assumes that we have no idea what we are doing. > > > > Reported-by: guang.a.yang at intel.com > > References: https://bugs.freedesktop.org/show_bug.cgi?id=49712 > > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk> > > Cc: Jesse Barnes <jbarnes at virtuousgeek.org> > > I guess as-is this patch will blow up on hsw. I think we need to change > the BUG_ON(!pll) in there into a return, like in the disable code. Eugeni? Whoops, should have looked harder. My thinking was that on hsw, the pll would be NULL and so the routine would have no effect. Tricksy BUG_ON. -Chris -- Chris Wilson, Intel Open Source Technology Centre
