On Thu,  7 Nov 2013 15:23:27 +0800
Chon Ming Lee <chon.ming....@intel.com> wrote:

> The max frequency reporting is not correct.  But there is already an existing
> valleyview_rps_max_freq and valleyview_rps_min_freq to get the
> frequency.  Use that for i915_cur_delayinfo.
> 
> Signed-off-by: Chon Ming Lee <chon.ming....@intel.com>
> ---
>  drivers/gpu/drm/i915/i915_debugfs.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c 
> b/drivers/gpu/drm/i915/i915_debugfs.c
> index cae3e9c..5254d95 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -972,11 +972,11 @@ static int i915_cur_delayinfo(struct seq_file *m, void 
> *unused)
>               seq_printf(m, "PUNIT_REG_GPU_FREQ_STS: 0x%08x\n", freq_sts);
>               seq_printf(m, "DDR freq: %d MHz\n", dev_priv->mem_freq);
>  
> -             val = vlv_punit_read(dev_priv, PUNIT_FUSE_BUS1);
> +             val = valleyview_rps_max_freq(dev_priv);
>               seq_printf(m, "max GPU freq: %d MHz\n",
>                          vlv_gpu_freq(dev_priv->mem_freq, val));
>  
> -             val = vlv_punit_read(dev_priv, PUNIT_REG_GPU_LFM);
> +             val = valleyview_rps_min_freq(dev_priv);
>               seq_printf(m, "min GPU freq: %d MHz\n",
>                          vlv_gpu_freq(dev_priv->mem_freq, val));
>  

Ok yeah this is just ancient.  It never got fixed up when the other
bits were added.

Reviewed-by: Jesse Barnes <jbar...@virtuousgeek.org>

-- 
Jesse Barnes, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to