* according to register description and PRM.
         */
        drm_WARN_ON(&dev_priv->drm,
-                   intel_de_read(dev_priv, PFIT_CONTROL) & PFIT_ENABLE);
+                   intel_de_read(dev_priv, PFIT_CONTROL(dev_priv)) & PFIT_ENABLE);
        assert_transcoder_disabled(dev_priv, crtc_state->cpu_transcoder);
 
        intel_de_write(dev_priv, PFIT_PGM_RATIOS,
                       crtc_state->gmch_pfit.pgm_ratios);
-       intel_de_write(dev_priv, PFIT_CONTROL, crtc_state->gmch_pfit.control);
+       intel_de_write(dev_priv, PFIT_CONTROL(dev_priv),
+                      crtc_state->gmch_pfit.control);
 
        /* Border color in case we don't scale up to the full screen. Black by
         * default, change to something else for debugging. */
        assert_transcoder_disabled(dev_priv, old_crtc_state->cpu_transcoder);
 
        drm_dbg_kms(&dev_priv->drm, "disabling pfit, current: 0x%08x\n",
-                   intel_de_read(dev_priv, PFIT_CONTROL));
-       intel_de_write(dev_priv, PFIT_CONTROL, 0);
+                   intel_de_read(dev_priv, PFIT_CONTROL(dev_priv)));
+       intel_de_write(dev_priv, PFIT_CONTROL(dev_priv), 0);
 }
 
 static void i9xx_crtc_disable(struct intel_atomic_state *state,
        if (!i9xx_has_pfit(dev_priv))
                return;
 
-       tmp = intel_de_read(dev_priv, PFIT_CONTROL);
+       tmp = intel_de_read(dev_priv, PFIT_CONTROL(dev_priv));
        if (!(tmp & PFIT_ENABLE))
                return;
 
 
 #define   VIDEO_DIP_ENABLE_AS_ADL      REG_BIT(23)
 
 /* Panel fitting */
-#define PFIT_CONTROL   _MMIO(DISPLAY_MMIO_BASE(dev_priv) + 0x61230)
+#define PFIT_CONTROL(dev_priv) _MMIO(DISPLAY_MMIO_BASE(dev_priv) + 0x61230)
 #define   PFIT_ENABLE                  REG_BIT(31)
 #define   PFIT_PIPE_MASK               REG_GENMASK(30, 29) /* 965+ */
 #define   PFIT_PIPE(pipe)              REG_FIELD_PREP(PFIT_PIPE_MASK, (pipe))