<div dir="ltr"><div><br></div>Ok, so after trying it we saw that we really cannot trust on aux mutex.At least not on all SKL/KBL<div>It worked in a KBL but failed on a SKL that I have here...</div><div><br></div><div>So without aux mutex option we still need to get sink_crc more reliable and I see only 2 quick ways here:</div><div>- This read wake</div><div>- Return -EBUSY to force the drm retries on message size = 0.</div><div><br></div><div>Daniel, what do you believe?</div><div><br></div><div>Please let me know witch way and if necessary I rebase the patch and re-send.</div><div><br></div><div>Thanks,</div><div>Rodrigo.<br><div><br></div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Oct 21, 2015 at 8:27 PM Thulasimani, Sivakumar <<a href="mailto:sivakumar.thulasimani@intel.com">sivakumar.thulasimani@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
On 10/22/2015 1:44 AM, Damien Lespiau wrote:<br>
> On Thu, Oct 22, 2015 at 12:01:21AM +0530, Thulasimani, Sivakumar wrote:<br>
>><br>
>> On 8/25/2015 2:50 AM, Vivi, Rodrigo wrote:<br>
>>> On Mon, 2015-08-24 at 19:54 +0000, Zanoni, Paulo R wrote:<br>
>>>> Em Qui, 2015-08-20 às 16:23 -0700, Rodrigo Vivi escreveu:<br>
>>>>> Let's use a native read with retry as suggested per spec to<br>
>>>>> fix Sink CRC on SKL when PSR is enabled.<br>
>>>>><br>
>>>>> With PSR enabled panel is probably taking more time to wake<br>
>>>>> and dpcd read is faling.<br>
>>>> Does this commit actually fix any known problem with Sink CRC? Or is<br>
>>>> it<br>
>>>> just a try? It would be nice to have this clarified in the commit<br>
>>>> message.<br>
>>> It was just a try but that made sink crc working on my SKL when PSR is<br>
>>> enabled. nothing much to add...<br>
>> SKL has new register AUX_MUTEX which should be used when accessing dpcd<br>
>> on edp. just searched the nightly code and could not find it. it might be<br>
>> the reason<br>
>> for random dpcd failures reported in the other thread.<br>
> We had patches for that back in December 2013 :)<br>
><br>
> The feedback from Art was:<br>
><br>
>      The non-software aux users are PSR/SRD and GTC.<br>
>      Better leave out the mutex for now. Hardware is going to try do the<br>
>      arbitration itself. I expect you will then need to increase any software<br>
>      timeout you may have.<br>
><br>
> Do you know if anything has changed since then?<br>
><br>
Not sure, it is in the bspec sequence to use AUX hence forwarded. Art<br>
might be the<br>
right person to contact :). it might be due to some minor DPCD access<br>
issues we<br>
observed in BDW when PSR was enabled.<br>
<br>
regards,<br>
Sivakumar<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>