diff options
author | Dan Carpenter <error27@gmail.com> | 2011-08-23 21:29:22 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-08-26 12:55:18 -0400 |
commit | 44861f4455a77beaec2e53459c77b7fb0eda91a7 (patch) | |
tree | 601361470575247574b7b22f95d75e44804206df /drivers/net/ethernet | |
parent | 3756a89f3d039d8a6aa2556f8d57d4b2ec25d5dd (diff) |
bna: unlock on error path in pnad_pci_probe()
We introduced a new lock here, so there was error path which needs
an unlock now.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Rasesh Mody <rmody@brocade.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet')
-rw-r--r-- | drivers/net/ethernet/brocade/bna/bnad.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/ethernet/brocade/bna/bnad.c b/drivers/net/ethernet/brocade/bna/bnad.c index bdfda0779a84..6ad4b477a4ef 100644 --- a/drivers/net/ethernet/brocade/bna/bnad.c +++ b/drivers/net/ethernet/brocade/bna/bnad.c | |||
@@ -3167,7 +3167,7 @@ bnad_pci_probe(struct pci_dev *pdev, | |||
3167 | */ | 3167 | */ |
3168 | err = bnad_pci_init(bnad, pdev, &using_dac); | 3168 | err = bnad_pci_init(bnad, pdev, &using_dac); |
3169 | if (err) | 3169 | if (err) |
3170 | goto free_netdev; | 3170 | goto unlock_mutex; |
3171 | 3171 | ||
3172 | /* | 3172 | /* |
3173 | * Initialize bnad structure | 3173 | * Initialize bnad structure |
@@ -3296,9 +3296,9 @@ drv_uninit: | |||
3296 | bnad_uninit(bnad); | 3296 | bnad_uninit(bnad); |
3297 | pci_uninit: | 3297 | pci_uninit: |
3298 | bnad_pci_uninit(pdev); | 3298 | bnad_pci_uninit(pdev); |
3299 | unlock_mutex: | ||
3299 | mutex_unlock(&bnad->conf_mutex); | 3300 | mutex_unlock(&bnad->conf_mutex); |
3300 | bnad_lock_uninit(bnad); | 3301 | bnad_lock_uninit(bnad); |
3301 | free_netdev: | ||
3302 | free_netdev(netdev); | 3302 | free_netdev(netdev); |
3303 | return err; | 3303 | return err; |
3304 | } | 3304 | } |