On Tue, 04 May 2021, Ville Syrjala <ville.syrjala@xxxxxxxxxxxxxxx> wrote:
> From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
>
> Replace the open coded drm_mode_is_420_only() with the real thing.
>
> No functional changes.
>
> Cc: Werner Sembach <wse@xxxxxxxxxxxxxxxxxxx>
> Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx>
> ---
> 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
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
