thunderbolt: No need to loop over all retimers if access fails

When we read the NVM authentication status or unsetting the inbound SBTX
there is no point to continue the loop after first access to a retimer
fails because there won't be any more retimers after this anyway so bail
out from the loops early.

Signed-off-by: Mika Westerberg <[email protected]>

diff --git a/drivers/thunderbolt/retimer.c b/drivers/thunderbolt/retimer.c
index 6bb49bdcd6c18e6f1f56b981b40c522f67db0861..6eaaa5074ce82013d1322b0f69de7b316c88e5af 100644 (file)
--- a/drivers/thunderbolt/retimer.c
+++ b/drivers/thunderbolt/retimer.c
@@ -199,8 +199,10 @@ static void tb_retimer_nvm_authenticate_status(struct tb_port *port, u32 *status
         * If the retimer has it set, store it for the new retimer
         * device instance.
         */
-       for (i = 1; i <= TB_MAX_RETIMER_INDEX; i++)
-               usb4_port_retimer_nvm_authenticate_status(port, i, &status[i]);
+       for (i = 1; i <= TB_MAX_RETIMER_INDEX; i++) {
+               if (usb4_port_retimer_nvm_authenticate_status(port, i, &status[i]))
+                       break;
+       }
 }
 
 static void tb_retimer_set_inbound_sbtx(struct tb_port *port)
@@ -234,8 +236,10 @@ static void tb_retimer_unset_inbound_sbtx(struct tb_port *port)
 
        tb_port_dbg(port, "disabling sideband transactions\n");
 
-       for (i = TB_MAX_RETIMER_INDEX; i >= 1; i--)
-               usb4_port_retimer_unset_inbound_sbtx(port, i);
+       for (i = TB_MAX_RETIMER_INDEX; i >= 1; i--) {
+               if (usb4_port_retimer_unset_inbound_sbtx(port, i))
+                       break;
+       }
 }
 
 static ssize_t nvm_authenticate_store(struct device *dev,