drm/i915: Drop the eDP check from intel_dp_connector_destroy()

As long as the connector was zeroed during allocation calling
intel_panel_fini() is safe even if we haven't initialized
the panel struct explicitly. So let's drop the useless eDP
check from dp connector destruction.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20181008134641.24868-2-ville.syrjala@linux.intel.com
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>

diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
index 19f0c3f59cbe1b594a4187d5119c90cf62dd5e80..d12f987a6c43e2f53dd0a7fed001d2fbc3d5d42a 100644 (file)
--- a/drivers/gpu/drm/i915/intel_dp.c
+++ b/drivers/gpu/drm/i915/intel_dp.c
@@ -5261,12 +5261,7 @@ intel_dp_connector_destroy(struct drm_connector *connector)
        if (!IS_ERR_OR_NULL(intel_connector->edid))
                kfree(intel_connector->edid);
 
-       /*
-        * Can't call intel_dp_is_edp() since the encoder may have been
-        * destroyed already.
-        */
-       if (connector->connector_type == DRM_MODE_CONNECTOR_eDP)
-               intel_panel_fini(&intel_connector->panel);
+       intel_panel_fini(&intel_connector->panel);
 
        drm_connector_cleanup(connector);
        kfree(connector);