<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEEDINFO "
   title="NEEDINFO - [IVB] 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#c8">Comment # 8</a>
              on <a class="bz_bug_link 
          bz_status_NEEDINFO "
   title="NEEDINFO - [IVB] 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 Ville Syrjala from <a href="show_bug.cgi?id=97204#c7">comment #7</a>)
<span class="quote">> (In reply to Nick Yamane from <a href="show_bug.cgi?id=97204#c6">comment #6</a>)
> > 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] [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;
> > +

> 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.</span >

Ping?</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>