<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body><span class="vcard"><a class="email" href="mailto:ville.syrjala@linux.intel.com" title="Ville Syrjala <ville.syrjala@linux.intel.com>"> <span class="fn">Ville Syrjala</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEEDINFO "
title="NEEDINFO - Can't enable a third monitor even with 3 CRTCs available (1 LVDS, 1 VGA, 1 HDMI/VGA)"
href="https://bugs.freedesktop.org/show_bug.cgi?id=97204">bug 97204</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;">Status</td>
<td>NEW
</td>
<td>NEEDINFO
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEEDINFO "
title="NEEDINFO - Can't enable a third monitor even with 3 CRTCs available (1 LVDS, 1 VGA, 1 HDMI/VGA)"
href="https://bugs.freedesktop.org/show_bug.cgi?id=97204#c7">Comment # 7</a>
on <a class="bz_bug_link
bz_status_NEEDINFO "
title="NEEDINFO - Can't enable a third monitor even with 3 CRTCs available (1 LVDS, 1 VGA, 1 HDMI/VGA)"
href="https://bugs.freedesktop.org/show_bug.cgi?id=97204">bug 97204</a>
from <span class="vcard"><a class="email" href="mailto:ville.syrjala@linux.intel.com" title="Ville Syrjala <ville.syrjala@linux.intel.com>"> <span class="fn">Ville Syrjala</span></a>
</span></b>
<pre>(In reply to Nick Yamane from <a href="show_bug.cgi?id=97204#c6">comment #6</a>)
<span class="quote">> Created <span class=""><a href="attachment.cgi?id=125546" name="attach_125546" title="[PATCH] drm/i915: Allow PCH DPLL sharing regardless of DPLL_SDVO_HIGH_SPEED (linux 4.4.16)">attachment 125546</a> <a href="attachment.cgi?id=125546&action=edit" title="[PATCH] drm/i915: Allow PCH DPLL sharing regardless of DPLL_SDVO_HIGH_SPEED (linux 4.4.16)">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=97204&attachment=125546'>[review]</a> [review]
> [PATCH] drm/i915: Allow PCH DPLL sharing regardless of DPLL_SDVO_HIGH_SPEED
> (linux 4.4.16)
>
> It worked. Just modified your patch to be able to apply in my kernel
> (v4.4.14 LTS), using "intel_pipe_has_type" instead of "intel_crtc_has_type".
> Is that correct?
>
> + if (INTEL_INFO(dev_priv)->num_pipes == 3 &&
> + intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_ANALOG))
> + dpll |= DPLL_SDVO_HIGH_SPEED;
> +</span >
Almost correct. intel_pipe_will_have_type(crtc_state, INTEL_OUTPUT_ANALOG)
is what should be used on 4.4. Can you double check with that?
Actually can you also attach a dmesg (w/ drm.debug=0xe) with the patch applied?
I'd like to see that it really enables three pipes. In theory it might just
setup a clone between HDMI and VGA and use just one pipe for both). That may
happen if you don't specify a unique x/y position for each output, and the
outputs are using the same mode.</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>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>