Skip to content

Commit

Permalink
wusb: release mutex in the error path of whci-hcd's whc_do_gencmd()
Browse files Browse the repository at this point in the history
Signed-off-by: David Vrabel <david.vrabel@csr.com>
  • Loading branch information
David Vrabel committed Oct 28, 2008
1 parent 6a4b587 commit b09ac64
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions drivers/usb/host/whci/hw.c
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ int whc_do_gencmd(struct whc *whc, u32 cmd, u32 params, void *addr, size_t len)
unsigned long flags;
dma_addr_t dma_addr;
int t;
int ret = 0;

mutex_lock(&whc->mutex);

Expand All @@ -61,7 +62,8 @@ int whc_do_gencmd(struct whc *whc, u32 cmd, u32 params, void *addr, size_t len)
dev_err(&whc->umc->dev, "generic command timeout (%04x/%04x)\n",
le_readl(whc->base + WUSBGENCMDSTS),
le_readl(whc->base + WUSBGENCMDPARAMS));
return -ETIMEDOUT;
ret = -ETIMEDOUT;
goto out;
}

if (addr) {
Expand All @@ -80,8 +82,8 @@ int whc_do_gencmd(struct whc *whc, u32 cmd, u32 params, void *addr, size_t len)
whc->base + WUSBGENCMDSTS);

spin_unlock_irqrestore(&whc->lock, flags);

out:
mutex_unlock(&whc->mutex);

return 0;
return ret;
}

0 comments on commit b09ac64

Please sign in to comment.