From: Danielle Ratson <danieller@nvidia.com>
Date: Tue, 23 Nov 2021 09:54:46 +0200
Subject: mlxsw: spectrum_router: Remove deadcode in
mlxsw_sp_rif_mac_profile_find
Patch-mainline: v5.17-rc1
Git-commit: ed1607e2ddf4b3196703b209428cbce3de869ff8
References: jsc#PED-1549
The function idr_for_each_entry() already checks that the next entry in
the IDR is not NULL.
Therefore, checking that again in every iteration leads to deadcode.
Remove the unnecessary check in order to avoid that.
Addresses-Coverity: ("Logically dead code")
Signed-off-by: Danielle Ratson <danieller@nvidia.com>
Signed-off-by: Ido Schimmel <idosch@nvidia.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Acked-by: Thomas Bogendoerfer <tbogendoerfer@suse.de>
---
drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c | 3 ---
1 file changed, 3 deletions(-)
--- a/drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c
@@ -8239,9 +8239,6 @@ mlxsw_sp_rif_mac_profile_find(const stru
int id;
idr_for_each_entry(&router->rif_mac_profiles_idr, profile, id) {
- if (!profile)
- continue;
-
if (ether_addr_equal_masked(profile->mac_prefix, mac,
mlxsw_sp->mac_mask))
return profile;