Quoting Jani Nikula (2024-03-20 13:01:23-03:00) >Prefer the intel_de_wait*() functions over the uncore interface. > >Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx> Reviewed-by: Gustavo Sousa <gustavo.sousa@xxxxxxxxx> >--- > drivers/gpu/drm/i915/display/intel_dpio_phy.c | 7 ++----- > drivers/gpu/drm/i915/display/intel_hdcp.c | 6 +++--- > 2 files changed, 5 insertions(+), 8 deletions(-) > >diff --git a/drivers/gpu/drm/i915/display/intel_dpio_phy.c b/drivers/gpu/drm/i915/display/intel_dpio_phy.c >index 4ca910874a4f..2d7a71c8c69c 100644 >--- a/drivers/gpu/drm/i915/display/intel_dpio_phy.c >+++ b/drivers/gpu/drm/i915/display/intel_dpio_phy.c >@@ -399,11 +399,8 @@ static void _bxt_ddi_phy_init(struct drm_i915_private *dev_priv, > * The flag should get set in 100us according to the HW team, but > * use 1ms due to occasional timeouts observed with that. > */ >- if (intel_wait_for_register_fw(&dev_priv->uncore, >- BXT_PORT_CL1CM_DW0(phy), >- PHY_RESERVED | PHY_POWER_GOOD, >- PHY_POWER_GOOD, >- 1)) >+ if (intel_de_wait_fw(dev_priv, BXT_PORT_CL1CM_DW0(phy), >+ PHY_RESERVED | PHY_POWER_GOOD, PHY_POWER_GOOD, 1)) > drm_err(&dev_priv->drm, "timeout during PHY%d power on\n", > phy); > >diff --git a/drivers/gpu/drm/i915/display/intel_hdcp.c b/drivers/gpu/drm/i915/display/intel_hdcp.c >index 9edac27bab26..d5ed4c7dfbc0 100644 >--- a/drivers/gpu/drm/i915/display/intel_hdcp.c >+++ b/drivers/gpu/drm/i915/display/intel_hdcp.c >@@ -369,9 +369,9 @@ static int intel_hdcp_load_keys(struct drm_i915_private *i915) > } > > /* Wait for the keys to load (500us) */ >- ret = __intel_wait_for_register(&i915->uncore, HDCP_KEY_STATUS, >- HDCP_KEY_LOAD_DONE, HDCP_KEY_LOAD_DONE, >- 10, 1, &val); >+ ret = intel_de_wait_custom(i915, HDCP_KEY_STATUS, >+ HDCP_KEY_LOAD_DONE, HDCP_KEY_LOAD_DONE, >+ 10, 1, &val); > if (ret) > return ret; > else if (!(val & HDCP_KEY_LOAD_STATUS)) >-- >2.39.2 >