From: Saeed Mahameed <saeedm@mellanox.com>
Date: Mon, 29 Jul 2019 21:13:08 +0000
Subject: net/mlx5: E-Switch, remove redundant error handling
Patch-mainline: v5.4-rc1
Git-commit: 9ddb830a14dbd88308354d27cd17009fc97d3a6f
References: jsc#SLE-8464
We don't need to handle error flow of esw_create_legacy_table() in the
same branch, it is already being handled directly after the if statement,
for both legacy and switchdev modes in one place.
Signed-off-by: Saeed Mahameed <saeedm@mellanox.com>
Reviewed-by: Roi Dayan <roid@mellanox.com>
Signed-off-by: Saeed Mahameed <saeedm@mellanox.com>
Acked-by: Thomas Bogendoerfer <tbogendoerfer@suse.de>
---
drivers/net/ethernet/mellanox/mlx5/core/eswitch.c | 2 --
1 file changed, 2 deletions(-)
--- a/drivers/net/ethernet/mellanox/mlx5/core/eswitch.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/eswitch.c
@@ -1855,8 +1855,6 @@ int mlx5_eswitch_enable(struct mlx5_eswi
if (mode == MLX5_ESWITCH_LEGACY) {
err = esw_create_legacy_table(esw);
- if (err)
- goto abort;
} else {
mlx5_reload_interface(esw->dev, MLX5_INTERFACE_PROTOCOL_ETH);
mlx5_reload_interface(esw->dev, MLX5_INTERFACE_PROTOCOL_IB);