drm/i915: Simplify aux_ch_to_digital_port()

Just return the correct thing from within the loop to make
the code more readable. We have no ref counts/etc. to deal
with here so no point in breaking from the loop just to return
something.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240223203216.15210-2-ville.syrjala@linux.intel.com
Reviewed-by: Imre Deak <imre.deak@intel.com>

diff --git a/drivers/gpu/drm/i915/display/intel_display_power_well.c b/drivers/gpu/drm/i915/display/intel_display_power_well.c
index 949ae06ad727e8417eb9e3de14c205e348dfb9de..c20e80aded353de5504b69d7133a22a595a64354 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_display_power_well.c
+++ b/drivers/gpu/drm/i915/display/intel_display_power_well.c
@@ -217,27 +217,22 @@ static struct intel_digital_port *
 aux_ch_to_digital_port(struct drm_i915_private *dev_priv,
                       enum aux_ch aux_ch)
 {
-       struct intel_digital_port *dig_port = NULL;
        struct intel_encoder *encoder;
 
        for_each_intel_encoder(&dev_priv->drm, encoder) {
+               struct intel_digital_port *dig_port;
+
                /* We'll check the MST primary port */
                if (encoder->type == INTEL_OUTPUT_DP_MST)
                        continue;
 
                dig_port = enc_to_dig_port(encoder);
-               if (!dig_port)
-                       continue;
-
-               if (dig_port->aux_ch != aux_ch) {
-                       dig_port = NULL;
-                       continue;
-               }
 
-               break;
+               if (dig_port && dig_port->aux_ch == aux_ch)
+                       return dig_port;
        }
 
-       return dig_port;
+       return NULL;
 }
 
 static enum phy icl_aux_pw_to_phy(struct drm_i915_private *i915,