From: Vadim Pasternak <vadimp@nvidia.com>
Date: Wed, 8 Feb 2023 06:33:29 +0000 (+0200)
Subject: platform: mellanox: mlx-platform: Move bus shift assignment out of the loop
X-Git-Tag: v6.6.17~5511^2~5
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=50b823fdd357ec0f2fd0a1a0b5104704916fd620;p=platform%2Fkernel%2Flinux-rpi.git

platform: mellanox: mlx-platform: Move bus shift assignment out of the loop

Move assignment of bus shift setting out of the loop to avoid redundant
operation.

Signed-off-by: Vadim Pasternak <vadimp@nvidia.com>
Reviewed-by: Michael Shych <michaelsh@nvidia.com>
Link: https://lore.kernel.org/r/20230208063331.15560-13-vadimp@nvidia.com
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---

diff --git a/drivers/platform/x86/mlx-platform.c b/drivers/platform/x86/mlx-platform.c
index 60062d9..7b6779c 100644
--- a/drivers/platform/x86/mlx-platform.c
+++ b/drivers/platform/x86/mlx-platform.c
@@ -6008,10 +6008,11 @@ static int mlxplat_mlxcpld_verify_bus_topology(int *nr)
 		shift = *nr - mlxplat_mux_data[i].parent;
 		mlxplat_mux_data[i].parent = *nr;
 		mlxplat_mux_data[i].base_nr += shift;
-		if (shift > 0)
-			mlxplat_hotplug->shift_nr = shift;
 	}
 
+	if (shift > 0)
+		mlxplat_hotplug->shift_nr = shift;
+
 	return 0;
 }