Early Transport is possible and in our HW mandatory on eDP Panel
Replay. Add parameter to intel_psr2_config_et_valid to differentiate
validity check for Panel Replay.

Signed-off-by: Jouni Högander <jouni.hogan...@intel.com>
---
 drivers/gpu/drm/i915/display/intel_psr.c | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_psr.c 
b/drivers/gpu/drm/i915/display/intel_psr.c
index 4a4124a92a0d..fd861db96db8 100644
--- a/drivers/gpu/drm/i915/display/intel_psr.c
+++ b/drivers/gpu/drm/i915/display/intel_psr.c
@@ -674,16 +674,17 @@ static void hsw_psr_setup_aux(struct intel_dp *intel_dp)
                       aux_ctl);
 }
 
-static bool psr2_su_region_et_valid(struct intel_dp *intel_dp)
+static bool psr2_su_region_et_valid(struct intel_dp *intel_dp, bool 
panel_replay)
 {
        struct drm_i915_private *i915 = dp_to_i915(intel_dp);
 
-       if (DISPLAY_VER(i915) >= 20 &&
-           intel_dp->psr_dpcd[0] == DP_PSR2_WITH_Y_COORD_ET_SUPPORTED &&
-           !(intel_dp->psr.debug & I915_PSR_DEBUG_SU_REGION_ET_DISABLE))
-               return true;
+       if (DISPLAY_VER(i915) < 20 || !intel_dp_is_edp(intel_dp) ||
+           intel_dp->psr.debug & I915_PSR_DEBUG_SU_REGION_ET_DISABLE)
+               return false;
 
-       return false;
+       return panel_replay ?
+               intel_dp->pr_dpcd & DP_PANEL_REPLAY_EARLY_TRANSPORT_SUPPORT :
+               intel_dp->psr_dpcd[0] != DP_PSR2_WITH_Y_COORD_ET_SUPPORTED;
 }
 
 static unsigned int intel_psr_get_enable_sink_offset(struct intel_dp *intel_dp)
@@ -1355,7 +1356,7 @@ static bool intel_psr2_config_valid(struct intel_dp 
*intel_dp,
 
        tgl_dc3co_exitline_compute_config(intel_dp, crtc_state);
 
-       if (psr2_su_region_et_valid(intel_dp))
+       if (psr2_su_region_et_valid(intel_dp, crtc_state->has_panel_replay))
                crtc_state->enable_psr2_su_region_et = true;
 
        return true;
-- 
2.34.1

Reply via email to