Gentle reminder to review this patch. Thanks, Vinod P. > -----Original Message----- > From: Vinod Polimera <quic_vpolimer@xxxxxxxxxxx> > Sent: Monday, July 11, 2022 6:27 PM > To: dri-devel@xxxxxxxxxxxxxxxxxxxxx; linux-arm-msm@xxxxxxxxxxxxxxx; > freedreno@xxxxxxxxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx > Cc: Vinod Polimera (QUIC) <quic_vpolimer@xxxxxxxxxxx>; linux- > kernel@xxxxxxxxxxxxxxx; robdclark@xxxxxxxxx; dianders@xxxxxxxxxxxx; > swboyd@xxxxxxxxxxxx; Kalyan Thota (QUIC) <quic_kalyant@xxxxxxxxxxx>; > dmitry.baryshkov@xxxxxxxxxx; Kuogee Hsieh (QUIC) > <quic_khsieh@xxxxxxxxxxx>; Vishnuvardhan Prodduturi (QUIC) > <quic_vproddut@xxxxxxxxxxx>; bjorn.andersson@xxxxxxxxxx; Aravind > Venkateswaran (QUIC) <quic_aravindh@xxxxxxxxxxx>; Abhinav Kumar > (QUIC) <quic_abhinavk@xxxxxxxxxxx>; Sankeerth Billakanti (QUIC) > <quic_sbillaka@xxxxxxxxxxx> > Subject: [PATCH v6 02/10] drm: add helper functions to retrieve old and new > crtc > > Add new helper functions, drm_atomic_get_old_crtc_for_encoder > and drm_atomic_get_new_crtc_for_encoder to retrieve the > corresponding crtc for the encoder. > > Signed-off-by: Sankeerth Billakanti <quic_sbillaka@xxxxxxxxxxx> > Signed-off-by: Vinod Polimera <quic_vpolimer@xxxxxxxxxxx> > --- > drivers/gpu/drm/drm_atomic.c | 60 > ++++++++++++++++++++++++++++++++++++++++++++ > include/drm/drm_atomic.h | 7 ++++++ > 2 files changed, 67 insertions(+) > > diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c > index 58c0283..87fcb55 100644 > --- a/drivers/gpu/drm/drm_atomic.c > +++ b/drivers/gpu/drm/drm_atomic.c > @@ -983,6 +983,66 @@ > drm_atomic_get_new_connector_for_encoder(struct drm_atomic_state > *state, > EXPORT_SYMBOL(drm_atomic_get_new_connector_for_encoder); > > /** > + * drm_atomic_get_old_crtc_for_encoder - Get old crtc for an encoder > + * @state: Atomic state > + * @encoder: The encoder to fetch the crtc state for > + * > + * This function finds and returns the crtc that was connected to @encoder > + * as specified by the @state. > + * > + * Returns: The old crtc connected to @encoder, or NULL if the encoder is > + * not connected. > + */ > +struct drm_crtc * > +drm_atomic_get_old_crtc_for_encoder(struct drm_atomic_state *state, > + struct drm_encoder *encoder) > +{ > + struct drm_connector *connector; > + struct drm_connector_state *conn_state; > + > + connector = drm_atomic_get_old_connector_for_encoder(state, > encoder); > + if (!connector) > + return NULL; > + > + conn_state = drm_atomic_get_old_connector_state(state, > connector); > + if (!conn_state) > + return NULL; > + > + return conn_state->crtc; > +} > +EXPORT_SYMBOL(drm_atomic_get_old_crtc_for_encoder); > + > +/** > + * drm_atomic_get_new_crtc_for_encoder - Get new crtc for an encoder > + * @state: Atomic state > + * @encoder: The encoder to fetch the crtc state for > + * > + * This function finds and returns the crtc that will be connected to > @encoder > + * as specified by the @state. > + * > + * Returns: The new crtc connected to @encoder, or NULL if the encoder is > + * not connected. > + */ > +struct drm_crtc * > +drm_atomic_get_new_crtc_for_encoder(struct drm_atomic_state *state, > + struct drm_encoder *encoder) > +{ > + struct drm_connector *connector; > + struct drm_connector_state *conn_state; > + > + connector = drm_atomic_get_new_connector_for_encoder(state, > encoder); > + if (!connector) > + return NULL; > + > + conn_state = drm_atomic_get_new_connector_state(state, > connector); > + if (!conn_state) > + return NULL; > + > + return conn_state->crtc; > +} > +EXPORT_SYMBOL(drm_atomic_get_new_crtc_for_encoder); > + > +/** > * drm_atomic_get_connector_state - get connector state > * @state: global atomic state object > * @connector: connector to get state object for > diff --git a/include/drm/drm_atomic.h b/include/drm/drm_atomic.h > index 0777725..7001f12 100644 > --- a/include/drm/drm_atomic.h > +++ b/include/drm/drm_atomic.h > @@ -528,6 +528,13 @@ struct drm_connector * > drm_atomic_get_new_connector_for_encoder(struct drm_atomic_state > *state, > struct drm_encoder *encoder); > > +struct drm_crtc * > +drm_atomic_get_old_crtc_for_encoder(struct drm_atomic_state *state, > + struct drm_encoder *encoder); > +struct drm_crtc * > +drm_atomic_get_new_crtc_for_encoder(struct drm_atomic_state *state, > + struct drm_encoder *encoder); > + > /** > * drm_atomic_get_existing_crtc_state - get CRTC state, if it exists > * @state: global atomic state object > -- > 2.7.4