For pmdemand cases, no need to even calculate the masks based
on the qgv points index. Though the current logic avoids setting
the registers based on the pmdemand support, some qgv point masks
are compared in vain and do nothing. So leave early if pmdemand
is supported.

Signed-off-by: Vinod Govindapillai <[email protected]>
---
 drivers/gpu/drm/i915/display/skl_watermark.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/drivers/gpu/drm/i915/display/skl_watermark.c 
b/drivers/gpu/drm/i915/display/skl_watermark.c
index c942ccfe6897..30a2031aa613 100644
--- a/drivers/gpu/drm/i915/display/skl_watermark.c
+++ b/drivers/gpu/drm/i915/display/skl_watermark.c
@@ -275,6 +275,9 @@ void intel_sagv_pre_plane_update(struct intel_atomic_state 
*state)
        if (!intel_has_sagv(display))
                return;
 
+       if (HAS_PMDEMAND(display))
+               return;
+
        if (DISPLAY_VER(display) >= 11)
                icl_sagv_pre_plane_update(state);
        else
@@ -295,6 +298,9 @@ void intel_sagv_post_plane_update(struct intel_atomic_state 
*state)
        if (!intel_has_sagv(display))
                return;
 
+       if (HAS_PMDEMAND(display))
+               return;
+
        if (DISPLAY_VER(display) >= 11)
                icl_sagv_post_plane_update(state);
        else
-- 
2.43.0

Reply via email to