Re: [PATCH 6/8] drm: vmwgfx: Replace CRTC .commit() helper operation with .enable()

[Thomas Hellstrom <thomas@xxxxxxxxxxxx>]

Reviewed-by: Thomas Hellstrom <thellstrom@xxxxxxxxxx>


On 06/27/2017 11:16 PM, Laurent Pinchart wrote:
> The CRTC helper .commit() operation is legacy code, the atomic helpers
> prefer the .enable() operation. Replace the .commit() helper operation
> with .enable() in the driver.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx>
> ---
>   drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c  | 6 +++---
>   drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c | 6 +++---
>   drivers/gpu/drm/vmwgfx/vmwgfx_stdu.c | 4 ++--
>   3 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c b/drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c
> index 639e16703b80..f8acd3a15523 100644
> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c
> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c
> @@ -203,7 +203,7 @@ static void vmw_ldu_crtc_mode_set_nofb(struct drm_crtc *crtc)
>   }
>   
>   /**
> - * vmw_ldu_crtc_helper_commit - Noop
> + * vmw_ldu_crtc_helper_enable - Noop
>    *
>    * @crtc: CRTC associated with the new screen
>    *
> @@ -212,7 +212,7 @@ static void vmw_ldu_crtc_mode_set_nofb(struct drm_crtc *crtc)
>    * but since for LDU the display plane is closely tied to the
>    * CRTC, it makes more sense to do those at plane update time.
>    */
> -static void vmw_ldu_crtc_helper_commit(struct drm_crtc *crtc)
> +static void vmw_ldu_crtc_helper_enable(struct drm_crtc *crtc)
>   {
>   }
>   
> @@ -376,7 +376,7 @@ drm_plane_helper_funcs vmw_ldu_primary_plane_helper_funcs = {
>   };
>   
>   static const struct drm_crtc_helper_funcs vmw_ldu_crtc_helper_funcs = {
> -	.commit = vmw_ldu_crtc_helper_commit,
> +	.enable = vmw_ldu_crtc_helper_enable,
>   	.disable = vmw_ldu_crtc_helper_disable,
>   	.mode_set_nofb = vmw_ldu_crtc_mode_set_nofb,
>   	.atomic_check = vmw_du_crtc_atomic_check,
> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c b/drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c
> index 3b917c9b0c21..1cb826c503bf 100644
> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c
> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c
> @@ -270,13 +270,13 @@ static void vmw_sou_crtc_helper_prepare(struct drm_crtc *crtc)
>   }
>   
>   /**
> - * vmw_sou_crtc_helper_commit - Noop
> + * vmw_sou_crtc_helper_enable - Noop
>    *
>    * @crtc: CRTC associated with the new screen
>    *
>    * This is called after a mode set has been completed.
>    */
> -static void vmw_sou_crtc_helper_commit(struct drm_crtc *crtc)
> +static void vmw_sou_crtc_helper_enable(struct drm_crtc *crtc)
>   {
>   }
>   
> @@ -573,7 +573,7 @@ drm_plane_helper_funcs vmw_sou_primary_plane_helper_funcs = {
>   
>   static const struct drm_crtc_helper_funcs vmw_sou_crtc_helper_funcs = {
>   	.prepare = vmw_sou_crtc_helper_prepare,
> -	.commit = vmw_sou_crtc_helper_commit,
> +	.enable = vmw_sou_crtc_helper_enable,
>   	.disable = vmw_sou_crtc_helper_disable,
>   	.mode_set_nofb = vmw_sou_crtc_mode_set_nofb,
>   	.atomic_check = vmw_du_crtc_atomic_check,
> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_stdu.c b/drivers/gpu/drm/vmwgfx/vmwgfx_stdu.c
> index 6aecba6cd5e2..4eb93b47d6db 100644
> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_stdu.c
> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_stdu.c
> @@ -412,7 +412,7 @@ static void vmw_stdu_crtc_helper_prepare(struct drm_crtc *crtc)
>   }
>   
>   
> -static void vmw_stdu_crtc_helper_commit(struct drm_crtc *crtc)
> +static void vmw_stdu_crtc_helper_enable(struct drm_crtc *crtc)
>   {
>   	struct vmw_private *dev_priv;
>   	struct vmw_screen_target_display_unit *stdu;
> @@ -1415,7 +1415,7 @@ drm_plane_helper_funcs vmw_stdu_primary_plane_helper_funcs = {
>   
>   static const struct drm_crtc_helper_funcs vmw_stdu_crtc_helper_funcs = {
>   	.prepare = vmw_stdu_crtc_helper_prepare,
> -	.commit = vmw_stdu_crtc_helper_commit,
> +	.enable = vmw_stdu_crtc_helper_enable,
>   	.disable = vmw_stdu_crtc_helper_disable,
>   	.mode_set_nofb = vmw_stdu_crtc_mode_set_nofb,
>   	.atomic_check = vmw_du_crtc_atomic_check,


_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel