drm/i915/dp: Use always vsc revision 0x6 for Panel Replay

We are about to enable Panel Replay Selective update mode. Vsc revision 0x6
for Panel Replay no matter if it is selective update or full frame update
mode. Take this into account when preparing VSC SDP package.

Signed-off-by: Jouni Högander <jouni.hogander@intel.com>
Reviewed-by: Animesh Manna <animesh.manna@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240510093823.3146455-4-jouni.hogander@intel.com

diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
index a6ddeadb61db39b2e67757bce114b92423d41bbd..0984542626098bae19ca8a057798a60b89390a8d 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_dp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp.c
@@ -2663,14 +2663,6 @@ static void intel_dp_compute_vsc_sdp(struct intel_dp *intel_dp,
        if (intel_dp_needs_vsc_sdp(crtc_state, conn_state)) {
                intel_dp_compute_vsc_colorimetry(crtc_state, conn_state,
                                                 vsc);
-       } else if (crtc_state->has_sel_update) {
-               /*
-                * [PSR2 without colorimetry]
-                * Prepare VSC Header for SU as per eDP 1.4 spec, Table 6-11
-                * 3D stereo + PSR/PSR2 + Y-coordinate.
-                */
-               vsc->revision = 0x4;
-               vsc->length = 0xe;
        } else if (crtc_state->has_panel_replay) {
                /*
                 * [Panel Replay without colorimetry info]
@@ -2679,6 +2671,14 @@ static void intel_dp_compute_vsc_sdp(struct intel_dp *intel_dp,
                 */
                vsc->revision = 0x6;
                vsc->length = 0x10;
+       } else if (crtc_state->has_sel_update) {
+               /*
+                * [PSR2 without colorimetry]
+                * Prepare VSC Header for SU as per eDP 1.4 spec, Table 6-11
+                * 3D stereo + PSR/PSR2 + Y-coordinate.
+                */
+               vsc->revision = 0x4;
+               vsc->length = 0xe;
        } else {
                /*
                 * [PSR1]