[PATCH 40/42] drm/omap: remove last uses of omap_overlay_manager

Laurent Pinchart laurent.pinchart at ideasonboard.com
Mon Mar 7 09:01:41 UTC 2016


Hi Tomi,

Thank you for the patch.

On Monday 22 February 2016 19:10:46 Tomi Valkeinen wrote:
> We have now removed all uses of 'struct omap_overlay_manager', so we can
> now remove the last places where it is set.

Shouldn't you also move the definition of the structure to a header private to 
omapfb, as it's not used by omapdrm anymore ?

> Signed-off-by: Tomi Valkeinen <tomi.valkeinen at ti.com>
> ---
>  drivers/gpu/drm/omapdrm/omap_crtc.c | 10 ----------
>  1 file changed, 10 deletions(-)
> 
> diff --git a/drivers/gpu/drm/omapdrm/omap_crtc.c
> b/drivers/gpu/drm/omapdrm/omap_crtc.c index f195641b510c..932196e30024
> 100644
> --- a/drivers/gpu/drm/omapdrm/omap_crtc.c
> +++ b/drivers/gpu/drm/omapdrm/omap_crtc.c
> @@ -98,8 +98,6 @@ static struct omap_dss_device *omap_crtc_output[8];
>  static int omap_crtc_dss_connect(enum omap_channel channel,
>  		struct omap_dss_device *dst)
>  {
> -	struct omap_overlay_manager *mgr = omap_dss_get_overlay_manager(channel);
> -
>  	if (omap_crtc_output[channel])
>  		return -EINVAL;
> 
> @@ -109,22 +107,14 @@ static int omap_crtc_dss_connect(enum omap_channel
> channel, omap_crtc_output[channel] = dst;
>  	dst->dispc_channel_connected = true;
> 
> -	dst->manager = mgr;
> -	mgr->output = dst;
> -
>  	return 0;
>  }
> 
>  static void omap_crtc_dss_disconnect(enum omap_channel channel,
>  		struct omap_dss_device *dst)
>  {
> -	struct omap_overlay_manager *mgr = omap_dss_get_overlay_manager(channel);
> -
>  	omap_crtc_output[channel] = NULL;
>  	dst->dispc_channel_connected = false;
> -
> -	mgr->output->manager = NULL;
> -	mgr->output = NULL;
>  }
> 
>  static void omap_crtc_dss_start_update(enum omap_channel channel)

-- 
Regards,

Laurent Pinchart



More information about the dri-devel mailing list