drm/i915/xe2lpd: Extend Wa_15010685871
authorLucas De Marchi <lucas.demarchi@intel.com>
Tue, 19 Sep 2023 19:21:23 +0000 (12:21 -0700)
committerLucas De Marchi <lucas.demarchi@intel.com>
Thu, 21 Sep 2023 15:18:06 +0000 (08:18 -0700)
Xe2_LPD also needs workaround 15010685871. While adding the new display
version, also re-order the condition to follow the convention of new
version first.

v2: Remove redundant HAS_CDCLK_SQUASH(). As the platform or IP version
    needing the workaround are handpicked, there is no need to also
    check if tha platform has squashing support (Matt Roper)

Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Reviewed-by: Matt Roper <matthew.d.roper@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230919192128.2045154-17-lucas.demarchi@intel.com
drivers/gpu/drm/i915/display/intel_cdclk.c

index ad5251ba6fe1392a1e718042eda094d4b7ad5434..656ff50def399aa44b8708375be16da01ea940a5 100644 (file)
@@ -1841,9 +1841,10 @@ static bool cdclk_compute_crawl_and_squash_midpoint(struct drm_i915_private *i91
 
 static bool pll_enable_wa_needed(struct drm_i915_private *dev_priv)
 {
-       return ((IS_DG2(dev_priv) || DISPLAY_VER_FULL(dev_priv) == IP_VER(14, 0)) &&
-               dev_priv->display.cdclk.hw.vco > 0 &&
-               HAS_CDCLK_SQUASH(dev_priv));
+       return (DISPLAY_VER_FULL(dev_priv) == IP_VER(20, 0) ||
+               DISPLAY_VER_FULL(dev_priv) == IP_VER(14, 0) ||
+               IS_DG2(dev_priv)) &&
+               dev_priv->display.cdclk.hw.vco > 0;
 }
 
 static void _bxt_set_cdclk(struct drm_i915_private *dev_priv,