Add debugfs support which will print source and sink status
per connector basis.

Cc: Jouni Högander <jouni.hogan...@intel.com>
Signed-off-by: Animesh Manna <animesh.ma...@intel.com>
---
 drivers/gpu/drm/i915/display/intel_psr.c | 70 ++++++++++++++++--------
 1 file changed, 48 insertions(+), 22 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_psr.c 
b/drivers/gpu/drm/i915/display/intel_psr.c
index 5cbf08a4c94c..f50f110feb09 100644
--- a/drivers/gpu/drm/i915/display/intel_psr.c
+++ b/drivers/gpu/drm/i915/display/intel_psr.c
@@ -3044,7 +3044,7 @@ psr_source_status(struct intel_dp *intel_dp, struct 
seq_file *m)
                        status = live_status[status_val];
        }
 
-       seq_printf(m, "Source PSR status: %s [0x%08x]\n", status, val);
+       seq_printf(m, "Source PSR/PanelReplay status: %s [0x%08x]\n", status, 
val);
 }
 
 static int intel_psr_status(struct seq_file *m, struct intel_dp *intel_dp)
@@ -3057,18 +3057,23 @@ static int intel_psr_status(struct seq_file *m, struct 
intel_dp *intel_dp)
        bool enabled;
        u32 val;
 
-       seq_printf(m, "Sink support: %s", str_yes_no(psr->sink_support));
-       if (psr->sink_support)
+       seq_printf(m, "Sink support: PSR = %s, Panel Replay = %s",
+                  str_yes_no(psr->sink_support),
+                  str_yes_no(psr->sink_panel_replay_support));
+
+       if (psr->sink_support || psr->sink_panel_replay_support)
                seq_printf(m, " [0x%02x]", intel_dp->psr_dpcd[0]);
        seq_puts(m, "\n");
 
-       if (!psr->sink_support)
+       if (!(psr->sink_support || psr->sink_panel_replay_support))
                return 0;
 
        wakeref = intel_runtime_pm_get(&dev_priv->runtime_pm);
        mutex_lock(&psr->lock);
 
-       if (psr->enabled)
+       if (psr->panel_replay_enabled)
+               status = "Panel Replay Enabled";
+       else if (psr->enabled)
                status = psr->psr2_enabled ? "PSR2 enabled" : "PSR1 enabled";
        else
                status = "disabled";
@@ -3081,14 +3086,17 @@ static int intel_psr_status(struct seq_file *m, struct 
intel_dp *intel_dp)
                goto unlock;
        }
 
-       if (psr->psr2_enabled) {
+       if (psr->panel_replay_enabled) {
+               val = intel_de_read(dev_priv, TRANS_DP2_CTL(cpu_transcoder));
+               enabled = val & TRANS_DP2_PANEL_REPLAY_ENABLE;
+       } else if (psr->psr2_enabled) {
                val = intel_de_read(dev_priv, EDP_PSR2_CTL(cpu_transcoder));
                enabled = val & EDP_PSR2_ENABLE;
        } else {
                val = intel_de_read(dev_priv, psr_ctl_reg(dev_priv, 
cpu_transcoder));
                enabled = val & EDP_PSR_ENABLE;
        }
-       seq_printf(m, "Source PSR ctl: %s [0x%08x]\n",
+       seq_printf(m, "Source PSR/PanelReplay ctl: %s [0x%08x]\n",
                   str_enabled_disabled(enabled), val);
        psr_source_status(intel_dp, m);
        seq_printf(m, "Busy frontbuffer bits: 0x%08x\n",
@@ -3230,6 +3238,7 @@ static int i915_psr_sink_status_show(struct seq_file *m, 
void *data)
 {
        struct intel_connector *connector = m->private;
        struct intel_dp *intel_dp = intel_attached_dp(connector);
+       struct intel_psr *psr = &intel_dp->psr;
        static const char * const sink_status[] = {
                "inactive",
                "transition to active, capture and display",
@@ -3240,27 +3249,47 @@ static int i915_psr_sink_status_show(struct seq_file 
*m, void *data)
                "reserved",
                "sink internal error",
        };
+       static const char * const panel_replay_status[] = {
+               "Sink device frame is locked to the Source device",
+               "Sink device is coasting, using the VTotal target",
+               "Sink device is governing the frame rate (frame rate unlock is 
granted)",
+               "Sink device in the process of re-locking with the Source 
device",
+       };
        const char *str;
        int ret;
-       u8 val;
+       u8 val, temp;
 
-       if (!CAN_PSR(intel_dp)) {
-               seq_puts(m, "PSR Unsupported\n");
+       if (!(CAN_PSR(intel_dp) || CAN_PANEL_REPLAY(intel_dp))) {
+               seq_puts(m, "PSR/Panel-Replay Unsupported\n");
                return -ENODEV;
        }
 
        if (connector->base.status != connector_status_connected)
                return -ENODEV;
 
-       ret = drm_dp_dpcd_readb(&intel_dp->aux, DP_PSR_STATUS, &val);
-       if (ret != 1)
-               return ret < 0 ? ret : -EIO;
+       if (psr->panel_replay_enabled) {
+               ret = drm_dp_dpcd_readb(&intel_dp->aux,
+                                       
DP_SINK_DEVICE_PR_AND_FRAME_LOCK_STATUS, &val);
+               if (ret != 1)
+                       return ret < 0 ? ret : -EIO;
 
-       val &= DP_PSR_SINK_STATE_MASK;
-       if (val < ARRAY_SIZE(sink_status))
-               str = sink_status[val];
-       else
-               str = "unknown";
+               temp = val & DP_SINK_FRAME_LOCKED_MASK;
+               temp >>= DP_SINK_FRAME_LOCKED_SHIFT;
+               if (temp < ARRAY_SIZE(panel_replay_status))
+                       str = panel_replay_status[temp];
+               else
+                       str = "unknown";
+       } else {
+               ret = drm_dp_dpcd_readb(&intel_dp->aux, DP_PSR_STATUS, &val);
+               if (ret != 1)
+                       return ret < 0 ? ret : -EIO;
+
+               val &= DP_PSR_SINK_STATE_MASK;
+               if (val < ARRAY_SIZE(sink_status))
+                       str = sink_status[val];
+               else
+                       str = "unknown";
+       }
 
        seq_printf(m, "Sink PSR status: 0x%x [%s]\n", val, str);
 
@@ -3282,13 +3311,10 @@ void intel_psr_connector_debugfs_add(struct 
intel_connector *connector)
        struct drm_i915_private *i915 = to_i915(connector->base.dev);
        struct dentry *root = connector->base.debugfs_entry;
 
-       if (connector->base.connector_type != DRM_MODE_CONNECTOR_eDP)
-               return;
-
        debugfs_create_file("i915_psr_sink_status", 0444, root,
                            connector, &i915_psr_sink_status_fops);
 
-       if (HAS_PSR(i915))
+       if (HAS_PSR(i915) || HAS_DP20(i915))
                debugfs_create_file("i915_psr_status", 0444, root,
                                    connector, &i915_psr_status_fops);
 }
-- 
2.29.0

Reply via email to