[Intel-gfx] [PATCH 3/8] drm/i915/dsc: have intel_dsc_pps_read() return the value

[Jani Nikula]

Register read functions usually return the value instead of passing via
pointer parameters. Return the multiple register verification results
via a pointer parameter, which can also be NULL to skip the extra
checks.

Make the name conform to existing style better while at it.

Cc: Suraj Kandpal <suraj.kand...@intel.com>
Cc: Ankit Nautiyal <ankit.k.nauti...@intel.com>
Signed-off-by: Jani Nikula <jani.nik...@intel.com>
---
 drivers/gpu/drm/i915/display/intel_vdsc.c | 32 ++++++++++++++---------
 1 file changed, 19 insertions(+), 13 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_vdsc.c 
b/drivers/gpu/drm/i915/display/intel_vdsc.c
index abb2c4370231..b0be6615a865 100644
--- a/drivers/gpu/drm/i915/display/intel_vdsc.c
+++ b/drivers/gpu/drm/i915/display/intel_vdsc.c
@@ -809,13 +809,14 @@ void intel_dsc_disable(const struct intel_crtc_state 
*old_crtc_state)
        }
 }
 
-static bool intel_dsc_read_pps_reg(struct intel_crtc_state *crtc_state,
-                                  int pps, u32 *pps_val)
+static u32 intel_dsc_pps_read(struct intel_crtc_state *crtc_state, int pps,
+                             bool *check_equal)
 {
        struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
        struct drm_i915_private *i915 = to_i915(crtc->base.dev);
        i915_reg_t dsc_reg[2];
        int i, vdsc_per_pipe, dsc_reg_num;
+       u32 val = 0;
 
        vdsc_per_pipe = intel_dsc_get_vdsc_per_pipe(crtc_state);
        dsc_reg_num = min_t(int, ARRAY_SIZE(dsc_reg), vdsc_per_pipe);
@@ -824,20 +825,25 @@ static bool intel_dsc_read_pps_reg(struct 
intel_crtc_state *crtc_state,
 
        intel_dsc_get_pps_reg(crtc_state, pps, dsc_reg, dsc_reg_num);
 
-       *pps_val = 0;
+       if (check_equal)
+               *check_equal = true;
 
        for (i = 0; i < dsc_reg_num; i++) {
-               u32 pps_temp;
+               u32 tmp;
 
-               pps_temp = intel_de_read(i915, dsc_reg[i]);
+               tmp = intel_de_read(i915, dsc_reg[i]);
 
-               if (i == 0)
-                       *pps_val = pps_temp;
-               else if (pps_temp != *pps_val)
-                       return false;
+               if (i == 0) {
+                       val = tmp;
+               } else if (check_equal && tmp != val) {
+                       *check_equal = false;
+                       break;
+               } else if (!check_equal) {
+                       break;
+               }
        }
 
-       return true;
+       return val;
 }
 
 static u32 intel_dsc_pps_read_and_verify(struct intel_crtc_state *crtc_state, 
int pps)
@@ -845,10 +851,10 @@ static u32 intel_dsc_pps_read_and_verify(struct 
intel_crtc_state *crtc_state, in
        struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
        struct drm_i915_private *i915 = to_i915(crtc->base.dev);
        u32 val;
-       int ret;
+       bool all_equal;
 
-       ret = intel_dsc_read_pps_reg(crtc_state, pps, &val);
-       drm_WARN_ON(&i915->drm, !ret);
+       val = intel_dsc_pps_read(crtc_state, pps, &all_equal);
+       drm_WARN_ON(&i915->drm, !all_equal);
 
        return val;
 }
-- 
2.39.2