Attachment 102203 Details for Bug 80684 – return -EIO for flags not zero

[patch] return -EIO for flags not zero

0001-drm-radeon-dp-return-EIO-for-flags-not-zero-case.patch (text/plain), 1.07 KB, created by Alex Deucher on 2014-07-03 15:23:44 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Alex Deucher

Created: 2014-07-03 15:23:44 UTC

Size: 1.07 KB

patch

obsolete

>From 819073ec89a06c3afe3e0ec1367329c62d89a4f4 Mon Sep 17 00:00:00 2001
>From: Alex Deucher <alexander.deucher@amd.com>
>Date: Thu, 3 Jul 2014 11:17:55 -0400
>Subject: [PATCH] drm/radeon/dp: return -EIO for flags not zero case
>
>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 | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/gpu/drm/radeon/atombios_dp.c b/drivers/gpu/drm/radeon/atombios_dp.c
>index 35f4182..b1e11f8 100644
>--- a/drivers/gpu/drm/radeon/atombios_dp.c
>+++ b/drivers/gpu/drm/radeon/atombios_dp.c
>@@ -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;
> 	}
> 
>-- 
>1.8.3.1
>

Actions: View | Splinter Review

Attachments on bug 80684: 102076 | 102173 | 102203

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.