[Intel-gfx] [PATCH] drm/i915: Don't pretend ips is always enabled on BDW.
Paulo Zanoni
przanoni at gmail.com
Mon Jun 30 21:17:00 CEST 2014
2014-06-30 8:45 GMT-03:00 Rodrigo Vivi <rodrigo.vivi at intel.com>:
> As pointed out before we don't have a reliable way to read back ips
> status on BDW without the risk to disable it when reading.
> However now we are pretending that IPS on BDW is always on and getting
> people confused about it.
>
> So this patch allows people to know if ips was ever attempted to be enabled.
> Even if the current status is impossible to be ascertain.
>
> v2: (spotted by Paulo):
> * A version that at least compiles
> * with more clear messages
> * let Cheryview on the safe side until we aren't sure that checking ips
> state on ips won't disable it.
>
> Cc: Paulo Zanoni <paulo.r.zanoni at intel.com>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
Reviewed-by: Paulo Zanoni <paulo.r.zanoni at intel.com>
> ---
> drivers/gpu/drm/i915/i915_debugfs.c | 15 +++++++++++----
> 1 file changed, 11 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> index a93b3bf..380be89 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -1523,10 +1523,17 @@ static int i915_ips_status(struct seq_file *m, void *unused)
>
> intel_runtime_pm_get(dev_priv);
>
> - if (IS_BROADWELL(dev) || I915_READ(IPS_CTL) & IPS_ENABLE)
> - seq_puts(m, "enabled\n");
> - else
> - seq_puts(m, "disabled\n");
> + seq_printf(m, "Enabled by kernel parameter: %s\n",
> + yesno(i915.enable_ips));
> +
> + if (INTEL_INFO(dev)->gen >= 8) {
> + seq_puts(m, "Currently: unknown\n");
> + } else {
> + if (I915_READ(IPS_CTL) & IPS_ENABLE)
> + seq_puts(m, "Currently: enabled\n");
> + else
> + seq_puts(m, "Currently: disabled\n");
> + }
>
> intel_runtime_pm_put(dev_priv);
>
> --
> 1.9.3
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Paulo Zanoni
More information about the Intel-gfx
mailing list