<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Regression: DP outputs out of sync on dual-DP tiled 5k screen"
href="https://bugs.freedesktop.org/show_bug.cgi?id=110671#c10">Comment # 10</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Regression: DP outputs out of sync on dual-DP tiled 5k screen"
href="https://bugs.freedesktop.org/show_bug.cgi?id=110671">bug 110671</a>
from <span class="vcard"><a class="email" href="mailto:d.budelkov@gmail.com" title="Denys <d.budelkov@gmail.com>"> <span class="fn">Denys</span></a>
</span></b>
<pre>Just added some debug to rc3 and tried to check what happens(in context of
5fc0cbfad4564856ee0f323d3f88a7cff19cc3f1).
So in program_timing_sync() there is preparing of groups of pipes for sync.
And looks like(in my case) pipe_set[j]->plane_state is always true,
and all elements > 0 is removed from the pipe_set in this case,
hence group_size == 1 and dc->hwss.enable_timing_synchronization() newer
called.
Contrary with old check
!pipe_set[j]->stream_res.tg->funcs->is_blanked(pipe_set[j]->stream_res.tg) is
always false and we have our sync.
Maybe it should be !pipe_set[j]->plane_state instead of
pipe_set[j]->plane_state ?
I applied this change to my 5.3.0-rc3 build and so far everything looks ok.
I do not understand the purpose of is_blanked or plane_state, maybe with mst
hubs or other stuff it may be a different story, but in my simple config it
looks like just a typo.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>