drm/i915: Remove pointless register read
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Mon, 13 Feb 2023 22:52:58 +0000 (00:52 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Mon, 20 Feb 2023 21:26:59 +0000 (23:26 +0200)
We just wrote the EDP transcoder's VTOTAL register a few lines
earlier, so instead of reading it back out again let's just
generate the same value for the transocder B/C register.

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

index 463fa4835d12f42858d07bccb7cb6f89016f9fe2..e882d68fdca96185c7500d6be98a98f9cb6d3cc4 100644 (file)
@@ -2794,8 +2794,8 @@ static void intel_set_transcoder_timings(const struct intel_crtc_state *crtc_sta
        if (IS_HASWELL(dev_priv) && cpu_transcoder == TRANSCODER_EDP &&
            (pipe == PIPE_B || pipe == PIPE_C))
                intel_de_write(dev_priv, TRANS_VTOTAL(pipe),
-                              intel_de_read(dev_priv, TRANS_VTOTAL(cpu_transcoder)));
-
+                              VACTIVE(crtc_vdisplay - 1) |
+                              VTOTAL(crtc_vtotal - 1));
 }
 
 static void intel_set_pipe_src_size(const struct intel_crtc_state *crtc_state)