On Mon, 17 Oct 2022, Ankit Nautiyal <[email protected]> wrote:
> Since the DP/HDMI connector do not set connector->doublescan_allowed,
> the doublescan modes will get automatically filtered during
> drm_helper_probe_single_connector_modes().
>
> Therefore check for double scan modes is not required and is dropped
> from modevalid functions for both DP and HDMI.
>
> Signed-off-by: Ankit Nautiyal <[email protected]>

Reviewed-by: Jani Nikula <[email protected]>

> ---
>  drivers/gpu/drm/i915/display/intel_dp.c   | 3 ---
>  drivers/gpu/drm/i915/display/intel_hdmi.c | 3 ---
>  2 files changed, 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c 
> b/drivers/gpu/drm/i915/display/intel_dp.c
> index a060903891b2..1f83ddf13928 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -973,9 +973,6 @@ intel_dp_mode_valid(struct drm_connector *_connector,
>       enum drm_mode_status status;
>       bool dsc = false, bigjoiner = false;
>  
> -     if (mode->flags & DRM_MODE_FLAG_DBLSCAN)
> -             return MODE_NO_DBLESCAN;
> -
>       if (mode->flags & DRM_MODE_FLAG_DBLCLK)
>               return MODE_H_ILLEGAL;
>  
> diff --git a/drivers/gpu/drm/i915/display/intel_hdmi.c 
> b/drivers/gpu/drm/i915/display/intel_hdmi.c
> index 93519fb23d9d..e21deb6ef7fa 100644
> --- a/drivers/gpu/drm/i915/display/intel_hdmi.c
> +++ b/drivers/gpu/drm/i915/display/intel_hdmi.c
> @@ -1987,9 +1987,6 @@ intel_hdmi_mode_valid(struct drm_connector *connector,
>       bool has_hdmi_sink = intel_has_hdmi_sink(hdmi, connector->state);
>       bool ycbcr_420_only;
>  
> -     if (mode->flags & DRM_MODE_FLAG_DBLSCAN)
> -             return MODE_NO_DBLESCAN;
> -
>       if ((mode->flags & DRM_MODE_FLAG_3D_MASK) == 
> DRM_MODE_FLAG_3D_FRAME_PACKING)
>               clock *= 2;

-- 
Jani Nikula, Intel Open Source Graphics Center

Reply via email to