diff mbox

[9/9] drm/i915/dp: Use the same aux retry interval as the core.

Message ID 20180127024923.3093-9-dhinakaran.pandiyan@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Dhinakaran Pandiyan Jan. 27, 2018, 2:49 a.m. UTC 
Keeps the wait times consistent.

Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
---
 drivers/gpu/drm/i915/intel_dp.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

Rodrigo Vivi Feb. 1, 2018, 7:04 a.m. UTC | #1 
On Sat, Jan 27, 2018 at 02:49:23AM +0000, Dhinakaran Pandiyan wrote:
> Keeps the wait times consistent.
> 
> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>

in case the other one goes through:

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

;)

> ---
>  drivers/gpu/drm/i915/intel_dp.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
> index 06619998b5a3..3c64b2e92575 100644
> --- a/drivers/gpu/drm/i915/intel_dp.c
> +++ b/drivers/gpu/drm/i915/intel_dp.c
> @@ -1145,9 +1145,11 @@ intel_dp_aux_ch(struct intel_dp *intel_dp,
>  				continue;
>  
>  			if (status & DP_AUX_CH_CTL_RECEIVE_ERROR) {
> -				usleep_range(400, 500);
> +				usleep_range(DP_AUX_RETRY_INTERVAL,
> +					     DP_AUX_RETRY_INTERVAL + 100);
>  				continue;
>  			}
> +
>  			if (status & DP_AUX_CH_CTL_DONE)
>  				goto done;
>  		}
> -- 
> 2.14.1
>
diff mbox

Patch

diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
index 06619998b5a3..3c64b2e92575 100644
--- a/drivers/gpu/drm/i915/intel_dp.c
+++ b/drivers/gpu/drm/i915/intel_dp.c
@@ -1145,9 +1145,11 @@  intel_dp_aux_ch(struct intel_dp *intel_dp,
 				continue;
 
 			if (status & DP_AUX_CH_CTL_RECEIVE_ERROR) {
-				usleep_range(400, 500);
+				usleep_range(DP_AUX_RETRY_INTERVAL,
+					     DP_AUX_RETRY_INTERVAL + 100);
 				continue;
 			}
+
 			if (status & DP_AUX_CH_CTL_DONE)
 				goto done;
 		}