<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body><span class="vcard"><a class="email" href="mailto:conselvan2@gmail.com" title="Ander Conselvan de Oliveira <conselvan2@gmail.com>"> <span class="fn">Ander Conselvan de Oliveira</span></a>
</span> changed
<a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - system freezes during boot, [drm:intel_dp_start_link_train [i915]] and [drm:intel_psr_work [i915]] ERROR"
href="https://bugs.freedesktop.org/show_bug.cgi?id=96569">bug 96569</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">CC</td>
<td>
</td>
<td>conselvan2@gmail.com
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - system freezes during boot, [drm:intel_dp_start_link_train [i915]] and [drm:intel_psr_work [i915]] ERROR"
href="https://bugs.freedesktop.org/show_bug.cgi?id=96569#c2">Comment # 2</a>
on <a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - system freezes during boot, [drm:intel_dp_start_link_train [i915]] and [drm:intel_psr_work [i915]] ERROR"
href="https://bugs.freedesktop.org/show_bug.cgi?id=96569">bug 96569</a>
from <span class="vcard"><a class="email" href="mailto:conselvan2@gmail.com" title="Ander Conselvan de Oliveira <conselvan2@gmail.com>"> <span class="fn">Ander Conselvan de Oliveira</span></a>
</span></b>
<pre>Created <span class=""><a href="attachment.cgi?id=125056" name="attach_125056" title="Workaround: don't retrain the link from long pulse">attachment 125056</a> <a href="attachment.cgi?id=125056&action=edit" title="Workaround: don't retrain the link from long pulse">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=96569&attachment=125056'>[review]</a>
Workaround: don't retrain the link from long pulse
(In reply to Jani Nikula from <a href="show_bug.cgi?id=96569#c1">comment #1</a>)
<span class="quote">> This is silly.
>
> Based on the logs, I presume this happens before we enable eDP for the first
> time:
>
> -> intel_dp_detect
> -> intel_dp_long_pulse
> -> intel_dp_check_link_status (since it's eDP, connector status remains
> connected)
> -> apparently the crtc is active, but the channel eq is not okay, because
> well, we've never trained before, and then we go on trying to retrain
>
> Ander, any ideas when we caused this to happen?</span >
Probably in the commit below. We had logic to retrain the link from long hpd,
and that series changed intel_dp_detect() to share code with the long pulse
handling. The problem is that the long pulse handling is also called from
output polling during boot and resume from suspend and ends up calling that.
I'm not sure what the proper fix would be, but the attached patch would confirm
that's the issue.
commit 7d23e3c37bb3fc6952dc84007ee60cb533fd2d5c
Author: Shubhangi Shrivastava <<a href="mailto:shubhangi.shrivastava@intel.com">shubhangi.shrivastava@intel.com</a>>
Date: Wed Mar 30 18:05:23 2016 +0530
drm/i915: Cleaning up intel_dp_hpd_pulse</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are on the CC list for the bug.</li>
<li>You are the QA Contact for the bug.</li>
</ul>
</body>
</html>