drm/i915: Remove intel_crtc->atomic.disable_ips.
authorMaarten Lankhorst <maarten.lankhorst@linux.intel.com>
Wed, 3 Feb 2016 15:53:23 +0000 (16:53 +0100)
committerMaarten Lankhorst <maarten.lankhorst@linux.intel.com>
Mon, 8 Feb 2016 13:21:35 +0000 (14:21 +0100)
This is a revert of commit 066cf55b9ce3 "drm/i915: Fix IPS related flicker".
intel_pre_disable_primary already handles this, and now everything
goes through the atomic path there's no need to try to disable ips twice.

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Reviewed-by: Ander Conselvan de Oliveira <conselvan2@gmail.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1454514805-10595-2-git-send-email-maarten.lankhorst@linux.intel.com
drivers/gpu/drm/i915/intel_display.c
drivers/gpu/drm/i915/intel_drv.h

index 186d6cadba849fb9112820e0f666ecffe89f7e4e..4760ecb6ae06abaa04eb410672c94e0b946096a4 100644 (file)
@@ -4815,9 +4815,6 @@ static void intel_pre_plane_update(struct intel_crtc *crtc)
        if (atomic->update_fbc)
                intel_fbc_pre_update(crtc);
 
-       if (crtc->atomic.disable_ips)
-               hsw_disable_ips(crtc);
-
        if (atomic->pre_disable_primary)
                intel_pre_disable_primary(&crtc->base);
 
@@ -11889,18 +11886,6 @@ int intel_plane_atomic_calc_changes(struct drm_crtc_state *crtc_state,
                intel_crtc->atomic.post_enable_primary = turn_on;
                intel_crtc->atomic.update_fbc = true;
 
-               if (turn_off) {
-                       /*
-                        * FIXME: Actually if we will still have any other
-                        * plane enabled on the pipe we could let IPS enabled
-                        * still, but for now lets consider that when we make
-                        * primary invisible by setting DSPCNTR to 0 on
-                        * update_primary_plane function IPS needs to be
-                        * disable.
-                        */
-                       intel_crtc->atomic.disable_ips = true;
-               }
-
                /*
                 * BDW signals flip done immediately if the plane
                 * is disabled, even if the plane enable is already
index 1251a7ae4a6ac03a95c2c3b2ec7c81290f828912..5fd3e13936e4ff29e15d7d9c1549562c40abd6bc 100644 (file)
@@ -544,7 +544,6 @@ struct intel_mmio_flip {
  */
 struct intel_crtc_atomic_commit {
        /* Sleepable operations to perform before commit */
-       bool disable_ips;
        bool pre_disable_primary;
 
        /* Sleepable operations to perform after commit */