Browse Source

cxgb3: Remove superfluous call to pci_disable_msix()

There is no need to call pci_disable_msix() in case
the previous call to pci_enable_msix() failed

Signed-off-by: Alexander Gordeev <agordeev@redhat.com>
Cc: Santosh Raspatur <santosh@chelsio.com>
Cc: netdev@vger.kernel.org
Cc: linux-pci@vger.kernel.org
Signed-off-by: David S. Miller <davem@davemloft.net>
Alexander Gordeev 11 years ago
parent
commit
c0b2551681
1 changed files with 0 additions and 3 deletions
  1. 0 3
      drivers/net/ethernet/chelsio/cxgb3/cxgb3_main.c

+ 0 - 3
drivers/net/ethernet/chelsio/cxgb3/cxgb3_main.c

@@ -3097,9 +3097,6 @@ static int cxgb_enable_msix(struct adapter *adap)
 	while ((err = pci_enable_msix(adap->pdev, entries, vectors)) > 0)
 	while ((err = pci_enable_msix(adap->pdev, entries, vectors)) > 0)
 		vectors = err;
 		vectors = err;
 
 
-	if (err < 0)
-		pci_disable_msix(adap->pdev);
-
 	if (!err && vectors < (adap->params.nports + 1)) {
 	if (!err && vectors < (adap->params.nports + 1)) {
 		pci_disable_msix(adap->pdev);
 		pci_disable_msix(adap->pdev);
 		err = -1;
 		err = -1;