[PATCH] drm/rockchip: Rely on the default best_encoder() behavior

Sean Paul seanpaul at chromium.org
Wed Sep 27 20:06:21 UTC 2017


On Wed, Sep 27, 2017 at 12:23:17PM -0600, Haneen Mohammed wrote:
> Since the output has 1:1 relationship between connectors and encoders,
> and the driver is relying on the atomic helpers, remove the custom
> best_encoder() and let the core call drm_atomic_helper_best_encoder().
> 
> Signed-off-by: Haneen Mohammed <hamohammed.sa at gmail.com>

Thanks for the patch, I've applied it to drm-misc-next.

I noticed a few instances of .best_encoder = drm_atomic_helper_best_encoder
hanging around. Any interest in removing those as well?

Sean


> ---
>  drivers/gpu/drm/rockchip/cdn-dp-core.c | 9 ---------
>  1 file changed, 9 deletions(-)
> 
> diff --git a/drivers/gpu/drm/rockchip/cdn-dp-core.c b/drivers/gpu/drm/rockchip/cdn-dp-core.c
> index a57da05..275844d 100644
> --- a/drivers/gpu/drm/rockchip/cdn-dp-core.c
> +++ b/drivers/gpu/drm/rockchip/cdn-dp-core.c
> @@ -287,14 +287,6 @@ static int cdn_dp_connector_get_modes(struct drm_connector *connector)
>  	return ret;
>  }
>  
> -static struct drm_encoder *
> -cdn_dp_connector_best_encoder(struct drm_connector *connector)
> -{
> -	struct cdn_dp_device *dp = connector_to_dp(connector);
> -
> -	return &dp->encoder;
> -}
> -
>  static int cdn_dp_connector_mode_valid(struct drm_connector *connector,
>  				       struct drm_display_mode *mode)
>  {
> @@ -346,7 +338,6 @@ static int cdn_dp_connector_mode_valid(struct drm_connector *connector,
>  
>  static struct drm_connector_helper_funcs cdn_dp_connector_helper_funcs = {
>  	.get_modes = cdn_dp_connector_get_modes,
> -	.best_encoder = cdn_dp_connector_best_encoder,
>  	.mode_valid = cdn_dp_connector_mode_valid,
>  };
>  
> -- 
> 2.7.4
> 

-- 
Sean Paul, Software Engineer, Google / Chromium OS


More information about the dri-devel mailing list