[Intel-gfx] [PATCH] drm/i915/dp: Check DP sink status after training is complete

Adam Jackson ajax at redhat.com
Tue Feb 14 20:49:01 CET 2012


This patch is wrong and please do not merge it as is.  My hope here is
that the sink will actually _tell_ us when it can't sink something.

If the bug that prompted this is doing what I think it's doing, then we
should see SINK_STATUS fail to light up, and we should then retry link
setup at the lower link rate like the spec suggests.

Bugzilla: https://bugs.freedesktop.org/45801

Not-signed-off-by: Adam Jackson <ajax at redhat.com>
---
 drivers/gpu/drm/i915/intel_dp.c |   11 +++++++++++
 1 files changed, 11 insertions(+), 0 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
index 94f860c..a10775e 100644
--- a/drivers/gpu/drm/i915/intel_dp.c
+++ b/drivers/gpu/drm/i915/intel_dp.c
@@ -1271,6 +1271,17 @@ static void intel_dp_commit(struct drm_encoder *encoder)
 
 	if (HAS_PCH_CPT(dev))
 		intel_cpt_verify_modeset(dev, intel_crtc->pipe);
+
+	/* check sink status. this doesn't belong here. */
+	do {
+		uint8_t byte;
+		int ret = intel_dp_aux_native_read(intel_dp, DP_SINK_STATUS,
+						   &byte, 1);
+		if (ret != 1)
+			DRM_ERROR("Checking DP sink status failed\n");
+		else
+			DRM_DEBUG_KMS("Sink status after training: %x\n", byte);
+	} while (0);
 }
 
 static void
-- 
1.7.7.6




More information about the Intel-gfx mailing list