drm/i915: Remove unwarranted clamping for hsw/bdw

We always start off at an "efficient frequency" and can let the system
autotune from there, eliminating the need to clamp the available range.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Sagar Arun Kamble <sagar.a.kamble@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190418205358.11450-1-chris@chris-wilson.co.uk

diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index 44be676fabd6ad7ab80ba2facaedbcf7dc270b20..87f6fc6d550293e0fd001af72e9b6d56fb5a29ac 100644 (file)
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu/drm/i915/intel_pm.c
@@ -8528,18 +8528,9 @@ void intel_init_gt_powersave(struct drm_i915_private *dev_priv)
                gen6_init_rps_frequencies(dev_priv);
 
        /* Derive initial user preferences/limits from the hardware limits */
-       rps->idle_freq = rps->min_freq;
-       rps->cur_freq = rps->idle_freq;
-
        rps->max_freq_softlimit = rps->max_freq;
        rps->min_freq_softlimit = rps->min_freq;
 
-       if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv))
-               rps->min_freq_softlimit =
-                       max_t(int,
-                             rps->efficient_freq,
-                             intel_freq_opcode(dev_priv, 450));
-
        /* After setting max-softlimit, find the overclock max freq */
        if (IS_GEN(dev_priv, 6) ||
            IS_IVYBRIDGE(dev_priv) || IS_HASWELL(dev_priv)) {
@@ -8556,6 +8547,8 @@ void intel_init_gt_powersave(struct drm_i915_private *dev_priv)
 
        /* Finally allow us to boost to max by default */
        rps->boost_freq = rps->max_freq;
+       rps->idle_freq = rps->min_freq;
+       rps->cur_freq = rps->idle_freq;
 
        mutex_unlock(&dev_priv->pcu_lock);
 }