drm/i915: reduce duplicate conditions in i9xx_hpd_irq_handler
authorJani Nikula <jani.nikula@intel.com>
Wed, 27 May 2015 12:03:40 +0000 (15:03 +0300)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 29 May 2015 08:15:19 +0000 (10:15 +0200)
Move dp aux irq handling within the same branch instead of duplicating
the conditions. No functional changes.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/i915_irq.c

index f0206f63617ec6f3132dce0aaf715c329799124a..333b3d9dd5fcbd072102b728cf3791a19b21abdf 100644 (file)
@@ -1770,15 +1770,14 @@ static void i9xx_hpd_irq_handler(struct drm_device *dev)
                u32 hotplug_trigger = hotplug_status & HOTPLUG_INT_STATUS_G4X;
 
                intel_hpd_irq_handler(dev, hotplug_trigger, 0, hpd_status_g4x);
+
+               if (hotplug_status & DP_AUX_CHANNEL_MASK_INT_STATUS_G4X)
+                       dp_aux_irq_handler(dev);
        } else {
                u32 hotplug_trigger = hotplug_status & HOTPLUG_INT_STATUS_I915;
 
                intel_hpd_irq_handler(dev, hotplug_trigger, 0, hpd_status_i915);
        }
-
-       if ((IS_G4X(dev) || IS_VALLEYVIEW(dev)) &&
-           hotplug_status & DP_AUX_CHANNEL_MASK_INT_STATUS_G4X)
-               dp_aux_irq_handler(dev);
 }
 
 static irqreturn_t valleyview_irq_handler(int irq, void *arg)