On Wed, Oct 19, 2016 at 03:55:38PM +0200, Maarten Lankhorst wrote: > This gets rid of a warning that the connectors are used without locking > when doing a nonblocking modeset. > > Signed-off-by: Maarten Lankhorst <[email protected]>
Reviewed-by: Ville Syrjälä <[email protected]> > --- > drivers/gpu/drm/i915/intel_display.c | 24 +++++++++++++++--------- > 1 file changed, 15 insertions(+), 9 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_display.c > b/drivers/gpu/drm/i915/intel_display.c > index ba7f7be3aa4f..54a4b2704179 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -13501,11 +13501,15 @@ static void verify_wm_state(struct drm_crtc *crtc, > } > > static void > -verify_connector_state(struct drm_device *dev, struct drm_crtc *crtc) > +verify_connector_state(struct drm_device *dev, > + struct drm_atomic_state *state, > + struct drm_crtc *crtc) > { > struct drm_connector *connector; > + struct drm_connector_state *old_conn_state; > + int i; > > - drm_for_each_connector(connector, dev) { > + for_each_connector_in_state(state, connector, old_conn_state, i) { > struct drm_encoder *encoder = connector->encoder; > struct drm_connector_state *state = connector->state; > > @@ -13713,15 +13717,16 @@ verify_shared_dpll_state(struct drm_device *dev, > struct drm_crtc *crtc, > > static void > intel_modeset_verify_crtc(struct drm_crtc *crtc, > - struct drm_crtc_state *old_state, > - struct drm_crtc_state *new_state) > + struct drm_atomic_state *state, > + struct drm_crtc_state *old_state, > + struct drm_crtc_state *new_state) > { > if (!needs_modeset(new_state) && > !to_intel_crtc_state(new_state)->update_pipe) > return; > > verify_wm_state(crtc, new_state); > - verify_connector_state(crtc->dev, crtc); > + verify_connector_state(crtc->dev, state, crtc); > verify_crtc_state(crtc, old_state, new_state); > verify_shared_dpll_state(crtc->dev, crtc, old_state, new_state); > } > @@ -13737,10 +13742,11 @@ verify_disabled_dpll_state(struct drm_device *dev) > } > > static void > -intel_modeset_verify_disabled(struct drm_device *dev) > +intel_modeset_verify_disabled(struct drm_device *dev, > + struct drm_atomic_state *state) > { > verify_encoder_state(dev); > - verify_connector_state(dev, NULL); > + verify_connector_state(dev, state, NULL); > verify_disabled_dpll_state(dev); > } > > @@ -14399,7 +14405,7 @@ static void intel_atomic_commit_tail(struct > drm_atomic_state *state) > if (!intel_can_enable_sagv(state)) > intel_disable_sagv(dev_priv); > > - intel_modeset_verify_disabled(dev); > + intel_modeset_verify_disabled(dev, state); > } > > /* Complete the events for pipes that have now been disabled */ > @@ -14451,7 +14457,7 @@ static void intel_atomic_commit_tail(struct > drm_atomic_state *state) > if (put_domains[i]) > modeset_put_power_domains(dev_priv, put_domains[i]); > > - intel_modeset_verify_crtc(crtc, old_crtc_state, crtc->state); > + intel_modeset_verify_crtc(crtc, state, old_crtc_state, > crtc->state); > } > > if (intel_state->modeset && intel_can_enable_sagv(state)) > -- > 2.7.4 > > _______________________________________________ > Intel-gfx mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
