[PATCH 03/14] drm/i915: Extract intel_dbuf_bw_changed()

[Ville Syrjala]

From: Ville Syrjälä <ville.syrj...@linux.intel.com>

Extract the struct intel_dbuf_bw comparison into a small
helper. We'll get more users later.

Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
---
 drivers/gpu/drm/i915/display/intel_bw.c | 23 +++++++++++++++++------
 1 file changed, 17 insertions(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_bw.c 
b/drivers/gpu/drm/i915/display/intel_bw.c
index 2bc020690515..640a24e83b6d 100644
--- a/drivers/gpu/drm/i915/display/intel_bw.c
+++ b/drivers/gpu/drm/i915/display/intel_bw.c
@@ -1117,6 +1117,21 @@ static int intel_bw_check_qgv_points(struct 
drm_i915_private *i915,
                                           old_bw_state, new_bw_state);
 }
 
+static bool intel_dbuf_bw_changed(struct drm_i915_private *i915,
+                                 const struct intel_dbuf_bw *old_dbuf_bw,
+                                 const struct intel_dbuf_bw *new_dbuf_bw)
+{
+       enum dbuf_slice slice;
+
+       for_each_dbuf_slice(i915, slice) {
+               if (old_dbuf_bw->max_bw[slice] != new_dbuf_bw->max_bw[slice] ||
+                   old_dbuf_bw->active_planes[slice] != 
new_dbuf_bw->active_planes[slice])
+                       return true;
+       }
+
+       return false;
+}
+
 static bool intel_bw_state_changed(struct drm_i915_private *i915,
                                   const struct intel_bw_state *old_bw_state,
                                   const struct intel_bw_state *new_bw_state)
@@ -1128,13 +1143,9 @@ static bool intel_bw_state_changed(struct 
drm_i915_private *i915,
                        &old_bw_state->dbuf_bw[pipe];
                const struct intel_dbuf_bw *new_dbuf_bw =
                        &new_bw_state->dbuf_bw[pipe];
-               enum dbuf_slice slice;
 
-               for_each_dbuf_slice(i915, slice) {
-                       if (old_dbuf_bw->max_bw[slice] != 
new_dbuf_bw->max_bw[slice] ||
-                           old_dbuf_bw->active_planes[slice] != 
new_dbuf_bw->active_planes[slice])
-                               return true;
-               }
+               if (intel_dbuf_bw_changed(i915, old_dbuf_bw, new_dbuf_bw))
+                       return true;
 
                if (intel_bw_crtc_min_cdclk(i915, 
old_bw_state->data_rate[pipe]) !=
                    intel_bw_crtc_min_cdclk(i915, 
new_bw_state->data_rate[pipe]))
-- 
2.45.3