aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/gpu/drm
diff options
context:
space:
mode:
authorAdam Jackson <ajax@redhat.com>2011-07-26 15:39:46 -0400
committerDaniel Vetter <daniel.vetter@ffwll.ch>2012-01-17 10:47:00 -0500
commitd7e96feab83d29e27d14c60f1fa7c716ef7880cd (patch)
tree84e29428164c44545ff3d533d73034e61f12fc7b /drivers/gpu/drm
parent092945e11c5b84f66dd08f0b87fb729715d377bc (diff)
drm/i915/dp: Check for AUXCH error before checking for success
This is paranoid, but I am entirely willing to believe the hardware could come up with a condition where I get a status with both the 'done' and 'receive error' bits set. Signed-off-by: Adam Jackson <ajax@redhat.com> Acked-by: Jesse Barnes <jbarnes@virtuousgeek.org> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/gpu/drm')
-rw-r--r--drivers/gpu/drm/i915/intel_dp.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
index 2f4766385797..8f1148c04108 100644
--- a/drivers/gpu/drm/i915/intel_dp.c
+++ b/drivers/gpu/drm/i915/intel_dp.c
@@ -426,6 +426,10 @@ intel_dp_aux_ch(struct intel_dp *intel_dp,
426 DP_AUX_CH_CTL_DONE | 426 DP_AUX_CH_CTL_DONE |
427 DP_AUX_CH_CTL_TIME_OUT_ERROR | 427 DP_AUX_CH_CTL_TIME_OUT_ERROR |
428 DP_AUX_CH_CTL_RECEIVE_ERROR); 428 DP_AUX_CH_CTL_RECEIVE_ERROR);
429
430 if (status & (DP_AUX_CH_CTL_TIME_OUT_ERROR |
431 DP_AUX_CH_CTL_RECEIVE_ERROR))
432 continue;
429 if (status & DP_AUX_CH_CTL_DONE) 433 if (status & DP_AUX_CH_CTL_DONE)
430 break; 434 break;
431 } 435 }