]> www.infradead.org Git - users/hch/configfs.git/commitdiff
drm/i915: pass dev_priv explicitly to DSPADDR
authorJani Nikula <jani.nikula@intel.com>
Thu, 23 May 2024 12:59:31 +0000 (15:59 +0300)
committerJani Nikula <jani.nikula@intel.com>
Fri, 24 May 2024 07:40:37 +0000 (10:40 +0300)
Avoid the implicit dev_priv local variable use, and pass dev_priv
explicitly to the DSPADDR register macro.

Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/777d4189c18c16392015dd2770f5c56d94bb88a9.1716469091.git.jani.nikula@intel.com
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
drivers/gpu/drm/i915/display/i9xx_plane.c
drivers/gpu/drm/i915/display/i9xx_plane_regs.h
drivers/gpu/drm/i915/display/intel_fbc.c
drivers/gpu/drm/i915/intel_gvt_mmio_table.c

index ceb0a969357fe7fcfe52af0202d651ba7153f52e..79280fe2662daf9ef1627267766ff812578026be 100644 (file)
@@ -502,7 +502,7 @@ static void i9xx_plane_update_arm(struct intel_plane *plane,
                intel_de_write_fw(dev_priv, DSPSURF(i9xx_plane),
                                  intel_plane_ggtt_offset(plane_state) + dspaddr_offset);
        else
-               intel_de_write_fw(dev_priv, DSPADDR(i9xx_plane),
+               intel_de_write_fw(dev_priv, DSPADDR(dev_priv, i9xx_plane),
                                  intel_plane_ggtt_offset(plane_state) + dspaddr_offset);
 }
 
@@ -544,7 +544,7 @@ static void i9xx_plane_disable_arm(struct intel_plane *plane,
        if (DISPLAY_VER(dev_priv) >= 4)
                intel_de_write_fw(dev_priv, DSPSURF(i9xx_plane), 0);
        else
-               intel_de_write_fw(dev_priv, DSPADDR(i9xx_plane), 0);
+               intel_de_write_fw(dev_priv, DSPADDR(dev_priv, i9xx_plane), 0);
 }
 
 static void
@@ -1045,7 +1045,7 @@ i9xx_get_initial_plane_config(struct intel_crtc *crtc,
                base = intel_de_read(dev_priv, DSPSURF(i9xx_plane)) & DISP_ADDR_MASK;
        } else {
                offset = 0;
-               base = intel_de_read(dev_priv, DSPADDR(i9xx_plane));
+               base = intel_de_read(dev_priv, DSPADDR(dev_priv, i9xx_plane));
        }
        plane_config->base = base;
 
@@ -1096,7 +1096,7 @@ bool i9xx_fixup_initial_plane_config(struct intel_crtc *crtc,
        if (DISPLAY_VER(dev_priv) >= 4)
                intel_de_write(dev_priv, DSPSURF(i9xx_plane), base);
        else
-               intel_de_write(dev_priv, DSPADDR(i9xx_plane), base);
+               intel_de_write(dev_priv, DSPADDR(dev_priv, i9xx_plane), base);
 
        return true;
 }
index d483569e414787cb870cf45663e9759176f47a42..a68d7b228187154f8ff2b4816f93d00a266b9a7f 100644 (file)
@@ -44,7 +44,7 @@
 #define   DISP_MIRROR                  REG_BIT(8) /* CHV pipe B */
 
 #define _DSPAADDR                              0x70184 /* pre-i965 */
-#define DSPADDR(plane)                         _MMIO_PIPE2(dev_priv, plane, _DSPAADDR)
+#define DSPADDR(dev_priv, plane)               _MMIO_PIPE2(dev_priv, plane, _DSPAADDR)
 
 #define _DSPALINOFF                            0x70184 /* i965+ */
 #define DSPLINOFF(plane)                       _MMIO_PIPE2(dev_priv, plane, _DSPALINOFF)
index 680d7fc3950382a4894f73adc30460db9a02cea8..f46e01cad05372cc218f04f2015bc84968a129b1 100644 (file)
@@ -327,8 +327,8 @@ static void i8xx_fbc_nuke(struct intel_fbc *fbc)
        enum i9xx_plane_id i9xx_plane = fbc_state->plane->i9xx_plane;
        struct drm_i915_private *dev_priv = fbc->i915;
 
-       intel_de_write_fw(dev_priv, DSPADDR(i9xx_plane),
-                         intel_de_read_fw(dev_priv, DSPADDR(i9xx_plane)));
+       intel_de_write_fw(dev_priv, DSPADDR(dev_priv, i9xx_plane),
+                         intel_de_read_fw(dev_priv, DSPADDR(dev_priv, i9xx_plane)));
 }
 
 static void i8xx_fbc_program_cfb(struct intel_fbc *fbc)
index 909823d7ed1b499b66768347a27dea53c4ca968f..944765fe22e7ba58212421cb52779979bee4694e 100644 (file)
@@ -166,7 +166,7 @@ static int iterate_generic_mmio(struct intel_gvt_mmio_table_iter *iter)
        MMIO_D(_MMIO(0x70098));
        MMIO_D(_MMIO(0x7009c));
        MMIO_D(DSPCNTR(dev_priv, PIPE_A));
-       MMIO_D(DSPADDR(PIPE_A));
+       MMIO_D(DSPADDR(dev_priv, PIPE_A));
        MMIO_D(DSPSTRIDE(PIPE_A));
        MMIO_D(DSPPOS(PIPE_A));
        MMIO_D(DSPSIZE(PIPE_A));
@@ -175,7 +175,7 @@ static int iterate_generic_mmio(struct intel_gvt_mmio_table_iter *iter)
        MMIO_D(DSPSURFLIVE(PIPE_A));
        MMIO_D(REG_50080(PIPE_A, PLANE_PRIMARY));
        MMIO_D(DSPCNTR(dev_priv, PIPE_B));
-       MMIO_D(DSPADDR(PIPE_B));
+       MMIO_D(DSPADDR(dev_priv, PIPE_B));
        MMIO_D(DSPSTRIDE(PIPE_B));
        MMIO_D(DSPPOS(PIPE_B));
        MMIO_D(DSPSIZE(PIPE_B));
@@ -184,7 +184,7 @@ static int iterate_generic_mmio(struct intel_gvt_mmio_table_iter *iter)
        MMIO_D(DSPSURFLIVE(PIPE_B));
        MMIO_D(REG_50080(PIPE_B, PLANE_PRIMARY));
        MMIO_D(DSPCNTR(dev_priv, PIPE_C));
-       MMIO_D(DSPADDR(PIPE_C));
+       MMIO_D(DSPADDR(dev_priv, PIPE_C));
        MMIO_D(DSPSTRIDE(PIPE_C));
        MMIO_D(DSPPOS(PIPE_C));
        MMIO_D(DSPSIZE(PIPE_C));