<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - [eDP regression] blank screen on boot"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=74628#c13">Comment # 13</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - [eDP regression] blank screen on boot"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=74628">bug 74628</a>
              from <span class="vcard"><a class="email" href="mailto:patrik.r.jakobsson@gmail.com" title="Patrik Jakobsson <patrik.r.jakobsson@gmail.com>"> <span class="fn">Patrik Jakobsson</span></a>
</span></b>
        <pre>Moving the conversation back here (my bad)...

Chris Wilson wrote:

I'd feel more comfortable with;

diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
index 46d065b..876184f 100644
--- a/drivers/gpu/drm/i915/intel_dp.c
+++ b/drivers/gpu/drm/i915/intel_dp.c
@@ -1872,9 +1872,11 @@ static void intel_disable_dp(struct intel_encoder
*encoder)

        /* Make sure the panel is off before trying to change the mode. But
also
         * ensure that we have vdd while we switch off the panel. */
+       edp_panel_vdd_on(intel_dp);
        intel_edp_backlight_off(intel_dp);
        intel_dp_sink_dpms(intel_dp, DRM_MODE_DPMS_OFF);
        intel_edp_panel_off(intel_dp);
+       edp_panel_vdd_off(intel_dp, true);

        /* cpu edp my only be disable _after_ the cpu pipe/plane is disabled.
*/
        if (!(port == PORT_A || IS_VALLEYVIEW(dev)))

Or that vdd_off may have to be post-disable.</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the QA Contact for the bug.</li>
          <li>You are on the CC list for the bug.</li>
      </ul>
    </body>
</html>