net/mlx5: Lag, Remove redundant bool allocation on the stack
authorRoi Dayan <roid@nvidia.com>
Thu, 1 Dec 2022 09:14:00 +0000 (11:14 +0200)
committerSaeed Mahameed <saeedm@nvidia.com>
Sat, 4 Feb 2023 10:07:02 +0000 (02:07 -0800)
There is no need to allocate the bool variable and can just return the value.

Signed-off-by: Roi Dayan <roid@nvidia.com>
Reviewed-by: Maor Dickman <maord@nvidia.com>
Signed-off-by: Saeed Mahameed <saeedm@nvidia.com>
drivers/net/ethernet/mellanox/mlx5/core/lag/mp.c
drivers/net/ethernet/mellanox/mlx5/core/lag/mpesw.c

index d9fcb9e..d85a8df 100644 (file)
@@ -28,13 +28,9 @@ static bool mlx5_lag_multipath_check_prereq(struct mlx5_lag *ldev)
 
 bool mlx5_lag_is_multipath(struct mlx5_core_dev *dev)
 {
-       struct mlx5_lag *ldev;
-       bool res;
-
-       ldev = mlx5_lag_dev(dev);
-       res  = ldev && __mlx5_lag_is_multipath(ldev);
+       struct mlx5_lag *ldev = mlx5_lag_dev(dev);
 
-       return res;
+       return ldev && __mlx5_lag_is_multipath(ldev);
 }
 
 /**
index d2fec72..3799f89 100644 (file)
@@ -115,10 +115,8 @@ int mlx5_lag_mpesw_do_mirred(struct mlx5_core_dev *mdev,
 bool mlx5_lag_mpesw_is_activated(struct mlx5_core_dev *dev)
 {
        struct mlx5_lag *ldev = mlx5_lag_dev(dev);
-       bool ret;
 
-       ret = ldev && ldev->mode == MLX5_LAG_MODE_MPESW;
-       return ret;
+       return ldev && ldev->mode == MLX5_LAG_MODE_MPESW;
 }
 
 void mlx5_lag_mpesw_init(struct mlx5_lag *ldev)