]> git.kernelconcepts.de Git - karo-tx-linux.git/commit
drm/i915: rip out intel_disable_pch_ports
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Thu, 6 Sep 2012 20:08:32 +0000 (22:08 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Thu, 20 Sep 2012 12:23:08 +0000 (14:23 +0200)
commita14d33592061032be0f8373627eb13882c857742
tree04c242e37e7d3f0d0a00892a13b3c71f5d2fe3f0
parentedef7e685da05c13cce50c0126189c80fe2c8f71
drm/i915: rip out intel_disable_pch_ports

Even with the old crtc helper code we should have disabled all
encoders on that pipe by now, and with the new code this would
definitely paper over a bug. We already have the necessary checks
in place in intel_disable_transcoder, so if we accidentally leave
a pch port on, this will be caught.

Hence just rip this all out.

Note that up to the patch in this giant modeset series that removes
the LVDS special case to avoid disabling LVDS in the encoder->prepare
callback ("drm/i915/lvds: ditch ->prepare special case"), this was not
the case for all outputs.

Also note that in

commit 1b3c7a47f993bf9ab6c4c7cc3bbf5588052b58f4
Author: Zhenyu Wang <zhenyuw@linux.intel.com>
Date:   Wed Nov 25 13:09:38 2009 +0800

    drm/i915: Fix LVDS stability issue on Ironlake

this was already discovered independently and worked around. How I
bloody hate this entire mess of cludges piled on top of other cludges.

Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_display.c