diff options
| author | Ankit Nautiyal <ankit.k.nautiyal@intel.com> | 2026-02-02 13:37:15 +0300 |
|---|---|---|
| committer | Ankit Nautiyal <ankit.k.nautiyal@intel.com> | 2026-02-03 11:17:00 +0300 |
| commit | efc85c10f5c211e9dcbb74d228560ef80b29fab6 (patch) | |
| tree | 97f5f707783782ecf0c29903e1511953f5c13720 | |
| parent | c766952016f2a42d0711260f2f56bdec17786cb7 (diff) | |
| download | linux-efc85c10f5c211e9dcbb74d228560ef80b29fab6.tar.xz | |
drm/i915/dp: Early reject bad hdisplay in intel_dp_mode_valid
Move check for bad hdisplay early as it is independent on other checks.
Signed-off-by: Ankit Nautiyal <ankit.k.nautiyal@intel.com>
Reviewed-by: Imre Deak <imre.deak@intel.com>
Link: https://patch.msgid.link/20260202103731.357416-2-ankit.k.nautiyal@intel.com
| -rw-r--r-- | drivers/gpu/drm/i915/display/intel_dp.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index 380d9801c377..fe9f8392ca8f 100644 --- a/drivers/gpu/drm/i915/display/intel_dp.c +++ b/drivers/gpu/drm/i915/display/intel_dp.c @@ -1460,6 +1460,9 @@ intel_dp_mode_valid(struct drm_connector *_connector, if (mode->clock < 10000) return MODE_CLOCK_LOW; + if (intel_dp_hdisplay_bad(display, mode->hdisplay)) + return MODE_H_ILLEGAL; + fixed_mode = intel_panel_fixed_mode(connector, mode); if (intel_dp_is_edp(intel_dp) && fixed_mode) { status = intel_panel_mode_valid(connector, mode); @@ -1483,9 +1486,6 @@ intel_dp_mode_valid(struct drm_connector *_connector, if (target_clock > max_dotclk) return MODE_CLOCK_HIGH; - if (intel_dp_hdisplay_bad(display, mode->hdisplay)) - return MODE_H_ILLEGAL; - max_link_clock = intel_dp_max_link_rate(intel_dp); max_lanes = intel_dp_max_lane_count(intel_dp); |