Re: [PATCH 4/6] drm/i915: add i915_pc8_status debugfs file

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



I haven't reviewed the patches that add pc8 itself, but I can assume
it is correct since it was already rv-b Chris.

So:

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>

On Mon, Aug 19, 2013 at 1:18 PM, Paulo Zanoni <przanoni@xxxxxxxxx> wrote:
> From: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>
>
> Make it print the value of the variables on the PC8 struct.
>
> v2: Update to recent renames and add the new fields.
>
> Signed-off-by: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>
> ---
>  drivers/gpu/drm/i915/i915_debugfs.c | 25 +++++++++++++++++++++++++
>  1 file changed, 25 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> index 4785d8c..4d836f1 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -1769,6 +1769,30 @@ static int i915_edp_psr_status(struct seq_file *m, void *data)
>         return 0;
>  }
>
> +static int i915_pc8_status(struct seq_file *m, void *unused)
> +{
> +       struct drm_info_node *node = (struct drm_info_node *) m->private;
> +       struct drm_device *dev = node->minor->dev;
> +       struct drm_i915_private *dev_priv = dev->dev_private;
> +
> +       if (!IS_HASWELL(dev)) {
> +               seq_puts(m, "not supported\n");
> +               return 0;
> +       }
> +
> +       mutex_lock(&dev_priv->pc8.lock);
> +       seq_printf(m, "Requirements met: %s\n",
> +                  yesno(dev_priv->pc8.requirements_met));
> +       seq_printf(m, "GPU idle: %s\n", yesno(dev_priv->pc8.gpu_idle));
> +       seq_printf(m, "Disable count: %d\n", dev_priv->pc8.disable_count);
> +       seq_printf(m, "IRQs disabled: %s\n",
> +                  yesno(dev_priv->pc8.irqs_disabled));
> +       seq_printf(m, "Enabled: %s\n", yesno(dev_priv->pc8.enabled));
> +       mutex_unlock(&dev_priv->pc8.lock);
> +
> +       return 0;
> +}
> +
>  static int
>  i915_wedged_get(void *data, u64 *val)
>  {
> @@ -2208,6 +2232,7 @@ static struct drm_info_list i915_debugfs_list[] = {
>         {"i915_dpio", i915_dpio_info, 0},
>         {"i915_llc", i915_llc, 0},
>         {"i915_edp_psr_status", i915_edp_psr_status, 0},
> +       {"i915_pc8_status", i915_pc8_status, 0},
>  };
>  #define I915_DEBUGFS_ENTRIES ARRAY_SIZE(i915_debugfs_list)
>
> --
> 1.8.1.2
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 
Rodrigo Vivi
Blog: http://blog.vivi.eng.br
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux