drm/i915/dp: Remove redundant sleep after AUX transaction length check.
authorDhinakaran Pandiyan <[email protected]>
Fri, 23 Feb 2018 22:15:19 +0000 (14:15 -0800)
committerRodrigo Vivi <[email protected]>
Tue, 27 Feb 2018 20:06:34 +0000 (12:06 -0800)
The core already takes care of the delay before retrying. The delay now
changes to (500, 600)us instead of (500 + 1000, 600 + 1500)us.

Cc: Rodrigo Vivi <[email protected]>
Signed-off-by: Dhinakaran Pandiyan <[email protected]>
Reviewed-by: David Weinehall <[email protected]>
Signed-off-by: Rodrigo Vivi <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
drivers/gpu/drm/i915/intel_dp.c

index 62fcf77f0551eb6af2278887c92af28cf36ac226..41b0e27a157c81a3b37975d60188cee473439c6b 100644 (file)
@@ -1205,14 +1205,6 @@ done:
        if (recv_bytes == 0 || recv_bytes > 20) {
                DRM_DEBUG_KMS("Forbidden recv_bytes = %d on aux transaction\n",
                              recv_bytes);
-               /*
-                * FIXME: This patch was created on top of a series that
-                * organize the retries at drm level. There EBUSY should
-                * also take care for 1ms wait before retrying.
-                * That aux retries re-org is still needed and after that is
-                * merged we remove this sleep from here.
-                */
-               usleep_range(1000, 1500);
                ret = -EBUSY;
                goto out;
        }