<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [KBL-R] DP link training failures lead to downgraded link parameters and resolution"
href="https://bugs.freedesktop.org/show_bug.cgi?id=106223#c24">Comment # 24</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [KBL-R] DP link training failures lead to downgraded link parameters and resolution"
href="https://bugs.freedesktop.org/show_bug.cgi?id=106223">bug 106223</a>
from <span class="vcard"><a class="email" href="mailto:ricardo.ribalda@gmail.com" title="Ricardo Ribalda <ricardo.ribalda@gmail.com>"> <span class="fn">Ricardo Ribalda</span></a>
</span></b>
<pre>(In reply to Clinton Taylor from <a href="show_bug.cgi?id=106223#c23">comment #23</a>)
<span class="quote">> The msleep(200); should be before the first clock recovery call. Right after
> the int tries;
>
> I'm trying to give extra time for the LSPCON and micro controller in the
> USB-C dongle to complete its initialization before we try and use it. Clock
> Recovery is a very basic function and should not be causing this much issue.
> Equalization I can see compatibility issue, but not clock recovery.</span >
I believe I placed the msleep where you are saying: This is the code that I
tested:
intel_dp_start_link_train(struct intel_dp *intel_dp)
{
struct intel_connector *intel_connector = intel_dp->attached_connector;
int tries;
msleep(200);
for (tries = 0; tries < 5; tries++) {
if (!intel_dp_link_training_clock_recovery(intel_dp))
continue;
if (intel_dp_link_training_channel_equalization(intel_dp))
break;
}
...</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
<li>You are on the CC list for the bug.</li>
<li>You are the QA Contact for the bug.</li>
</ul>
</body>
</html>