<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Jul 24, 2015 at 11:25 AM Rafael Antognolli <<a href="mailto:rafael.antognolli@intel.com">rafael.antognolli@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Jul 23, 2015 at 04:35:46PM -0700, Rodrigo Vivi wrote:<br>
> If we got to the point where we are trying to stop sink CRC<br>
> the main output of this function was already gotten properly,<br>
> so don't return the error and let userspace use the crc data.<br>
><br>
> Let's replace the errnos returns with some log messages.<br>
<br>
So, up to this commit, there's no way to know that an error has ocurred<br>
and the next CRC calculation can go wrong.<br>
<br>
I know that in a follow up patch this is fixed by trying to stop the<br>
calculation at the beginning too, but just pointing out that this one<br>
specifically has this problem.<br></blockquote><div><br></div><div>Actually it isn't a problem if crc calculation continue enabled. Mainly with the mask s/0x7/0xf fix.</div><div>But it is good to disable and force counter reset anyway. (the following patch you mentioned)</div><div><br></div><div>This patch here is just to highlight that errors during read has more priority than errors on stopping crc.</div><div><br></div><div>Another way would be to create a stop_ret and</div><div>if (!ret and stop_ret)</div><div>     ret = stop_ret;</div><div><br></div><div>But I decided this cleaned version would be better.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Not sure if this is a problem though, since the patches are submitted<br>
together. If not, then<br>
<br>
Reviewed-by: Rafael Antognolli <<a href="mailto:rafael.antognolli@intel.com" target="_blank">rafael.antognolli@intel.com</a>><br></blockquote><div><br></div><div>Thanks!</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
> Signed-off-by: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com" target="_blank">rodrigo.vivi@intel.com</a>><br>
> ---<br>
>  drivers/gpu/drm/i915/intel_dp.c | 4 ++--<br>
>  1 file changed, 2 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c<br>
> index 70a4a37..44f8a32 100644<br>
> --- a/drivers/gpu/drm/i915/intel_dp.c<br>
> +++ b/drivers/gpu/drm/i915/intel_dp.c<br>
> @@ -4021,12 +4021,12 @@ int intel_dp_sink_crc(struct intel_dp *intel_dp, u8 *crc)<br>
><br>
>  stop:<br>
>       if (drm_dp_dpcd_readb(&intel_dp->aux, DP_TEST_SINK, &buf) < 0) {<br>
> -             ret = -EIO;<br>
> +             DRM_DEBUG_KMS("Sink CRC couldn't be stopped properly\n");<br>
>               goto out;<br>
>       }<br>
>       if (drm_dp_dpcd_writeb(&intel_dp->aux, DP_TEST_SINK,<br>
>                              buf & ~DP_TEST_SINK_START) < 0) {<br>
> -             ret = -EIO;<br>
> +             DRM_DEBUG_KMS("Sink CRC couldn't be stopped properly\n");<br>
>               goto out;<br>
>       }<br>
>  out:<br>
> --<br>
> 2.1.0<br>
><br>
> _______________________________________________<br>
> Intel-gfx mailing list<br>
> <a href="mailto:Intel-gfx@lists.freedesktop.org" target="_blank">Intel-gfx@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/intel-gfx" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/intel-gfx</a><br>
<br>
--<br>
Rafael<br>
_______________________________________________<br>
Intel-gfx mailing list<br>
<a href="mailto:Intel-gfx@lists.freedesktop.org" target="_blank">Intel-gfx@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/intel-gfx" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/intel-gfx</a><br>
</blockquote></div></div>