[Intel-gfx] [PATCH] drm: Use drm_mode_is_420_only() instead of open coding it

Jani Nikula jani.nikula at linux.intel.com
Tue May 4 12:35:37 UTC 2021


On Tue, 04 May 2021, Ville Syrjala <ville.syrjala at linux.intel.com> wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
>
> Replace the open coded drm_mode_is_420_only() with the real thing.
>
> No functional changes.
>
> Cc: Werner Sembach <wse at tuxedocomputers.com>
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>

Reviewed-by: Jani Nikula <jani.nikula at intel.com>

> ---
>  drivers/gpu/drm/drm_modes.c | 13 ++++---------
>  1 file changed, 4 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c
> index 33a93fa24eb1..12fcbb7ce179 100644
> --- a/drivers/gpu/drm/drm_modes.c
> +++ b/drivers/gpu/drm/drm_modes.c
> @@ -1176,16 +1176,11 @@ enum drm_mode_status
>  drm_mode_validate_ycbcr420(const struct drm_display_mode *mode,
>  			   struct drm_connector *connector)
>  {
> -	u8 vic = drm_match_cea_mode(mode);
> -	enum drm_mode_status status = MODE_OK;
> -	struct drm_hdmi_info *hdmi = &connector->display_info.hdmi;
> -
> -	if (test_bit(vic, hdmi->y420_vdb_modes)) {
> -		if (!connector->ycbcr_420_allowed)
> -			status = MODE_NO_420;
> -	}
> +	if (!connector->ycbcr_420_allowed &&
> +	    drm_mode_is_420_only(&connector->display_info, mode))
> +		return MODE_NO_420;
>  
> -	return status;
> +	return MODE_OK;
>  }
>  EXPORT_SYMBOL(drm_mode_validate_ycbcr420);

-- 
Jani Nikula, Intel Open Source Graphics Center


More information about the Intel-gfx mailing list