On Sun, May 13, 2012 at 5:54 AM, Chris Wilson <chris at chris-wilson.co.uk>wrote: > + intel_enable_pch_pll(intel_crtc); > + > if (HAS_PCH_LPT(dev)) { > DRM_DEBUG_KMS("LPT detected: programming iCLKIP\n"); > lpt_program_iclkip(crtc); > } else if (HAS_PCH_CPT(dev)) { > Could we just replace this 'else if()' with 'else' to achieve the same functionality? This way, everything will work and we won't touch wrong registers on LPT. -- Eugeni Dodonov <http://eugeni.dodonov.net/> -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20120513/295a1f10/attachment-0001.html>