[PATCH] drm: fixes when i2c encoder slave mode_fixup is null.

Daniel Vetter daniel at ffwll.ch
Wed Feb 10 12:25:14 UTC 2016


On Wed, Feb 10, 2016 at 12:08:38PM +0000, Carlos Palminha wrote:
> Avoid i2c slave encoder drivers to copy/paste code to implement functions that will
> only return true.
> 
> Signed-off-by: Carlos Palminha <palminha at synopsys.com>

lgtm, applied to drm-misc. sil164_drv.c seems to have a dummy mode_fixup
function that's now uneeded. Can you pls follow up with a patch to remove
that one?

Thanks, Daniel

> ---
>  drivers/gpu/drm/drm_encoder_slave.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/gpu/drm/drm_encoder_slave.c b/drivers/gpu/drm/drm_encoder_slave.c
> index e862907..418b94c 100644
> --- a/drivers/gpu/drm/drm_encoder_slave.c
> +++ b/drivers/gpu/drm/drm_encoder_slave.c
> @@ -140,6 +140,8 @@ bool drm_i2c_encoder_mode_fixup(struct drm_encoder *encoder,
>  		const struct drm_display_mode *mode,
>  		struct drm_display_mode *adjusted_mode)
>  {
> +	if(!get_slave_funcs(encoder)->mode_fixup)
> +		return true;
>  	return get_slave_funcs(encoder)->mode_fixup(encoder, mode, adjusted_mode);
>  }
>  EXPORT_SYMBOL(drm_i2c_encoder_mode_fixup);
> --
> 2.5.0
> 

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


More information about the dri-devel mailing list