<div dir="ltr">Yep. Good to go.<br><br>
Reviewed-by: Todd Previte <<a href="mailto:tprevite@gmail.com" target="_blank">tprevite@gmail.com</a>><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Sep 27, 2013 at 4:51 AM, Jani Nikula <span dir="ltr"><<a href="mailto:jani.nikula@linux.intel.com" target="_blank">jani.nikula@linux.intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Fri, 20 Sep 2013, Todd Previte <<a href="mailto:tprevite@gmail.com">tprevite@gmail.com</a>> wrote:<br>

</div><div><div class="h5">> On 09/20/2013 06:42 AM, Jani Nikula wrote:<br>
>> Per DP1.2 spec.<br>
>><br>
>> Signed-off-by: Jani Nikula <<a href="mailto:jani.nikula@intel.com">jani.nikula@intel.com</a>><br>
>> ---<br>
>>   drivers/gpu/drm/i915/intel_dp.c |    7 ++++++-<br>
>>   1 file changed, 6 insertions(+), 1 deletion(-)<br>
>><br>
>> diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c<br>
>> index 9770160..6626514 100644<br>
>> --- a/drivers/gpu/drm/i915/intel_dp.c<br>
>> +++ b/drivers/gpu/drm/i915/intel_dp.c<br>
>> @@ -654,7 +654,12 @@ intel_dp_i2c_aux_ch(struct i2c_adapter *adapter, int mode,<br>
>>              break;<br>
>>      }<br>
>><br>
>> -    for (retry = 0; retry < 5; retry++) {<br>
>> +    /*<br>
>> +     * DP1.2 sections 2.7.7.1.5.6.1 and 2.7.7.1.6.6.1: A DP Source device is<br>
>> +     * required to retry at least seven times upon receiving AUX_DEFER<br>
>> +     * before giving up the AUX transaction.<br>
>> +     */<br>
>> +    for (retry = 0; retry < 7; retry++) {<br>
>>              ret = intel_dp_aux_ch(intel_dp,<br>
>>                                    msg, msg_bytes,<br>
>>                                    reply, reply_bytes);<br>
>><br>
> Hey Jani,<br>
><br>
> Although it's not explicitly stated in the specification (I went through<br>
> both the 1.1a and 1.2a specifications and couldn't find it), I think the<br>
> the retry count of 7 applies to all AUX transactions, both native and<br>
> I2C. That's alluded to in the description of the link training sequence<br>
> (pg 382, 2nd paragraph from the bottom) where the sink may defer up to<br>
> seven times before the source can abort training.<br>
><br>
> With that in mind, it might be a good idea to use a defined constant for<br>
> the retry count for both native and I2C AUX transactions. That would<br>
> keep it consistent throughout the code and be easier to update moving<br>
> forward should the specification change.<br>
<br>
</div></div>Todd, are you willing to give your reviewed-by on this one patch per our<br>
irc discussion? We can do further cleanups later.<br>
<br>
I've sent a new version of 3/4 with the #defines separately.<br>
<br>
<br>
BR,<br>
Jani.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
--<br>
Jani Nikula, Intel Open Source Technology Center<br>
</div></div></blockquote></div><br></div>