drm/i915: rip out encoder->disable/enable checks
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Sun, 1 Jul 2012 21:24:36 +0000 (23:24 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Thu, 6 Sep 2012 05:55:27 +0000 (07:55 +0200)
All encoders are now converted so there's no need for these checks any
more.

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

index 0d48ebe6ac9d8a183ce61e8cee89facaf068948c..4805385a49ed1d91dc4b27ec1ddb2d5dd4b79511 100644 (file)
@@ -3267,10 +3267,8 @@ static void ironlake_crtc_enable(struct drm_crtc *crtc)
        intel_crtc_update_cursor(crtc, true);
 
 encoders:
-       for_each_encoder_on_crtc(dev, crtc, encoder) {
-               if (encoder->enable)
-                       encoder->enable(encoder);
-       }
+       for_each_encoder_on_crtc(dev, crtc, encoder)
+               encoder->enable(encoder);
 }
 
 static void ironlake_crtc_disable(struct drm_crtc *crtc)
@@ -3285,10 +3283,8 @@ static void ironlake_crtc_disable(struct drm_crtc *crtc)
 
        /* XXX: For compatability with the crtc helper code, call the encoder's
         * disable function unconditionally for now. */
-       for_each_encoder_on_crtc(dev, crtc, encoder) {
-               if (encoder->disable)
-                       encoder->disable(encoder);
-       }
+       for_each_encoder_on_crtc(dev, crtc, encoder)
+               encoder->disable(encoder);
 
        if (!intel_crtc->active)
                return;
@@ -3412,10 +3408,8 @@ static void i9xx_crtc_enable(struct drm_crtc *crtc)
        intel_crtc_update_cursor(crtc, true);
 
 encoders:
-       for_each_encoder_on_crtc(dev, crtc, encoder) {
-               if (encoder->enable)
-                       encoder->enable(encoder);
-       }
+       for_each_encoder_on_crtc(dev, crtc, encoder)
+               encoder->enable(encoder);
 }
 
 static void i9xx_crtc_disable(struct drm_crtc *crtc)
@@ -3429,10 +3423,8 @@ static void i9xx_crtc_disable(struct drm_crtc *crtc)
 
        /* XXX: For compatability with the crtc helper code, call the encoder's
         * disable function unconditionally for now. */
-       for_each_encoder_on_crtc(dev, crtc, encoder) {
-               if (encoder->disable)
-                       encoder->disable(encoder);
-       }
+       for_each_encoder_on_crtc(dev, crtc, encoder)
+               encoder->disable(encoder);
 
        if (!intel_crtc->active)
                return;