On Mon, 14 May 2012 08:12:13 -0700, Jesse Barnes <jbarnes at virtuousgeek.org> wrote: > Thanks Chris. Only thing I'm not sure about is the LPT bit; does this > function do what we want there? The other patch transforms intel_pch_enable_pll (hmm, that's actually a better name ;-) into a no-op if the crtc does not require a pll. Which I think is the easier way of tracking PLLs - allocate on up front if the chipset/output requires a PLL and then we do no need any more checks other than has-pll? in the main body of the code. -Chris -- Chris Wilson, Intel Open Source Technology Centre