DRM/i915: Get rid if the 'hotplug_supported_mask' in struct drm_i915_private.
authorEgbert Eich <eich@suse.de>
Mon, 25 Feb 2013 17:06:52 +0000 (12:06 -0500)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 27 Mar 2013 16:13:43 +0000 (17:13 +0100)
Now since we have replaced the bits to show interest in hotplug IRQs
we can go and nuke the 'hotplug_supported_mask'.

Signed-off-by: Egbert Eich <eich@suse.de>
Acked-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/i915_drv.h
drivers/gpu/drm/i915/intel_crt.c
drivers/gpu/drm/i915/intel_dp.c
drivers/gpu/drm/i915/intel_hdmi.c
drivers/gpu/drm/i915/intel_sdvo.c

index cda598d89f173dea459a17b14c52f3fb89e19abf..1d091ea12fadbbba7f66d5578d98d6e9ce16d830 100644 (file)
@@ -922,7 +922,6 @@ typedef struct drm_i915_private {
        u32 irq_mask;
        u32 gt_irq_mask;
 
-       u32 hotplug_supported_mask;
        struct work_struct hotplug_work;
        bool enable_hotplug_processing;
 
index a4b3e4e01e654d4f877789c3151381f7fa450356..756ae552e7e57fa524fc103fb79476ba4b7262a9 100644 (file)
@@ -799,8 +799,6 @@ void intel_crt_init(struct drm_device *dev)
         */
        crt->force_hotplug_required = 0;
 
-       dev_priv->hotplug_supported_mask |= CRT_HOTPLUG_INT_STATUS;
-
        /*
         * TODO: find a proper way to discover whether we need to set the the
         * polarity and link reversal bits or not, instead of relying on the
index c7c5635f06e0ae6b53e1e93d0133000e39861a6c..ad6bba3ba06acc8a012188b9c623f885885787c4 100644 (file)
@@ -2825,17 +2825,14 @@ intel_dp_init_connector(struct intel_digital_port *intel_dig_port,
                name = "DPDDC-A";
                break;
        case PORT_B:
-               dev_priv->hotplug_supported_mask |= PORTB_HOTPLUG_INT_STATUS;
                intel_encoder->hpd_pin = HPD_PORT_B;
                name = "DPDDC-B";
                break;
        case PORT_C:
-               dev_priv->hotplug_supported_mask |= PORTC_HOTPLUG_INT_STATUS;
                intel_encoder->hpd_pin = HPD_PORT_C;
                name = "DPDDC-C";
                break;
        case PORT_D:
-               dev_priv->hotplug_supported_mask |= PORTD_HOTPLUG_INT_STATUS;
                intel_encoder->hpd_pin = HPD_PORT_D;
                name = "DPDDC-D";
                break;
index 6d92aebadfc3710ce209d8fac374ef4ce5262b61..b9a83d7c804b67d67e895cf285c418774393b40e 100644 (file)
@@ -989,17 +989,14 @@ void intel_hdmi_init_connector(struct intel_digital_port *intel_dig_port,
        case PORT_B:
                intel_hdmi->ddc_bus = GMBUS_PORT_DPB;
                intel_encoder->hpd_pin = HPD_PORT_B;
-               dev_priv->hotplug_supported_mask |= PORTB_HOTPLUG_INT_STATUS;
                break;
        case PORT_C:
                intel_hdmi->ddc_bus = GMBUS_PORT_DPC;
                intel_encoder->hpd_pin = HPD_PORT_C;
-               dev_priv->hotplug_supported_mask |= PORTC_HOTPLUG_INT_STATUS;
                break;
        case PORT_D:
                intel_hdmi->ddc_bus = GMBUS_PORT_DPD;
                intel_encoder->hpd_pin = HPD_PORT_D;
-               dev_priv->hotplug_supported_mask |= PORTD_HOTPLUG_INT_STATUS;
                break;
        case PORT_A:
                intel_encoder->hpd_pin = HPD_PORT_A;
index 0ff557fe2f60974bf86e8d3a992913082a9c580d..8fdd8f82f09d5d1c90c39a55358ae9cff8ebbab0 100644 (file)
@@ -2779,6 +2779,9 @@ bool intel_sdvo_init(struct drm_device *dev, uint32_t sdvo_reg, bool is_sdvob)
                        SDVOB_HOTPLUG_INT_STATUS_I915 : SDVOC_HOTPLUG_INT_STATUS_I915;
        }
 
+       /* Only enable the hotplug irq if we need it, to work around noisy
+        * hotplug lines.
+        */
        if (intel_sdvo->hotplug_active)
                intel_encoder->hpd_pin = HPD_SDVO_B ? HPD_SDVO_B : HPD_SDVO_C;
 
@@ -2810,12 +2813,6 @@ bool intel_sdvo_init(struct drm_device *dev, uint32_t sdvo_reg, bool is_sdvob)
         */
        intel_sdvo->base.cloneable = false;
 
-       /* Only enable the hotplug irq if we need it, to work around noisy
-        * hotplug lines.
-        */
-       if (intel_sdvo->hotplug_active)
-               dev_priv->hotplug_supported_mask |= hotplug_mask;
-
        intel_sdvo_select_ddc_bus(dev_priv, intel_sdvo, sdvo_reg);
 
        /* Set the input timing to the screen. Assume always input 0. */