<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>