On Mon, 17 Oct 2022, Ankit Nautiyal <ankit.k.nautiyal@xxxxxxxxx> 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 <ankit.k.nautiyal@xxxxxxxxx> Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx> > --- > 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