On Mon, Jan 04, 2016 at 12:53:18PM +0100, Maarten Lankhorst wrote:
> It can be useful to iterate over connectors without grabbing
> connection_mutex. It can also be used to see how many connectors
> are on a crtc without iterating over the list.
> 
> Signed-off-by: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>

Merged up to this patch here, thanks.
-Daniel

> ---
>  drivers/gpu/drm/drm_atomic.c | 11 +++++++++++
>  include/drm/drm_crtc.h       |  3 +++
>  2 files changed, 14 insertions(+)
> 
> diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c
> index 6a21e5c378c1..14b321580517 100644
> --- a/drivers/gpu/drm/drm_atomic.c
> +++ b/drivers/gpu/drm/drm_atomic.c
> @@ -1063,10 +1063,21 @@ drm_atomic_set_crtc_for_connector(struct 
> drm_connector_state *conn_state,
>  {
>       struct drm_crtc_state *crtc_state;
>  
> +     if (conn_state->crtc && conn_state->crtc != crtc) {
> +             crtc_state = 
> drm_atomic_get_existing_crtc_state(conn_state->state,
> +                                                             
> conn_state->crtc);
> +
> +             crtc_state->connector_mask &=
> +                     ~(1 << drm_connector_index(conn_state->connector));
> +     }
> +
>       if (crtc) {
>               crtc_state = drm_atomic_get_crtc_state(conn_state->state, crtc);
>               if (IS_ERR(crtc_state))
>                       return PTR_ERR(crtc_state);
> +
> +             crtc_state->connector_mask |=
> +                     1 << drm_connector_index(conn_state->connector);
>       }
>  
>       conn_state->crtc = crtc;
> diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
> index c2f98ba2bb98..dd0db4ceab26 100644
> --- a/include/drm/drm_crtc.h
> +++ b/include/drm/drm_crtc.h
> @@ -301,6 +301,7 @@ struct drm_plane_helper_funcs;
>   * @active_changed: crtc_state->active has been toggled.
>   * @connectors_changed: connectors to this crtc have been updated
>   * @plane_mask: bitmask of (1 << drm_plane_index(plane)) of attached planes
> + * @connector_mask: bitmask of (1 << drm_connector_index(connector)) of 
> attached connectors
>   * @last_vblank_count: for helpers and drivers to capture the vblank of the
>   *   update to ensure framebuffer cleanup isn't done too early
>   * @adjusted_mode: for use by helpers and drivers to compute adjusted mode 
> timings
> @@ -334,6 +335,8 @@ struct drm_crtc_state {
>        */
>       u32 plane_mask;
>  
> +     u32 connector_mask;
> +
>       /* last_vblank_count: for vblank waits before cleanup */
>       u32 last_vblank_count;
>  
> -- 
> 2.1.0
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch

Reply via email to