drm/i915: kill i915.powersave

This flag was being mostly used as a meta flag in some
cases and not covering other cases.

One of the risks is that it was masking some frontbuffer
trackings without disabling PSR.

So, better to kill this at once and avoid umbrella parameters.

Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Acked-by: Chris Wilson <chris@chris-wilson.co.uk>
[danvet: Drop unused out: label to appease gcc.]
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

1 files changed, 2 insertions, 6 deletions

diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
index f9dc5babcf27..d98df2739666 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -6024,7 +6024,7 @@ static void i9xx_update_pll_dividers(struct intel_crtc *crtc,
 
         crtc->lowfreq_avail = false;
         if (intel_pipe_will_have_type(crtc, INTEL_OUTPUT_LVDS) &&
-            reduced_clock && i915.powersave) {
+            reduced_clock) {
                 crtc_state->dpll_hw_state.fp1 = fp2;
                 crtc->lowfreq_avail = true;
         } else {
@@ -7807,7 +7807,7 @@ static int ironlake_crtc_compute_clock(struct intel_crtc *crtc,
                 }
         }
 
-        if (is_lvds && has_reduced_clock && i915.powersave)
+        if (is_lvds && has_reduced_clock)
                 crtc->lowfreq_avail = true;
         else
                 crtc->lowfreq_avail = false;
@@ -9322,9 +9322,6 @@ void intel_mark_idle(struct drm_device *dev)
 
         dev_priv->mm.busy = false;
 
-        if (!i915.powersave)
-                goto out;
-
         for_each_crtc(dev, crtc) {
                 if (!crtc->primary->fb)
                         continue;
@@ -9335,7 +9332,6 @@ void intel_mark_idle(struct drm_device *dev)
         if (INTEL_INFO(dev)->gen >= 6)
                 gen6_rps_idle(dev->dev_private);
 
-out:
         intel_runtime_pm_put(dev_priv);
 }