[PATCH] drm/i915/dp: remove redundant link state check

[tarund18303]

From: tarun-tarun143 <tarund18303@xxxxxxxxx>

The DP link state is checked and recovered through the encoder link
check work from the modeset and HPD paths.

The detect handler also used to queue the same link check work for
non-eDP sinks. This was added when link recovery was moved to the
encoder's check link state work, and the original change explicitly
left a TODO to remove the detect-time check once it became redundant.

Remove the detect-time link state check and rely on the existing link
recovery paths.

Signed-off-by: tarun-tarun143 <tarund18303@xxxxxxxxx>
---
 drivers/gpu/drm/i915/display/intel_dp.c | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
index 113d767e62e9..ffd39206b958 100644
--- a/drivers/gpu/drm/i915/display/intel_dp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp.c
@@ -6379,18 +6379,6 @@ intel_dp_detect(struct drm_connector *_connector,
 		status = connector_status_disconnected;
 		goto out_unset_edid;
 	}
-
-	/*
-	 * Some external monitors do not signal loss of link synchronization
-	 * with an IRQ_HPD, so force a link status check.
-	 *
-	 * TODO: this probably became redundant, so remove it: the link state
-	 * is rechecked/recovered now after modesets, where the loss of
-	 * synchronization tends to occur.
-	 */
-	if (!intel_dp_is_edp(intel_dp))
-		intel_dp_check_link_state(intel_dp);
-
 	/*
 	 * Clearing NACK and defer counts to get their exact values
 	 * while reading EDID which are required by Compliance tests
-- 
2.43.0