drm/radeon/dp: return -EIO for flags not zero case
authorAlex Deucher <alexander.deucher@amd.com>
Thu, 3 Jul 2014 15:17:55 +0000 (11:17 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 10 Jul 2014 21:01:28 +0000 (17:01 -0400)
If there are error flags in the aux transaction return
-EIO rather than -EBUSY.  -EIO restarts the whole transaction
while -EBUSY jus retries.  Fixes problematic aux transfers.

Bug:
https://bugs.freedesktop.org/show_bug.cgi?id=80684

Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Cc: stable@vger.kernel.org
drivers/gpu/drm/radeon/atombios_dp.c

index 35f4182c63b66e68c1d0b9e371fbcdeb5716c70f..b1e11f8434e28badd30572219b1d095a6a06adc4 100644 (file)
@@ -127,7 +127,7 @@ static int radeon_process_aux_ch(struct radeon_i2c_chan *chan,
        /* flags not zero */
        if (args.v1.ucReplyStatus == 2) {
                DRM_DEBUG_KMS("dp_aux_ch flags not zero\n");
-               r = -EBUSY;
+               r = -EIO;
                goto done;
        }