[Intel-gfx] [PATCH 2/3] drm/i915: Boost RPS frequency for CPU stalls

[Jesse Barnes]

On Tue, 1 Oct 2013 14:54:26 -0700
Jesse Barnes <jbarnes at virtuousgeek.org> wrote:

> On Wed, 25 Sep 2013 17:34:56 +0100
> Chris Wilson <chris at chris-wilson.co.uk> wrote:
> 
> > +void gen6_rps_idle(struct drm_i915_private *dev_priv)
> > +{
> > +	mutex_lock(&dev_priv->rps.hw_lock);
> > +	if (dev_priv->info->is_valleyview)
> > +		valleyview_set_rps(dev_priv->dev, dev_priv->rps.min_delay);
> > +	else
> > +		gen6_set_rps(dev_priv->dev, dev_priv->rps.min_delay);
> > +	mutex_unlock(&dev_priv->rps.hw_lock);
> > +}
> 
> Looks pretty good, but I think these should be rpe_delay instead.  Not
> much point in going down to a less efficient frequency...

Oh and you can have my
Reviewed-by: Jesse Barnes <jbarnes at virtuousgeek.org>

with that change (or if you can justify the above).

Jesse