drm/i915: Skip dsc readout if the transcoder is disabled
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Mon, 24 Jan 2022 19:26:34 +0000 (21:26 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Wed, 26 Jan 2022 11:00:58 +0000 (13:00 +0200)
Trying to do readout when we don't even have a cpu transcoder
is not a great idea. Don't do it.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220124192638.26262-1-ville.syrjala@linux.intel.com
Reviewed-by: Jani Nikula <jani.nikula@intel.com>
drivers/gpu/drm/i915/display/intel_display.c

index 2bf0c3c..a10f693 100644 (file)
@@ -4379,13 +4379,13 @@ static bool hsw_get_pipe_config(struct intel_crtc *crtc,
                active = true;
        }
 
+       if (!active)
+               goto out;
+
        intel_dsc_get_config(pipe_config);
        if (DISPLAY_VER(dev_priv) >= 13 && !pipe_config->dsc.compression_enable)
                intel_uncompressed_joiner_get_config(pipe_config);
 
-       if (!active)
-               goto out;
-
        if (!transcoder_is_dsi(pipe_config->cpu_transcoder) ||
            DISPLAY_VER(dev_priv) >= 11)
                intel_get_transcoder_timings(crtc, pipe_config);