The mlx5_dev_to_pci_addr function defines a variable called ret inside a loop and uses it.
During the loop, the function assigns a value within the variable and breaks from the loop, so that this assigning has done nothing and is actually unnecessary. Remove the unnecessary assigning. Signed-off-by: Michael Baum <michae...@mellanox.com> Acked-by: Matan Azrad <ma...@mellanox.com> --- drivers/common/mlx5/linux/mlx5_common_os.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/common/mlx5/linux/mlx5_common_os.c b/drivers/common/mlx5/linux/mlx5_common_os.c index e422e95..e74aa89 100644 --- a/drivers/common/mlx5/linux/mlx5_common_os.c +++ b/drivers/common/mlx5/linux/mlx5_common_os.c @@ -66,7 +66,6 @@ &pci_addr->bus, &pci_addr->devid, &pci_addr->function) == 4) { - ret = 0; break; } } -- 1.8.3.1