On Tue, Jun 04, 2024 at 06:25:31PM +0300, Jani Nikula wrote: > Avoid the implicit dev_priv local variable use, and pass dev_priv > explicitly to the PORT_HOTPLUG_EN register macro.
Reviewed-by: Rodrigo Vivi <rodrigo.v...@intel.com> > > Signed-off-by: Jani Nikula <jani.nik...@intel.com> > --- > drivers/gpu/drm/i915/display/intel_crt.c | 2 +- > drivers/gpu/drm/i915/display/intel_hotplug_irq.c | 3 ++- > drivers/gpu/drm/i915/i915_reg.h | 2 +- > 3 files changed, 4 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_crt.c > b/drivers/gpu/drm/i915/display/intel_crt.c > index 2660c4a53e6f..b7eab52b64b6 100644 > --- a/drivers/gpu/drm/i915/display/intel_crt.c > +++ b/drivers/gpu/drm/i915/display/intel_crt.c > @@ -603,7 +603,7 @@ static bool intel_crt_detect_hotplug(struct drm_connector > *connector) > CRT_HOTPLUG_FORCE_DETECT, > CRT_HOTPLUG_FORCE_DETECT); > /* wait for FORCE_DETECT to go off */ > - if (intel_de_wait_for_clear(dev_priv, PORT_HOTPLUG_EN, > + if (intel_de_wait_for_clear(dev_priv, PORT_HOTPLUG_EN(dev_priv), > CRT_HOTPLUG_FORCE_DETECT, 1000)) > drm_dbg_kms(&dev_priv->drm, > "timed out waiting for FORCE_DETECT to go > off"); > diff --git a/drivers/gpu/drm/i915/display/intel_hotplug_irq.c > b/drivers/gpu/drm/i915/display/intel_hotplug_irq.c > index d270bb7b9462..90fe5f8538e1 100644 > --- a/drivers/gpu/drm/i915/display/intel_hotplug_irq.c > +++ b/drivers/gpu/drm/i915/display/intel_hotplug_irq.c > @@ -186,7 +186,8 @@ void i915_hotplug_interrupt_update_locked(struct > drm_i915_private *dev_priv, > lockdep_assert_held(&dev_priv->irq_lock); > drm_WARN_ON(&dev_priv->drm, bits & ~mask); > > - intel_uncore_rmw(&dev_priv->uncore, PORT_HOTPLUG_EN, mask, bits); > + intel_uncore_rmw(&dev_priv->uncore, PORT_HOTPLUG_EN(dev_priv), mask, > + bits); > } > > /** > diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h > index 3bb895d030ab..4508c535f320 100644 > --- a/drivers/gpu/drm/i915/i915_reg.h > +++ b/drivers/gpu/drm/i915/i915_reg.h > @@ -1378,7 +1378,7 @@ > > > /* Hotplug control (945+ only) */ > -#define PORT_HOTPLUG_EN _MMIO(DISPLAY_MMIO_BASE(dev_priv) + > 0x61110) > +#define PORT_HOTPLUG_EN(dev_priv) > _MMIO(DISPLAY_MMIO_BASE(dev_priv) + 0x61110) > #define PORTB_HOTPLUG_INT_EN (1 << 29) > #define PORTC_HOTPLUG_INT_EN (1 << 28) > #define PORTD_HOTPLUG_INT_EN (1 << 27) > -- > 2.39.2 >
