diff options
| author |   Dan Carpenter <error27@gmail.com> | 2010-05-26 04:46:35 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2010-05-29 00:18:49 -0700 |
| commit | 2892d9c2d925e0d72a7a529852942e2592a970f8 (patch) | |
| tree | b2ffaafc603f4923ab5b50ff3f3ee7ad3b3d0c0c /drivers/net/benet | |
| parent | net/rds: Add missing mutex_unlock (diff) | |
| download | linux-dev-2892d9c2d925e0d72a7a529852942e2592a970f8.tar.xz linux-dev-2892d9c2d925e0d72a7a529852942e2592a970f8.zip | |
be2net: add unlock on error path
The unlock accidentally got removed from the error path in dd131e76e5:
"be2net: Bug fix to avoid disabling bottom half during firmware upgrade."
Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Sarveshwar Bandi <sarveshwarb@serverengines.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/benet')
| -rw-r--r-- | drivers/net/benet/be_cmds.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/net/benet/be_cmds.c b/drivers/net/benet/be_cmds.c index 9d11dbf5e4da..a4a9cf762441 100644 --- a/drivers/net/benet/be_cmds.c +++ b/drivers/net/benet/be_cmds.c @@ -1429,7 +1429,7 @@ int be_cmd_write_flashrom(struct be_adapter *adapter, struct be_dma_mem *cmd, wrb = wrb_from_mccq(adapter); if (!wrb) { status = -EBUSY; - goto err; + goto err_unlock; } req = cmd->va; sge = nonembedded_sgl(wrb); @@ -1457,7 +1457,10 @@ int be_cmd_write_flashrom(struct be_adapter *adapter, struct be_dma_mem *cmd, else status = adapter->flash_status; -err: + return status; + +err_unlock: + spin_unlock_bh(&adapter->mcc_lock); return status; } |