Scanline indication needs to be checked and configure for both PSR2 and
Panel Replay Selective Update. Move this check to
intel_sel_update_config_valid.

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

diff --git a/drivers/gpu/drm/i915/display/intel_psr.c 
b/drivers/gpu/drm/i915/display/intel_psr.c
index ae1728d2a15c..35425567005d 100644
--- a/drivers/gpu/drm/i915/display/intel_psr.c
+++ b/drivers/gpu/drm/i915/display/intel_psr.c
@@ -1426,12 +1426,6 @@ static bool intel_psr2_config_valid(struct intel_dp 
*intel_dp,
                return false;
        }
 
-       if (!_compute_psr2_sdp_prior_scanline_indication(intel_dp, crtc_state)) 
{
-               drm_dbg_kms(&dev_priv->drm,
-                           "PSR2 not enabled, PSR2 SDP indication do not fit 
in hblank\n");
-               return false;
-       }
-
        if (!alpm_config_valid(intel_dp, crtc_state, false))
                return false;
 
@@ -1473,6 +1467,12 @@ static bool intel_sel_update_config_valid(struct 
intel_dp *intel_dp,
        if (!crtc_state->has_panel_replay && !intel_psr2_config_valid(intel_dp, 
crtc_state))
                goto unsupported;
 
+       if (!_compute_psr2_sdp_prior_scanline_indication(intel_dp, crtc_state)) 
{
+               drm_dbg_kms(&dev_priv->drm,
+                           "Selective update not enabled, SDP indication do 
not fit in hblank\n");
+               goto unsupported;
+       }
+
        if (crtc_state->has_panel_replay && (DISPLAY_VER(dev_priv) < 14 ||
                                             
!intel_dp->psr.sink_panel_replay_su_support))
                goto unsupported;
-- 
2.34.1

Reply via email to