diff options
author | Adam Jackson <ajax@redhat.com> | 2011-07-26 15:39:46 -0400 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2012-01-17 10:47:00 -0500 |
commit | d7e96feab83d29e27d14c60f1fa7c716ef7880cd (patch) | |
tree | 84e29428164c44545ff3d533d73034e61f12fc7b /drivers/gpu/drm | |
parent | 092945e11c5b84f66dd08f0b87fb729715d377bc (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.c | 4 |
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 | } |