drm/i915: Fix compiler warnings for i915_edp_psr_status()
authorChris Wilson <chris@chris-wilson.co.uk>
Mon, 16 Jan 2017 13:06:21 +0000 (13:06 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Wed, 18 Jan 2017 10:24:10 +0000 (10:24 +0000)
Appease both the poor reader and the compiler by rewriting the string
lookup for EDP_PSR2_STATUS_CTL:

drivers/gpu/drm/i915/i915_debugfs.c:2662 i915_edp_psr_status() warn: if statement not indented

Fixes: 6ba1f9e1772f ("drm/i915/psr: report live PSR2 State")
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: Jim Bride <jim.bride@linux.intel.com>
Cc: Vathsala Nagaraju <vathsala.nagaraju@intel.com>
Cc: Patil Deepti <deepti.patil@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170116130622.20369-1-chris@chris-wilson.co.uk
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
drivers/gpu/drm/i915/i915_debugfs.c

index 5c7cd9d3d27d08dd056cb3b622de4a1c42d0755a..129aab71fadf8e1d12b11c33b80f4873b4efcc83 100644 (file)
@@ -2559,6 +2559,29 @@ DEFINE_SIMPLE_ATTRIBUTE(i915_guc_log_control_fops,
                        i915_guc_log_control_get, i915_guc_log_control_set,
                        "%lld\n");
 
+static const char *psr2_live_status(u32 val)
+{
+       static const char * const live_status[] = {
+               "IDLE",
+               "CAPTURE",
+               "CAPTURE_FS",
+               "SLEEP",
+               "BUFON_FW",
+               "ML_UP",
+               "SU_STANDBY",
+               "FAST_SLEEP",
+               "DEEP_SLEEP",
+               "BUF_ON",
+               "TG_ON"
+       };
+
+       val = (val & EDP_PSR2_STATUS_STATE_MASK) >> EDP_PSR2_STATUS_STATE_SHIFT;
+       if (val < ARRAY_SIZE(live_status))
+               return live_status[val];
+
+       return "unknown";
+}
+
 static int i915_edp_psr_status(struct seq_file *m, void *data)
 {
        struct drm_i915_private *dev_priv = node_to_i915(m->private);
@@ -2634,28 +2657,10 @@ static int i915_edp_psr_status(struct seq_file *m, void *data)
                seq_printf(m, "Performance_Counter: %u\n", psrperf);
        }
        if (dev_priv->psr.psr2_support) {
-               static const char * const live_status[] = {
-                                                       "IDLE",
-                                                       "CAPTURE",
-                                                       "CAPTURE_FS",
-                                                       "SLEEP",
-                                                       "BUFON_FW",
-                                                       "ML_UP",
-                                                       "SU_STANDBY",
-                                                       "FAST_SLEEP",
-                                                       "DEEP_SLEEP",
-                                                       "BUF_ON",
-                                                       "TG_ON" };
-               u8 pos = (I915_READ(EDP_PSR2_STATUS_CTL) &
-                       EDP_PSR2_STATUS_STATE_MASK) >>
-                       EDP_PSR2_STATUS_STATE_SHIFT;
-
-               seq_printf(m, "PSR2_STATUS_EDP: %x\n",
-                       I915_READ(EDP_PSR2_STATUS_CTL));
-
-               if (pos < ARRAY_SIZE(live_status))
-               seq_printf(m, "PSR2 live state %s\n",
-                       live_status[pos]);
+               u32 psr2 = I915_READ(EDP_PSR2_STATUS_CTL);
+
+               seq_printf(m, "EDP_PSR2_STATUS_CTL: %x [%s]\n",
+                          psr2, psr2_live_status(psr2));
        }
        mutex_unlock(&dev_priv->psr.lock);