[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