]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
drm/i915: pass dev_priv explicitly to DPLL_MD
authorJani Nikula <jani.nikula@intel.com>
Tue, 4 Jun 2024 15:25:20 +0000 (18:25 +0300)
committerJani Nikula <jani.nikula@intel.com>
Fri, 7 Jun 2024 08:13:16 +0000 (11:13 +0300)
Avoid the implicit dev_priv local variable use, and pass dev_priv
explicitly to the DPLL_MD register macro.

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/98d24284d4ec435c3acae6445943204dfa96617d.1717514638.git.jani.nikula@intel.com
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
drivers/gpu/drm/i915/display/intel_dpll.c
drivers/gpu/drm/i915/i915_reg.h

index a007ca5208b8ee951c329cdb1c46c39a7dbb5eaf..d67d5e2fd570c4583f7dc4d33b6888532f430b82 100644 (file)
@@ -398,7 +398,8 @@ void i9xx_dpll_get_hw_state(struct intel_crtc *crtc,
                if (IS_CHERRYVIEW(dev_priv) && crtc->pipe != PIPE_A)
                        tmp = dev_priv->display.state.chv_dpll_md[crtc->pipe];
                else
-                       tmp = intel_de_read(dev_priv, DPLL_MD(crtc->pipe));
+                       tmp = intel_de_read(dev_priv,
+                                           DPLL_MD(dev_priv, crtc->pipe));
 
                hw_state->dpll_md = tmp;
        }
@@ -1851,7 +1852,8 @@ void i9xx_enable_pll(const struct intel_crtc_state *crtc_state)
        udelay(150);
 
        if (DISPLAY_VER(dev_priv) >= 4) {
-               intel_de_write(dev_priv, DPLL_MD(pipe), hw_state->dpll_md);
+               intel_de_write(dev_priv, DPLL_MD(dev_priv, pipe),
+                              hw_state->dpll_md);
        } else {
                /* The pixel multiplier can only be updated once the
                 * DPLL is enabled and the clocks are stable.
@@ -2021,8 +2023,8 @@ void vlv_enable_pll(const struct intel_crtc_state *crtc_state)
                _vlv_enable_pll(crtc_state);
        }
 
-       intel_de_write(dev_priv, DPLL_MD(pipe), hw_state->dpll_md);
-       intel_de_posting_read(dev_priv, DPLL_MD(pipe));
+       intel_de_write(dev_priv, DPLL_MD(dev_priv, pipe), hw_state->dpll_md);
+       intel_de_posting_read(dev_priv, DPLL_MD(dev_priv, pipe));
 }
 
 static void chv_prepare_pll(const struct intel_crtc_state *crtc_state)
@@ -2175,7 +2177,8 @@ void chv_enable_pll(const struct intel_crtc_state *crtc_state)
                 * the value from DPLLBMD to either pipe B or C.
                 */
                intel_de_write(dev_priv, CBR4_VLV, CBR_DPLLBMD_PIPE(pipe));
-               intel_de_write(dev_priv, DPLL_MD(PIPE_B), hw_state->dpll_md);
+               intel_de_write(dev_priv, DPLL_MD(dev_priv, PIPE_B),
+                              hw_state->dpll_md);
                intel_de_write(dev_priv, CBR4_VLV, 0);
                dev_priv->display.state.chv_dpll_md[pipe] = hw_state->dpll_md;
 
@@ -2187,8 +2190,9 @@ void chv_enable_pll(const struct intel_crtc_state *crtc_state)
                            (intel_de_read(dev_priv, DPLL(dev_priv, PIPE_B)) &
                             DPLL_VGA_MODE_DIS) == 0);
        } else {
-               intel_de_write(dev_priv, DPLL_MD(pipe), hw_state->dpll_md);
-               intel_de_posting_read(dev_priv, DPLL_MD(pipe));
+               intel_de_write(dev_priv, DPLL_MD(dev_priv, pipe),
+                              hw_state->dpll_md);
+               intel_de_posting_read(dev_priv, DPLL_MD(dev_priv, pipe));
        }
 }
 
index 539c4e96d7362e64de518055cb66f9be38665ff4..3166925b9881b81d5f0630566035aecd30a83fc6 100644 (file)
 #define _DPLL_A_MD             0x601c
 #define _DPLL_B_MD             0x6020
 #define _CHV_DPLL_C_MD         0x603c
-#define DPLL_MD(pipe)          _MMIO_BASE_PIPE3(DISPLAY_MMIO_BASE(dev_priv), \
+#define DPLL_MD(dev_priv, pipe)                _MMIO_BASE_PIPE3(DISPLAY_MMIO_BASE(dev_priv), \
                                                 (pipe), _DPLL_A_MD, _DPLL_B_MD, _CHV_DPLL_C_MD)
 
 /*