drm/i915/crt: Configure connector->polled and encoder->hpd_pin consistently
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Tue, 21 Jan 2020 17:10:52 +0000 (19:10 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Tue, 4 Feb 2020 21:14:33 +0000 (23:14 +0200)
Let's make sure encoder->hpd_pin and connector->polled are mirror
images of each other (when we want to use polling). The other
potentially polled connectors (sdvo and tv) already get this right.

Also nuke the redundant force_hotplug_required initialization
(the thing is kzalloc()ed).

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200121171100.4370-4-ville.syrjala@linux.intel.com
Reviewed-by: Jani Nikula <jani.nikula@intel.com>
drivers/gpu/drm/i915/display/intel_crt.c

index e471175..0e2f63b 100644 (file)
@@ -1033,6 +1033,8 @@ void intel_crt_init(struct drm_i915_private *dev_priv)
            !dmi_check_system(intel_spurious_crt_detect)) {
                crt->base.hpd_pin = HPD_CRT;
                crt->base.hotplug = intel_encoder_hotplug;
+       } else {
+               intel_connector->polled = DRM_CONNECTOR_POLL_CONNECT;
        }
 
        if (HAS_DDI(dev_priv)) {
@@ -1063,14 +1065,6 @@ void intel_crt_init(struct drm_i915_private *dev_priv)
 
        drm_connector_helper_add(connector, &intel_crt_connector_helper_funcs);
 
-       if (!I915_HAS_HOTPLUG(dev_priv))
-               intel_connector->polled = DRM_CONNECTOR_POLL_CONNECT;
-
-       /*
-        * Configure the automatic hotplug detection stuff
-        */
-       crt->force_hotplug_required = false;
-
        /*
         * 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