drm/i915: vlv: fix RPS interrupt mask setting
authorImre Deak <imre.deak@intel.com>
Thu, 3 Apr 2014 17:02:42 +0000 (20:02 +0300)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 4 Apr 2014 07:30:52 +0000 (09:30 +0200)
This typo may lead to missed RPS interrupts and as a result a too
low or too high frequency for the current workload. The interrupt mask
will be set properly at a subsequent GPU idle event, but can get
corrupted again at the next RPS up/down event.

Signed-off-by: Imre Deak <imre.deak@intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_pm.c

index b20a26584e06246a8f8e41b550cdd27c30386b36..f18dec071df73d55c0993c21b19d0663f4fd0ffa 100644 (file)
@@ -3159,7 +3159,7 @@ void valleyview_set_rps(struct drm_device *dev, u8 val)
        if (val != dev_priv->rps.cur_freq)
                vlv_punit_write(dev_priv, PUNIT_REG_GPU_FREQ_REQ, val);
 
-       I915_WRITE(GEN6_PMINTRMSK, val);
+       I915_WRITE(GEN6_PMINTRMSK, gen6_rps_pm_mask(dev_priv, val));
 
        dev_priv->rps.cur_freq = val;
        trace_intel_gpu_freq_change(vlv_gpu_freq(dev_priv, val));