drm/i915/dp: Remove redundant sleep after AUX transaction length check.
authorDhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Fri, 23 Feb 2018 22:15:19 +0000 (14:15 -0800)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
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 <rodrigo.vivi@intel.com>
Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Reviewed-by: David Weinehall <david.weinehall@linux.intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180223221520.18464-5-dhinakaran.pandiyan@intel.com
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;
        }