Commit 844e3313 authored by Chris Wilson's avatar Chris Wilson

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: default avatarChris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: default avatarSagar Arun Kamble <sagar.a.kamble@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190418205358.11450-1-chris@chris-wilson.co.uk
parent 26ddc068
...@@ -8528,18 +8528,9 @@ void intel_init_gt_powersave(struct drm_i915_private *dev_priv) ...@@ -8528,18 +8528,9 @@ void intel_init_gt_powersave(struct drm_i915_private *dev_priv)
gen6_init_rps_frequencies(dev_priv); gen6_init_rps_frequencies(dev_priv);
/* Derive initial user preferences/limits from the hardware limits */ /* 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->max_freq_softlimit = rps->max_freq;
rps->min_freq_softlimit = rps->min_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 */ /* After setting max-softlimit, find the overclock max freq */
if (IS_GEN(dev_priv, 6) || if (IS_GEN(dev_priv, 6) ||
IS_IVYBRIDGE(dev_priv) || IS_HASWELL(dev_priv)) { IS_IVYBRIDGE(dev_priv) || IS_HASWELL(dev_priv)) {
...@@ -8556,6 +8547,8 @@ void intel_init_gt_powersave(struct drm_i915_private *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 */ /* Finally allow us to boost to max by default */
rps->boost_freq = rps->max_freq; rps->boost_freq = rps->max_freq;
rps->idle_freq = rps->min_freq;
rps->cur_freq = rps->idle_freq;
mutex_unlock(&dev_priv->pcu_lock); mutex_unlock(&dev_priv->pcu_lock);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment