diff mbox series

[1/4] drm/i915/dmc_wl: Pass offset instead of reg to range table iterator

Message ID 20250117220747.87927-2-gustavo.sousa@intel.com (mailing list archive)
State New
Headers show
Series drm/i915/dmc_wl: Introduce debugfs interface | expand

Commit Message

Gustavo Sousa Jan. 17, 2025, 10:06 p.m. UTC 
We will add another function that checks the offset in an upcoming
change. Instead of passing the reg variable to only extract the offset
later, let's extract the offset before so that we do not need to repeat
ourselves.

Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
 drivers/gpu/drm/i915/display/intel_dmc_wl.c | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)
diff mbox series

Patch

diff --git a/drivers/gpu/drm/i915/display/intel_dmc_wl.c b/drivers/gpu/drm/i915/display/intel_dmc_wl.c
index 43884740f8ea..330b43a72e08 100644
--- a/drivers/gpu/drm/i915/display/intel_dmc_wl.c
+++ b/drivers/gpu/drm/i915/display/intel_dmc_wl.c
@@ -227,11 +227,9 @@  static void __intel_dmc_wl_take(struct intel_display *display)
 	wl->taken = true;
 }
 
-static bool intel_dmc_wl_reg_in_range(i915_reg_t reg,
-				      const struct intel_dmc_wl_range ranges[])
+static bool intel_dmc_wl_offset_in_ranges(u32 offset,
+					  const struct intel_dmc_wl_range ranges[])
 {
-	u32 offset = i915_mmio_reg_offset(reg);
-
 	for (int i = 0; ranges[i].start; i++) {
 		u32 end = ranges[i].end ?: ranges[i].start;
 
@@ -247,6 +245,7 @@  static bool intel_dmc_wl_check_range(struct intel_display *display,
 				     u32 dc_state)
 {
 	const struct intel_dmc_wl_range *ranges;
+	u32 offset = i915_mmio_reg_offset(reg);
 
 	if (display->params.enable_dmc_wl == ENABLE_DMC_WL_ANY_REGISTER)
 		return true;
@@ -255,7 +254,7 @@  static bool intel_dmc_wl_check_range(struct intel_display *display,
 	 * Check that the offset is in one of the ranges for which
 	 * registers are powered off during DC states.
 	 */
-	if (intel_dmc_wl_reg_in_range(reg, powered_off_ranges))
+	if (intel_dmc_wl_offset_in_ranges(offset, powered_off_ranges))
 		return true;
 
 	/*
@@ -274,7 +273,7 @@  static bool intel_dmc_wl_check_range(struct intel_display *display,
 		ranges = NULL;
 	}
 
-	if (ranges && intel_dmc_wl_reg_in_range(reg, ranges))
+	if (ranges && intel_dmc_wl_offset_in_ranges(offset, ranges))
 		return true;
 
 	return false;