[Bug 106223] [KBL-R] DP link training failures lead to downgraded link parameters and resolution

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=106223

--- Comment #24 from Ricardo Ribalda <ricardo.ribalda at gmail.com> ---
(In reply to Clinton Taylor from comment #23)
> 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.

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

...

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/intel-gfx-bugs/attachments/20180426/36e0a186/attachment-0001.html>