]> www.infradead.org Git - users/dwmw2/linux.git/commitdiff
drm/i915/ddi: use intel_de_wait_custom() instead of wait_for_us()
authorJani Nikula <jani.nikula@intel.com>
Thu, 31 Jul 2025 10:05:09 +0000 (13:05 +0300)
committerJani Nikula <jani.nikula@intel.com>
Fri, 1 Aug 2025 08:37:10 +0000 (11:37 +0300)
Prefer the register read specific wait function over i915 wait_for_us().

v2: Wait for bits to clear in mtl_ddi_disable_d2d()

Reviewed-by: Gustavo Sousa <gustavo.sousa@intel.com>
Link: https://lore.kernel.org/r/232a554db6a327974c06f2491311b28f865467b9.1753956266.git.jani.nikula@intel.com
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
drivers/gpu/drm/i915/display/intel_ddi.c

index 0405396c7750eaf40dd35e25327b2d282c208c28..866ed3e4666458de510f521060398ad125f1b202 100644 (file)
@@ -2561,6 +2561,7 @@ mtl_ddi_enable_d2d(struct intel_encoder *encoder)
        enum port port = encoder->port;
        i915_reg_t reg;
        u32 set_bits, wait_bits;
+       int ret;
 
        if (DISPLAY_VER(display) < 14)
                return;
@@ -2576,7 +2577,11 @@ mtl_ddi_enable_d2d(struct intel_encoder *encoder)
        }
 
        intel_de_rmw(display, reg, 0, set_bits);
-       if (wait_for_us(intel_de_read(display, reg) & wait_bits, 100)) {
+
+       ret = intel_de_wait_custom(display, reg,
+                                  wait_bits, wait_bits,
+                                  100, 0, NULL);
+       if (ret) {
                drm_err(display->drm, "Timeout waiting for D2D Link enable for DDI/PORT_BUF_CTL %c\n",
                        port_name(port));
        }
@@ -3058,6 +3063,7 @@ mtl_ddi_disable_d2d(struct intel_encoder *encoder)
        enum port port = encoder->port;
        i915_reg_t reg;
        u32 clr_bits, wait_bits;
+       int ret;
 
        if (DISPLAY_VER(display) < 14)
                return;
@@ -3073,7 +3079,11 @@ mtl_ddi_disable_d2d(struct intel_encoder *encoder)
        }
 
        intel_de_rmw(display, reg, clr_bits, 0);
-       if (wait_for_us(!(intel_de_read(display, reg) & wait_bits), 100))
+
+       ret = intel_de_wait_custom(display, reg,
+                                  wait_bits, 0,
+                                  100, 0, NULL);
+       if (ret)
                drm_err(display->drm, "Timeout waiting for D2D Link disable for DDI/PORT_BUF_CTL %c\n",
                        port_name(port));
 }