From: Mahesh Kumar <mahesh1.ku...@intel.com>

DDB allocation optimization algorithm requires/assumes ddb allocation for
any memory C-state level DDB value to be as high as level below the
current level. Render decompression requires level WM to be as high as
wm level-0. This patch fulfils both the requirements.

v2: Changed plane_num to plane_id in skl_compute_wm_levels

Signed-off-by: Mahesh Kumar <mahesh1.ku...@intel.com>
Signed-off-by: Vidya Srinivas <vidya.srini...@intel.com>
---
 drivers/gpu/drm/i915/intel_pm.c | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index 5154ce3..77c268f 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu/drm/i915/intel_pm.c
@@ -4532,6 +4532,7 @@ static int skl_compute_plane_wm(const struct 
drm_i915_private *dev_priv,
                                uint16_t ddb_allocation,
                                int level,
                                const struct skl_wm_params *wp,
+                               const struct skl_wm_level *result_prev,
                                struct skl_wm_level *result /* out */)
 {
        const struct drm_plane_state *pstate = &intel_pstate->base;
@@ -4599,6 +4600,15 @@ static int skl_compute_plane_wm(const struct 
drm_i915_private *dev_priv,
                } else {
                        res_blocks++;
                }
+
+               /*
+                * Make sure result blocks for higher latency levels are atleast
+                * as high as level below the current level.
+                * Assumption in DDB algorithm optimization for special cases.
+                * Also covers Display WA #1125 for RC.
+                */
+               if (result_prev->plane_res_b > res_blocks)
+                       res_blocks = result_prev->plane_res_b;
        }
 
        if (INTEL_GEN(dev_priv) >= 11) {
@@ -4682,6 +4692,13 @@ skl_compute_wm_levels(const struct drm_i915_private 
*dev_priv,
        for (level = 0; level <= max_level; level++) {
                struct skl_wm_level *result = plane_id ? &wm->uv_wm[level] :
                                                          &wm->wm[level];
+               struct skl_wm_level *result_prev;
+
+               if (level)
+                       result_prev = plane_id ? &wm->uv_wm[level - 1] :
+                                                 &wm->wm[level - 1];
+               else
+                       result_prev = plane_id ? &wm->uv_wm[0] : &wm->wm[0];
 
                ret = skl_compute_plane_wm(dev_priv,
                                           cstate,
@@ -4689,6 +4706,7 @@ skl_compute_wm_levels(const struct drm_i915_private 
*dev_priv,
                                           ddb_blocks,
                                           level,
                                           wm_params,
+                                          result_prev,
                                           result);
                if (ret)
                        return ret;
-- 
2.7.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to