[PATCH 5/8] drm: vmwgfx: Remove unneeded CRTC .prepare() helper operation

Thomas Hellstrom thomas at shipmail.org
Thu Jun 29 12:56:53 UTC 2017


Reviewed-by: Thomas Hellstrom <thellstrom at vmware.com>

On 06/27/2017 11:16 PM, Laurent Pinchart wrote:
> The CRTC .prepare() helper operation is part of the legacy helpers and
> is deprecated in favour of the .disable() helper operation. As the
> vmwgfx driver provides a .disable() helper operation, and as the
> .prepare() helper operation implementation is empty, we can remove it.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas at ideasonboard.com>
> ---
>   drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c | 13 -------------
>   1 file changed, 13 deletions(-)
>
> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c b/drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c
> index 449ed4fba0f2..639e16703b80 100644
> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c
> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c
> @@ -203,18 +203,6 @@ static void vmw_ldu_crtc_mode_set_nofb(struct drm_crtc *crtc)
>   }
>   
>   /**
> - * vmw_ldu_crtc_helper_prepare - Noop
> - *
> - * @crtc: CRTC associated with the new screen
> - *
> - * Prepares the CRTC for a mode set, but we don't need to do anything here.
> - *
> - */
> -static void vmw_ldu_crtc_helper_prepare(struct drm_crtc *crtc)
> -{
> -}
> -
> -/**
>    * vmw_ldu_crtc_helper_commit - Noop
>    *
>    * @crtc: CRTC associated with the new screen
> @@ -388,7 +376,6 @@ drm_plane_helper_funcs vmw_ldu_primary_plane_helper_funcs = {
>   };
>   
>   static const struct drm_crtc_helper_funcs vmw_ldu_crtc_helper_funcs = {
> -	.prepare = vmw_ldu_crtc_helper_prepare,
>   	.commit = vmw_ldu_crtc_helper_commit,
>   	.disable = vmw_ldu_crtc_helper_disable,
>   	.mode_set_nofb = vmw_ldu_crtc_mode_set_nofb,




More information about the dri-devel mailing list