On Wed, 25 Sep 2013 17:34:56 +0100
Chris Wilson <[email protected]> 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...

Jesse
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to