Hi Maarten, Thank you for the patch.
On Thursday 16 Feb 2017 15:47:08 Maarten Lankhorst wrote: > After atomic commit, these macros should be used in place of > get_existing_state. Also after commit get_xx_state should no longer > be used because it may not have the required locks. > > The calls to drm_atomic_get_existing_$obj_state should no longer be > used, and converted over to these new calls. > > Changes since v1: > - Expand commit message. > - Deprecate get_existing_*_state functions in the documentation. > > Signed-off-by: Maarten Lankhorst <maarten.lankho...@linux.intel.com> Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com> > --- > include/drm/drm_atomic.h | 108 +++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 108 insertions(+) > > diff --git a/include/drm/drm_atomic.h b/include/drm/drm_atomic.h > index c6f355a970d2..0147a047878d 100644 > --- a/include/drm/drm_atomic.h > +++ b/include/drm/drm_atomic.h > @@ -277,6 +277,9 @@ int drm_atomic_connector_set_property(struct > drm_connector *connector, * > * This function returns the crtc state for the given crtc, or NULL > * if the crtc is not part of the global atomic state. > + * > + * This function is deprecated, @drm_atomic_get_old_crtc_state or > + * @drm_atomic_get_new_crtc_state should be used instead. > */ > static inline struct drm_crtc_state * > drm_atomic_get_existing_crtc_state(struct drm_atomic_state *state, > @@ -286,12 +289,44 @@ drm_atomic_get_existing_crtc_state(struct > drm_atomic_state *state, } > > /** > + * drm_atomic_get_old_crtc_state - get old crtc state, if it exists > + * @state: global atomic state object > + * @crtc: crtc to grab > + * > + * This function returns the old crtc state for the given crtc, or > + * NULL if the crtc is not part of the global atomic state. > + */ > +static inline struct drm_crtc_state * > +drm_atomic_get_old_crtc_state(struct drm_atomic_state *state, > + struct drm_crtc *crtc) > +{ > + return state->crtcs[drm_crtc_index(crtc)].old_state; > +} > +/** > + * drm_atomic_get_new_crtc_state - get new crtc state, if it exists > + * @state: global atomic state object > + * @crtc: crtc to grab > + * > + * This function returns the new crtc state for the given crtc, or > + * NULL if the crtc is not part of the global atomic state. > + */ > +static inline struct drm_crtc_state * > +drm_atomic_get_new_crtc_state(struct drm_atomic_state *state, > + struct drm_crtc *crtc) > +{ > + return state->crtcs[drm_crtc_index(crtc)].new_state; > +} > + > +/** > * drm_atomic_get_existing_plane_state - get plane state, if it exists > * @state: global atomic state object > * @plane: plane to grab > * > * This function returns the plane state for the given plane, or NULL > * if the plane is not part of the global atomic state. > + * > + * This function is deprecated, @drm_atomic_get_old_plane_state or > + * @drm_atomic_get_new_plane_state should be used instead. > */ > static inline struct drm_plane_state * > drm_atomic_get_existing_plane_state(struct drm_atomic_state *state, > @@ -301,12 +336,45 @@ drm_atomic_get_existing_plane_state(struct > drm_atomic_state *state, } > > /** > + * drm_atomic_get_old_plane_state - get plane state, if it exists > + * @state: global atomic state object > + * @plane: plane to grab > + * > + * This function returns the old plane state for the given plane, or > + * NULL if the plane is not part of the global atomic state. > + */ > +static inline struct drm_plane_state * > +drm_atomic_get_old_plane_state(struct drm_atomic_state *state, > + struct drm_plane *plane) > +{ > + return state->planes[drm_plane_index(plane)].old_state; > +} > + > +/** > + * drm_atomic_get_new_plane_state - get plane state, if it exists > + * @state: global atomic state object > + * @plane: plane to grab > + * > + * This function returns the new plane state for the given plane, or > + * NULL if the plane is not part of the global atomic state. > + */ > +static inline struct drm_plane_state * > +drm_atomic_get_new_plane_state(struct drm_atomic_state *state, > + struct drm_plane *plane) > +{ > + return state->planes[drm_plane_index(plane)].new_state; > +} > + > +/** > * drm_atomic_get_existing_connector_state - get connector state, if it > exists * @state: global atomic state object > * @connector: connector to grab > * > * This function returns the connector state for the given connector, > * or NULL if the connector is not part of the global atomic state. > + * > + * This function is deprecated, @drm_atomic_get_old_connector_state or > + * @drm_atomic_get_new_connector_state should be used instead. > */ > static inline struct drm_connector_state * > drm_atomic_get_existing_connector_state(struct drm_atomic_state *state, > @@ -321,6 +389,46 @@ drm_atomic_get_existing_connector_state(struct > drm_atomic_state *state, } > > /** > + * drm_atomic_get_old_connector_state - get connector state, if it exists > + * @state: global atomic state object > + * @connector: connector to grab > + * > + * This function returns the old connector state for the given connector, > + * or NULL if the connector is not part of the global atomic state. > + */ > +static inline struct drm_connector_state * > +drm_atomic_get_old_connector_state(struct drm_atomic_state *state, > + struct drm_connector *connector) > +{ > + int index = drm_connector_index(connector); > + > + if (index >= state->num_connector) > + return NULL; > + > + return state->connectors[index].old_state; > +} > + > +/** > + * drm_atomic_get_new_connector_state - get connector state, if it exists > + * @state: global atomic state object > + * @connector: connector to grab > + * > + * This function returns the new connector state for the given connector, > + * or NULL if the connector is not part of the global atomic state. > + */ > +static inline struct drm_connector_state * > +drm_atomic_get_new_connector_state(struct drm_atomic_state *state, > + struct drm_connector *connector) > +{ > + int index = drm_connector_index(connector); > + > + if (index >= state->num_connector) > + return NULL; > + > + return state->connectors[index].new_state; > +} > + > +/** > * __drm_atomic_get_current_plane_state - get current plane state > * @state: global atomic state object > * @plane: plane to grab -- Regards, Laurent Pinchart _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx