On Wed, Mar 18, 2015 at 04:45:08PM +0530, Deepak S wrote:
> >+    if (val != dev_priv->rps.cur_freq) {
> >             vlv_punit_write(dev_priv, PUNIT_REG_GPU_FREQ_REQ, val);
> >+            gen6_set_rps_thresholds(dev_priv, val);
> 
> I think gen6_set_rps_thresholds should be under baytrail specific with 
> platform check?

The only difference for cherryview is that it doesn't use
GEN6_RP_MEDIA_TURBO. Was that intentional?

The whole idea is that RPS should be autotuning for different workloads,
but those metrics are equivalent across GPU.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to