eDP1.5 support ALPM with Panel Replay as well. We need to check ALPM
related things for Panel Replay as well.

Bspec: 68920

Signed-off-by: Jouni Högander <[email protected]>
---
 drivers/gpu/drm/i915/display/intel_psr.c | 45 +++++++++++++-----------
 1 file changed, 24 insertions(+), 21 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_psr.c 
b/drivers/gpu/drm/i915/display/intel_psr.c
index daba46aa1149..4bbb0c05054f 100644
--- a/drivers/gpu/drm/i915/display/intel_psr.c
+++ b/drivers/gpu/drm/i915/display/intel_psr.c
@@ -1590,27 +1590,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 (!_compute_alpm_params(intel_dp, crtc_state)) {
-               drm_dbg_kms(&dev_priv->drm,
-                           "PSR2 not enabled, Unable to use long enough wake 
times\n");
-               return false;
-       }
-
-       /* Vblank >= PSR2_CTL Block Count Number maximum line count */
-       if (crtc_state->hw.adjusted_mode.crtc_vblank_end -
-           crtc_state->hw.adjusted_mode.crtc_vblank_start <
-           psr2_block_count_lines(intel_dp)) {
-               drm_dbg_kms(&dev_priv->drm,
-                           "PSR2 not enabled, too short vblank time\n");
-               return false;
-       }
-
        if (!crtc_state->enable_psr2_sel_fetch &&
            (crtc_hdisplay > psr_max_h || crtc_vdisplay > psr_max_v)) {
                drm_dbg_kms(&dev_priv->drm,
@@ -1649,6 +1628,30 @@ 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 (intel_dp_is_edp(intel_dp)) {
+               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 (!_compute_alpm_params(intel_dp, crtc_state)) {
+                       drm_dbg_kms(&dev_priv->drm,
+                                   "Selective update not enabled, Unable to 
use long enough wake times\n");
+                       goto unsupported;
+               }
+
+               /* Vblank >= PSR2_CTL Block Count Number maximum line count */
+               if (crtc_state->hw.adjusted_mode.crtc_vblank_end -
+                   crtc_state->hw.adjusted_mode.crtc_vblank_start <
+                   psr2_block_count_lines(intel_dp)) {
+                       drm_dbg_kms(&dev_priv->drm,
+                                   "Selective update not enabled, too short 
vblank time\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