[PATCH 16/38] drm/hdlcd: Clean up crtc hooks

Daniel Vetter daniel at ffwll.ch
Thu Jun 2 13:33:07 UTC 2016


On Thu, Jun 02, 2016 at 12:06:39AM +0200, Daniel Vetter wrote:
> Those are all no longer needed for a pure atomic driver.
> 
> Cc: Liviu Dudau <Liviu.Dudau at arm.com>
> Tested-by: Liviu Dudau <Liviu.Dudau at arm.com>
> Acked-by: Liviu Dudau <Liviu.Dudau at arm.com>
> Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch>

Applied this one to drm-misc, since Liviua already acked it. The other
prep patch is already in a -fixes pull request.
-Daniel

> ---
>  drivers/gpu/drm/arm/hdlcd_crtc.c | 19 -------------------
>  1 file changed, 19 deletions(-)
> 
> diff --git a/drivers/gpu/drm/arm/hdlcd_crtc.c b/drivers/gpu/drm/arm/hdlcd_crtc.c
> index fef1b04c2aab..b44f72722764 100644
> --- a/drivers/gpu/drm/arm/hdlcd_crtc.c
> +++ b/drivers/gpu/drm/arm/hdlcd_crtc.c
> @@ -196,30 +196,11 @@ static void hdlcd_crtc_atomic_begin(struct drm_crtc *crtc,
>  	}
>  }
>  
> -static void hdlcd_crtc_atomic_flush(struct drm_crtc *crtc,
> -				    struct drm_crtc_state *state)
> -{
> -}
> -
> -static bool hdlcd_crtc_mode_fixup(struct drm_crtc *crtc,
> -			const struct drm_display_mode *mode,
> -			struct drm_display_mode *adjusted_mode)
> -{
> -	return true;
> -}
> -
>  static const struct drm_crtc_helper_funcs hdlcd_crtc_helper_funcs = {
> -	.mode_fixup	= hdlcd_crtc_mode_fixup,
> -	.mode_set	= drm_helper_crtc_mode_set,
> -	.mode_set_base	= drm_helper_crtc_mode_set_base,
> -	.mode_set_nofb	= hdlcd_crtc_mode_set_nofb,
>  	.enable		= hdlcd_crtc_enable,
>  	.disable	= hdlcd_crtc_disable,
> -	.prepare	= hdlcd_crtc_disable,
> -	.commit		= hdlcd_crtc_enable,
>  	.atomic_check	= hdlcd_crtc_atomic_check,
>  	.atomic_begin	= hdlcd_crtc_atomic_begin,
> -	.atomic_flush	= hdlcd_crtc_atomic_flush,
>  };
>  
>  static int hdlcd_plane_atomic_check(struct drm_plane *plane,
> -- 
> 2.8.1
> 

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


More information about the dri-devel mailing list