diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-10-08 15:51:57 -0400 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-10-08 15:51:57 -0400 |
commit | 3a2fb2c394ff0f0ea90bacc557147d9ac733d13c (patch) | |
tree | dbf9b563f0b440ff80569bf70d79a5ff29703238 /drivers/gpu/drm/i915/intel_sdvo.c | |
parent | 8166fcead6b29589f264b9d900c0770f501964c3 (diff) |
Revert "drm/i915: Call encoder hotplug for init and resume cases"
This reverts commit 5d250b05918c002b63632c7db91c3c5f924c6a3b.
It results on a deadlock on platforms where we need to (at least
partially) re-init hpd interrupts from power domain code, since
->hot_plug might again grab a power well reference (to do edid/dp_aux
transactions. At least chv is affected.
Reported-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
References: http://mid.gmane.org/20151008133548.GX26517@intel.com
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Diffstat (limited to 'drivers/gpu/drm/i915/intel_sdvo.c')
-rw-r--r-- | drivers/gpu/drm/i915/intel_sdvo.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/gpu/drm/i915/intel_sdvo.c b/drivers/gpu/drm/i915/intel_sdvo.c index 853f4b2f50db..c42b636c2087 100644 --- a/drivers/gpu/drm/i915/intel_sdvo.c +++ b/drivers/gpu/drm/i915/intel_sdvo.c | |||
@@ -2460,6 +2460,7 @@ intel_sdvo_dvi_init(struct intel_sdvo *intel_sdvo, int device) | |||
2460 | * Ensure that they get re-enabled when an interrupt happens. | 2460 | * Ensure that they get re-enabled when an interrupt happens. |
2461 | */ | 2461 | */ |
2462 | intel_encoder->hot_plug = intel_sdvo_enable_hotplug; | 2462 | intel_encoder->hot_plug = intel_sdvo_enable_hotplug; |
2463 | intel_sdvo_enable_hotplug(intel_encoder); | ||
2463 | } else { | 2464 | } else { |
2464 | intel_connector->polled = DRM_CONNECTOR_POLL_CONNECT | DRM_CONNECTOR_POLL_DISCONNECT; | 2465 | intel_connector->polled = DRM_CONNECTOR_POLL_CONNECT | DRM_CONNECTOR_POLL_DISCONNECT; |
2465 | } | 2466 | } |