[Intel-gfx] [PATCH v2 4/7] drm/i915: Agressive downclocking on Baytrail

[Deepak S]

On Wednesday 18 March 2015 04:53 PM, Chris Wilson wrote:
> 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
>
Atleast based on the spec GEN6_RP_MEDIA_TURBO left out of CHV. I am yet to look at the latest Spec.

Also, Most of RP register for CHV falls under Comman well, I hope re-adjusting the rps_threshold will
not causing power issues since we have to wakeup both RENDER/MEDIA to access the register

-Deepak