[Intel-gfx] [PATCH v2 1/3] drm/i915: set min delay to rpe delay (Efficient frequency).

[deepak . s]

From: Deepak S deepa...@intel.com

We use RPe here since it should match the Vmin we were shooting for.
That should give us better perf than if we used the min freq available.
System thermal can take the system to lowest possible freq (RPn). We are
making sure, we calmp the freq to min_delay (RPe).

Signed-off-by: Deepak S deepa...@intel.com
---
 drivers/gpu/drm/i915/intel_pm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index ca7b537..7c98694 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu/drm/i915/intel_pm.c
@@ -4157,7 +4157,7 @@ static void valleyview_enable_rps(struct drm_device *dev)
 vlv_gpu_freq(dev_priv, dev_priv-rps.rpe_delay),
 dev_priv-rps.rpe_delay);
 
-   dev_priv-rps.min_delay = valleyview_rps_min_freq(dev_priv);
+   dev_priv-rps.min_delay = dev_priv-rps.rpe_delay;
DRM_DEBUG_DRIVER(min GPU freq: %d MHz (%u)\n,
 vlv_gpu_freq(dev_priv, dev_priv-rps.min_delay),
 dev_priv-rps.min_delay);
-- 
1.8.4.2

___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [Intel-gfx] [PATCH v2 1/3] drm/i915: set min delay to rpe delay (Efficient frequency).

[Chris Wilson]

On Tue, Dec 17, 2013 at 08:35:39PM +0530, deepa...@intel.com wrote:
 From: Deepak S deepa...@intel.com
 
 We use RPe here since it should match the Vmin we were shooting for.
 That should give us better perf than if we used the min freq available.
 System thermal can take the system to lowest possible freq (RPn). We are
 making sure, we calmp the freq to min_delay (RPe).
 
 Signed-off-by: Deepak S deepa...@intel.com

Nak. Read the code, explain how it currently fails.
-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