From 651d4a0fc4c5c6a2d20c95ef814f0d4381f71d55 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Ville=20Syrj=C3=A4l=C3=A4?= Date: Wed, 15 Jun 2022 20:48:51 +0300 Subject: [PATCH] drm/i915: Sanitize underrun reporting before turning off any pipes MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Make sure FIFO underrun reporting is flagged as disabled very early during the state readout so that we don't get any spurious FIFO underruns reports from intel_crtc_disable_noatomic(). Signed-off-by: Ville Syrjälä Link: https://patchwork.freedesktop.org/patch/msgid/20220615174851.20658-2-ville.syrjala@linux.intel.com Reviewed-by: Jani Nikula --- drivers/gpu/drm/i915/display/intel_display.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c index 3fac545414b45..88658090ac583 100644 --- a/drivers/gpu/drm/i915/display/intel_display.c +++ b/drivers/gpu/drm/i915/display/intel_display.c @@ -9962,8 +9962,6 @@ static void intel_sanitize_crtc(struct intel_crtc *crtc, if (crtc_state->hw.active && !intel_crtc_has_encoders(crtc) && !intel_crtc_is_bigjoiner_slave(crtc_state)) intel_crtc_disable_noatomic(crtc, ctx); - - intel_sanitize_fifo_underrun_reporting(crtc_state); } static bool has_bogus_dpll_config(const struct intel_crtc_state *crtc_state) @@ -10363,6 +10361,8 @@ intel_modeset_setup_hw_state(struct drm_device *dev, struct intel_crtc_state *crtc_state = to_intel_crtc_state(crtc->base.state); + intel_sanitize_fifo_underrun_reporting(crtc_state); + drm_crtc_vblank_reset(&crtc->base); if (crtc_state->hw.active) -- 2.30.2