drm/i915: Don't use staged config in intel_mst_pre_enable_dp()
authorAnder Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
Thu, 2 Apr 2015 11:48:01 +0000 (14:48 +0300)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 10 Apr 2015 06:55:57 +0000 (08:55 +0200)
For the conversion to atomic. The pre_enable() hooks are called as part
of the crtc enable sequence, at which point the staged config was
already made effective. Furthermore, the function actually changes
hardware state, so it should anyway deal with current and not staged
config.

Signed-off-by: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_dp_mst.c

index 5329c855acce71f4fab98385900e5c99511be986..adcc5e641347dbbd6ea26bba7d4aef9418f1d3f3 100644 (file)
@@ -150,14 +150,14 @@ static void intel_mst_pre_enable_dp(struct intel_encoder *encoder)
        enum port port = intel_dig_port->port;
        int ret;
        uint32_t temp;
-       struct intel_connector *found = NULL, *intel_connector;
+       struct intel_connector *found = NULL, *connector;
        int slots;
        struct drm_crtc *crtc = encoder->base.crtc;
        struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
 
-       for_each_intel_connector(dev, intel_connector) {
-               if (intel_connector->new_encoder == encoder) {
-                       found = intel_connector;
+       for_each_intel_connector(dev, connector) {
+               if (connector->base.state->best_encoder == &encoder->base) {
+                       found = connector;
                        break;
                }
        }