drm/i915: Only skip connector output for disable_display
authorChris Wilson <chris@chris-wilson.co.uk>
Wed, 15 Aug 2018 20:12:07 +0000 (21:12 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Wed, 15 Aug 2018 20:39:23 +0000 (21:39 +0100)
We want to add no connectors, encoders or crtcs if the display is
disabled, but we still need to hook up any existing HW so that we can
power it down.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Imre Deak <imre.deak@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180815201207.2203-1-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/intel_display.c

index 5138a921e0b5ff9971e76bb5b0de941215aafc31..3b41a247943a8bd3b5a239d921ce808d424dc83b 100644 (file)
@@ -14128,6 +14128,9 @@ static void intel_setup_outputs(struct drm_i915_private *dev_priv)
 
        intel_pps_init(dev_priv);
 
+       if (INTEL_INFO(dev_priv)->num_pipes == 0)
+               return;
+
        /*
         * intel_edp_init_connector() depends on this completing first, to
         * prevent the registeration of both eDP and LVDS and the incorrect
@@ -15206,9 +15209,6 @@ int intel_modeset_init(struct drm_device *dev)
 
        intel_init_pm(dev_priv);
 
-       if (INTEL_INFO(dev_priv)->num_pipes == 0)
-               return 0;
-
        /*
         * There may be no VBT; and if the BIOS enabled SSC we can
         * just keep using it to avoid unnecessary flicker.  Whereas if the