From: Ville Syrjälä <ville.syrj...@linux.intel.com>

Prepare the connector/encoder mask readout for the uapi vs. hw
state split. We'll want to do all readout into the hw state.

Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
---
 drivers/gpu/drm/i915/display/intel_display.c | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_display.c 
b/drivers/gpu/drm/i915/display/intel_display.c
index 8a2e780d85e1..9d4dfe2eace0 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -16795,24 +16795,28 @@ static void intel_modeset_readout_hw_state(struct 
drm_device *dev)
        drm_connector_list_iter_begin(dev, &conn_iter);
        for_each_intel_connector_iter(connector, &conn_iter) {
                if (connector->get_hw_state(connector)) {
+                       struct intel_crtc_state *crtc_state;
+                       struct intel_crtc *crtc;
+
                        connector->base.dpms = DRM_MODE_DPMS_ON;
 
                        encoder = connector->encoder;
                        connector->base.encoder = &encoder->base;
 
-                       if (encoder->base.crtc &&
-                           encoder->base.crtc->state->active) {
+                       crtc = to_intel_crtc(encoder->base.crtc);
+                       crtc_state = crtc ? 
to_intel_crtc_state(crtc->base.state) : NULL;
+
+                       if (crtc_state && crtc_state->base.active) {
                                /*
                                 * This has to be done during hardware readout
                                 * because anything calling .crtc_disable may
                                 * rely on the connector_mask being accurate.
                                 */
-                               encoder->base.crtc->state->connector_mask |=
+                               crtc_state->base.connector_mask |=
                                        drm_connector_mask(&connector->base);
-                               encoder->base.crtc->state->encoder_mask |=
+                               crtc_state->base.encoder_mask |=
                                        drm_encoder_mask(&encoder->base);
                        }
-
                } else {
                        connector->base.dpms = DRM_MODE_DPMS_OFF;
                        connector->base.encoder = NULL;
-- 
2.21.0

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to