On Fri, 04 Apr 2014, Jani Nikula <jani.nikula at linux.intel.com> wrote:
> On Fri, 04 Apr 2014, Dave Airlie <airlied at gmail.com> wrote:
>> From: Dave Airlie <airlied at redhat.com>
>>
>> If we get a msg.reply of REPLY_DEFER, we also get an err of 0
>> so we fail reads with 0 < size and return -EPROTO instead of trying
>> again.
>>
>> Found writing MST support.
>>
>
> Reviewed-by: Jani Nikula <jani.nikula at intel.com>
On second thought, I think you'll need the same for drm_dp_i2c_do_msg()
as well.
Cheers,
Jani.
>
>
>> Signed-off-by: Dave Airlie <airlied at redhat.com>
>> ---
>> drivers/gpu/drm/drm_dp_helper.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/drm_dp_helper.c
>> b/drivers/gpu/drm/drm_dp_helper.c
>> index f4babed..725354f 100644
>> --- a/drivers/gpu/drm/drm_dp_helper.c
>> +++ b/drivers/gpu/drm/drm_dp_helper.c
>> @@ -386,11 +386,11 @@ static int drm_dp_dpcd_access(struct drm_dp_aux *aux,
>> u8 request,
>> return err;
>> }
>>
>> - if (err < size)
>> - return -EPROTO;
>>
>> switch (msg.reply & DP_AUX_NATIVE_REPLY_MASK) {
>> case DP_AUX_NATIVE_REPLY_ACK:
>> + if (err < size)
>> + return -EPROTO;
>> return err;
>>
>> case DP_AUX_NATIVE_REPLY_NACK:
>> --
>> 1.8.5.3
>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
> --
> Jani Nikula, Intel Open Source Technology Center
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Jani Nikula, Intel Open Source Technology Center
