From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga12.intel.com ([192.55.52.136]:37921 "EHLO mga12.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753279AbeDSQPr (ORCPT ); Thu, 19 Apr 2018 12:15:47 -0400 Date: Thu, 19 Apr 2018 19:15:42 +0300 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= To: Imre Deak Cc: intel-gfx@lists.freedesktop.org, Lucas De Marchi , Rodrigo Vivi , Daniel Vetter , stable@vger.kernel.org Subject: Re: [PATCH] drm/i915: Enable display WA#1183 from its correct spot Message-ID: <20180419161542.GA13908@intel.com> References: <20180419155109.29451-1-imre.deak@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20180419155109.29451-1-imre.deak@intel.com> Sender: stable-owner@vger.kernel.org List-ID: On Thu, Apr 19, 2018 at 06:51:09PM +0300, Imre Deak wrote: > The DMC FW specific part of display WA#1183 is supposed to be enabled > whenever enabling DC5 or DC6, so move it to the DC6 enable function > from the DC6 disable function. That does make more sense :) Reviewed-by: Ville Syrj�l� > > I noticed this after Daniel's patch to remove the unused > skl_disable_dc6() function. > > Fixes: 53421c2fe99c ("drm/i915: Apply Display WA #1183 on skl, kbl, and cfl") > Cc: Lucas De Marchi > Cc: Rodrigo Vivi > Cc: Ville Syrj�l� > Cc: Daniel Vetter > Cc: > Signed-off-by: Imre Deak > --- > drivers/gpu/drm/i915/intel_runtime_pm.c | 11 +++++------ > 1 file changed, 5 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_runtime_pm.c b/drivers/gpu/drm/i915/intel_runtime_pm.c > index 53ea564f971e..66de4b2dc8b7 100644 > --- a/drivers/gpu/drm/i915/intel_runtime_pm.c > +++ b/drivers/gpu/drm/i915/intel_runtime_pm.c > @@ -641,19 +641,18 @@ void skl_enable_dc6(struct drm_i915_private *dev_priv) > > DRM_DEBUG_KMS("Enabling DC6\n"); > > - gen9_set_dc_state(dev_priv, DC_STATE_EN_UPTO_DC6); > + /* Wa Display #1183: skl,kbl,cfl */ > + if (IS_GEN9_BC(dev_priv)) > + I915_WRITE(GEN8_CHICKEN_DCPR_1, I915_READ(GEN8_CHICKEN_DCPR_1) | > + SKL_SELECT_ALTERNATE_DC_EXIT); > > + gen9_set_dc_state(dev_priv, DC_STATE_EN_UPTO_DC6); > } > > void skl_disable_dc6(struct drm_i915_private *dev_priv) > { > DRM_DEBUG_KMS("Disabling DC6\n"); > > - /* Wa Display #1183: skl,kbl,cfl */ > - if (IS_GEN9_BC(dev_priv)) > - I915_WRITE(GEN8_CHICKEN_DCPR_1, I915_READ(GEN8_CHICKEN_DCPR_1) | > - SKL_SELECT_ALTERNATE_DC_EXIT); > - > gen9_set_dc_state(dev_priv, DC_STATE_DISABLE); > } > > -- > 2.13.2 -- Ville Syrj�l� Intel