Re: [PATCH 07/16] drm/omapdrm: removed optional dummy crtc mode_fixup function.

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

Hi Carlos,

Thank you for the patch.

On Tuesday 16 February 2016 14:18:12 Carlos Palminha wrote:
> This patch set nukes all the dummy crtc mode_fixup implementations.
> (made on top of Daniel topic/drm-misc branch)
> 
> Signed-off-by: Carlos Palminha <palminha@synopsys.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
>  drivers/gpu/drm/omapdrm/omap_crtc.c | 8 --------
>  1 file changed, 8 deletions(-)
> 
> diff --git a/drivers/gpu/drm/omapdrm/omap_crtc.c
> b/drivers/gpu/drm/omapdrm/omap_crtc.c index d38fcbc..483acdb 100644
> --- a/drivers/gpu/drm/omapdrm/omap_crtc.c
> +++ b/drivers/gpu/drm/omapdrm/omap_crtc.c
> @@ -330,13 +330,6 @@ static void omap_crtc_destroy(struct drm_crtc *crtc)
>  	kfree(omap_crtc);
>  }
> 
> -static bool omap_crtc_mode_fixup(struct drm_crtc *crtc,
> -		const struct drm_display_mode *mode,
> -		struct drm_display_mode *adjusted_mode)
> -{
> -	return true;
> -}
> -
>  static void omap_crtc_enable(struct drm_crtc *crtc)
>  {
>  	struct omap_crtc *omap_crtc = to_omap_crtc(crtc);
> @@ -451,7 +444,6 @@ static const struct drm_crtc_funcs omap_crtc_funcs = {
>  };
> 
>  static const struct drm_crtc_helper_funcs omap_crtc_helper_funcs = {
> -	.mode_fixup = omap_crtc_mode_fixup,
>  	.mode_set_nofb = omap_crtc_mode_set_nofb,
>  	.disable = omap_crtc_disable,
>  	.enable = omap_crtc_enable,

-- 
Regards,

Laurent Pinchart