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@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx