diff options
author | David S. Miller <davem@davemloft.net> | 2016-03-07 11:39:16 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-03-07 11:39:16 -0500 |
commit | a383b7f0e0ba9f8610bdf88a58591a9cca96c38e (patch) | |
tree | b318dc709a1903a7a542496bbba13e42385d84a4 | |
parent | 4de13d7ed6ffdcbb34317acaa9236f121176f5f8 (diff) | |
parent | 5091730d7795ccb21eb880699b5194730641c70b (diff) |
Merge branch 'mlxsw-fixes'
Jiri Pirko says:
====================
mlxsw: couple of fixes
Couple of fixes from Ido.
====================
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/ethernet/mellanox/mlxsw/pci.c | 2 | ||||
-rw-r--r-- | drivers/net/ethernet/mellanox/mlxsw/spectrum.c | 4 |
2 files changed, 2 insertions, 4 deletions
diff --git a/drivers/net/ethernet/mellanox/mlxsw/pci.c b/drivers/net/ethernet/mellanox/mlxsw/pci.c index c071077aafbd..7992c553c1f5 100644 --- a/drivers/net/ethernet/mellanox/mlxsw/pci.c +++ b/drivers/net/ethernet/mellanox/mlxsw/pci.c | |||
@@ -215,7 +215,7 @@ mlxsw_pci_queue_elem_info_producer_get(struct mlxsw_pci_queue *q) | |||
215 | { | 215 | { |
216 | int index = q->producer_counter & (q->count - 1); | 216 | int index = q->producer_counter & (q->count - 1); |
217 | 217 | ||
218 | if ((q->producer_counter - q->consumer_counter) == q->count) | 218 | if ((u16) (q->producer_counter - q->consumer_counter) == q->count) |
219 | return NULL; | 219 | return NULL; |
220 | return mlxsw_pci_queue_elem_info_get(q, index); | 220 | return mlxsw_pci_queue_elem_info_get(q, index); |
221 | } | 221 | } |
diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum.c b/drivers/net/ethernet/mellanox/mlxsw/spectrum.c index 09ce451c283b..a94daa8c346c 100644 --- a/drivers/net/ethernet/mellanox/mlxsw/spectrum.c +++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum.c | |||
@@ -2358,9 +2358,7 @@ static int mlxsw_sp_port_lag_leave(struct mlxsw_sp_port *mlxsw_sp_port, | |||
2358 | if (mlxsw_sp_port->bridged) { | 2358 | if (mlxsw_sp_port->bridged) { |
2359 | mlxsw_sp_port_active_vlans_del(mlxsw_sp_port); | 2359 | mlxsw_sp_port_active_vlans_del(mlxsw_sp_port); |
2360 | mlxsw_sp_port_bridge_leave(mlxsw_sp_port, false); | 2360 | mlxsw_sp_port_bridge_leave(mlxsw_sp_port, false); |
2361 | 2361 | mlxsw_sp_master_bridge_dec(mlxsw_sp, NULL); | |
2362 | if (lag->ref_count == 1) | ||
2363 | mlxsw_sp_master_bridge_dec(mlxsw_sp, NULL); | ||
2364 | } | 2362 | } |
2365 | 2363 | ||
2366 | if (lag->ref_count == 1) { | 2364 | if (lag->ref_count == 1) { |