diff options
author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2013-03-11 00:40:14 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-03-12 08:04:08 -0400 |
commit | 3a918f4036f5c4689a091c2b9affcca2066803ee (patch) | |
tree | 303f693555b309c07291e262ba8bed616878678d /drivers/net/ethernet/broadcom/bnx2x | |
parent | 2e3a118f82a64f55a5cd0b44b6c00a884cad4ff9 (diff) |
bnx2x: use list_move instead of list_del/list_add
Using list_move() instead of list_del() + list_add().
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Acked-by: Ariel Elior <ariele@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/broadcom/bnx2x')
-rw-r--r-- | drivers/net/ethernet/broadcom/bnx2x/bnx2x_sriov.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sriov.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sriov.c index df930e30e1b1..faadd153f7ad 100644 --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sriov.c +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sriov.c | |||
@@ -557,8 +557,7 @@ static int bnx2x_vfop_config_list(struct bnx2x *bp, | |||
557 | rc = bnx2x_config_vlan_mac(bp, vlan_mac); | 557 | rc = bnx2x_config_vlan_mac(bp, vlan_mac); |
558 | if (rc >= 0) { | 558 | if (rc >= 0) { |
559 | cnt += pos->add ? 1 : -1; | 559 | cnt += pos->add ? 1 : -1; |
560 | list_del(&pos->link); | 560 | list_move(&pos->link, &rollback_list); |
561 | list_add(&pos->link, &rollback_list); | ||
562 | rc = 0; | 561 | rc = 0; |
563 | } else if (rc == -EEXIST) { | 562 | } else if (rc == -EEXIST) { |
564 | rc = 0; | 563 | rc = 0; |