| From 586758d9bec7b9acb46c94c37ae33c506f895bd1 Mon Sep 17 00:00:00 2001 |
| From: Imre Deak <imre.deak@intel.com> |
| Date: Wed, 17 Apr 2013 14:04:50 +0300 |
| Subject: drm/i915: HSW: allow PCH clock gating for suspend |
| |
| For the device to enter D3 we should enable PCH clock gating. |
| |
| v2: |
| - use HAS_PCH_LPT instead of IS_HASWELL (Ville, Paolo) |
| - rename lpt_allow_clock_gating to lpt_suspend_hw (Paolo) |
| |
| Signed-off-by: Imre Deak <imre.deak@intel.com> |
| Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com> |
| Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch> |
| (cherry picked from commit 7d708ee40a6b9ca1112a322e554c887df105b025) |
| Signed-off-by: Darren Hart <dvhart@linux.intel.com> |
| --- |
| drivers/gpu/drm/i915/i915_drv.c | 2 ++ |
| drivers/gpu/drm/i915/i915_drv.h | 1 + |
| drivers/gpu/drm/i915/intel_display.c | 5 +++++ |
| drivers/gpu/drm/i915/intel_drv.h | 1 + |
| drivers/gpu/drm/i915/intel_pm.c | 18 ++++++++++++++++++ |
| 5 files changed, 27 insertions(+) |
| |
| diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c |
| index 7610a28a1bf6..e91593a4d450 100644 |
| --- a/drivers/gpu/drm/i915/i915_drv.c |
| +++ b/drivers/gpu/drm/i915/i915_drv.c |
| @@ -553,6 +553,8 @@ static int i915_drm_freeze(struct drm_device *dev) |
| */ |
| list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) |
| dev_priv->display.crtc_disable(crtc); |
| + |
| + intel_modeset_suspend_hw(dev); |
| } |
| |
| i915_save_state(dev); |
| diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h |
| index eee5f8358579..aa81716db4f3 100644 |
| --- a/drivers/gpu/drm/i915/i915_drv.h |
| +++ b/drivers/gpu/drm/i915/i915_drv.h |
| @@ -1869,6 +1869,7 @@ static inline void intel_unregister_dsm_handler(void) { return; } |
| |
| /* modesetting */ |
| extern void intel_modeset_init_hw(struct drm_device *dev); |
| +extern void intel_modeset_suspend_hw(struct drm_device *dev); |
| extern void intel_modeset_init(struct drm_device *dev); |
| extern void intel_modeset_gem_init(struct drm_device *dev); |
| extern void intel_modeset_cleanup(struct drm_device *dev); |
| diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c |
| index e43dad8d28bb..87414ec78fd4 100644 |
| --- a/drivers/gpu/drm/i915/intel_display.c |
| +++ b/drivers/gpu/drm/i915/intel_display.c |
| @@ -9326,6 +9326,11 @@ void intel_modeset_init_hw(struct drm_device *dev) |
| mutex_unlock(&dev->struct_mutex); |
| } |
| |
| +void intel_modeset_suspend_hw(struct drm_device *dev) |
| +{ |
| + intel_suspend_hw(dev); |
| +} |
| + |
| void intel_modeset_init(struct drm_device *dev) |
| { |
| struct drm_i915_private *dev_priv = dev->dev_private; |
| diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h |
| index e0d68179d80e..cb5e0cea4e93 100644 |
| --- a/drivers/gpu/drm/i915/intel_drv.h |
| +++ b/drivers/gpu/drm/i915/intel_drv.h |
| @@ -717,6 +717,7 @@ extern void assert_pipe(struct drm_i915_private *dev_priv, enum pipe pipe, |
| #define assert_pipe_disabled(d, p) assert_pipe(d, p, false) |
| |
| extern void intel_init_clock_gating(struct drm_device *dev); |
| +extern void intel_suspend_hw(struct drm_device *dev); |
| extern void intel_write_eld(struct drm_encoder *encoder, |
| struct drm_display_mode *mode); |
| extern void intel_prepare_ddi(struct drm_device *dev); |
| diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c |
| index f88cfbedc90e..863eaec77481 100644 |
| --- a/drivers/gpu/drm/i915/intel_pm.c |
| +++ b/drivers/gpu/drm/i915/intel_pm.c |
| @@ -3988,6 +3988,18 @@ static void lpt_init_clock_gating(struct drm_device *dev) |
| PCH_LP_PARTITION_LEVEL_DISABLE); |
| } |
| |
| +static void lpt_suspend_hw(struct drm_device *dev) |
| +{ |
| + struct drm_i915_private *dev_priv = dev->dev_private; |
| + |
| + if (dev_priv->pch_id == INTEL_PCH_LPT_LP_DEVICE_ID_TYPE) { |
| + uint32_t val = I915_READ(SOUTH_DSPCLK_GATE_D); |
| + |
| + val &= ~PCH_LP_PARTITION_LEVEL_DISABLE; |
| + I915_WRITE(SOUTH_DSPCLK_GATE_D, val); |
| + } |
| +} |
| + |
| static void haswell_init_clock_gating(struct drm_device *dev) |
| { |
| struct drm_i915_private *dev_priv = dev->dev_private; |
| @@ -4340,6 +4352,12 @@ void intel_init_clock_gating(struct drm_device *dev) |
| dev_priv->display.init_clock_gating(dev); |
| } |
| |
| +void intel_suspend_hw(struct drm_device *dev) |
| +{ |
| + if (HAS_PCH_LPT(dev)) |
| + lpt_suspend_hw(dev); |
| +} |
| + |
| /** |
| * We should only use the power well if we explicitly asked the hardware to |
| * enable it, so check if it's enabled and also check if we've requested it to |
| -- |
| 1.8.5.rc3 |
| |