1 files changed, 15 insertions, 24 deletions

diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index 32ce9365d763..2f2ced7c6921 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu/drm/i915/intel_pm.c
@@ -3494,30 +3494,14 @@ static uint32_t skl_wm_method1(uint32_t pixel_rate, uint8_t cpp, uint32_t latenc
 }
 
 static uint32_t skl_wm_method2(uint32_t pixel_rate, uint32_t pipe_htotal,
-                               uint32_t horiz_pixels, uint8_t cpp,
-                               uint64_t tiling, uint32_t latency,
-                               uint32_t y_min_scanlines)
+                               uint32_t latency, uint32_t plane_blocks_per_line)
 {
         uint32_t ret;
-        uint32_t plane_bytes_per_line, plane_blocks_per_line;
         uint32_t wm_intermediate_val;
 
         if (latency == 0)
                 return UINT_MAX;
 
-        plane_bytes_per_line = horiz_pixels * cpp;
-
-        if (tiling == I915_FORMAT_MOD_Y_TILED ||
-            tiling == I915_FORMAT_MOD_Yf_TILED) {
-                plane_bytes_per_line *= y_min_scanlines;
-                plane_blocks_per_line = DIV_ROUND_UP(plane_bytes_per_line, 512);
-                plane_blocks_per_line /= y_min_scanlines;
-        } else if (tiling == DRM_FORMAT_MOD_NONE) {
-                plane_blocks_per_line = DIV_ROUND_UP(plane_bytes_per_line, 512) + 1;
-        } else {
-                plane_blocks_per_line = DIV_ROUND_UP(plane_bytes_per_line, 512);
-        }
-
         wm_intermediate_val = latency * pixel_rate;
         ret = DIV_ROUND_UP(wm_intermediate_val, pipe_htotal * 1000) *
                                 plane_blocks_per_line;
@@ -3606,17 +3590,24 @@ static int skl_compute_plane_wm(const struct drm_i915_private *dev_priv,
                 y_min_scanlines = 4;
         }
 
+        plane_bytes_per_line = width * cpp;
+        if (fb->modifier[0] == I915_FORMAT_MOD_Y_TILED ||
+            fb->modifier[0] == I915_FORMAT_MOD_Yf_TILED) {
+                plane_blocks_per_line =
+                      DIV_ROUND_UP(plane_bytes_per_line * y_min_scanlines, 512);
+                plane_blocks_per_line /= y_min_scanlines;
+        } else if (fb->modifier[0] == DRM_FORMAT_MOD_NONE) {
+                plane_blocks_per_line = DIV_ROUND_UP(plane_bytes_per_line, 512)
+                                        + 1;
+        } else {
+                plane_blocks_per_line = DIV_ROUND_UP(plane_bytes_per_line, 512);
+        }
+
         method1 = skl_wm_method1(plane_pixel_rate, cpp, latency);
         method2 = skl_wm_method2(plane_pixel_rate,
                                  cstate->base.adjusted_mode.crtc_htotal,
-                                 width,
-                                 cpp,
-                                 fb->modifier[0],
                                  latency,
-                                 y_min_scanlines);
-
-        plane_bytes_per_line = width * cpp;
-        plane_blocks_per_line = DIV_ROUND_UP(plane_bytes_per_line, 512);
+                                 plane_blocks_per_line);
 
         if (fb->modifier[0] == I915_FORMAT_MOD_Y_TILED ||
             fb->modifier[0] == I915_FORMAT_MOD_Yf_TILED) {