From: Ville Syrjälä <[email protected]>

Move the inner loop out from the outer loop in
sanitize_wm_latency() to flatten things a bit.
Easier to read flat code.

v2: Move the inner loop out completely (Luca)

Cc: Luca Coelho <[email protected]>
Signed-off-by: Ville Syrjälä <[email protected]>
---
 drivers/gpu/drm/i915/display/skl_watermark.c | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/skl_watermark.c 
b/drivers/gpu/drm/i915/display/skl_watermark.c
index 1acb9285bd05..d83772c6ea9a 100644
--- a/drivers/gpu/drm/i915/display/skl_watermark.c
+++ b/drivers/gpu/drm/i915/display/skl_watermark.c
@@ -3228,14 +3228,12 @@ static void sanitize_wm_latency(struct intel_display 
*display)
         * of the punit to satisfy this requirement.
         */
        for (level = 1; level < num_levels; level++) {
-               if (wm[level] == 0) {
-                       int i;
-
-                       for (i = level + 1; i < num_levels; i++)
-                               wm[i] = 0;
-                       return;
-               }
+               if (wm[level] == 0)
+                       break;
        }
+
+       for (level = level + 1; level < num_levels; level++)
+               wm[level] = 0;
 }
 
 static void
-- 
2.49.1

Reply via email to