diff options
| author | Daniel Ribeiro <drwyrm@gmail.com> | 2009-06-23 12:30:58 -0300 |
|---|---|---|
| committer | Samuel Ortiz <sameol@linux.intel.com> | 2009-07-03 11:54:57 +0200 |
| commit | 1c90ea2c7eb3b24a07a2f82164323588fb029bc1 (patch) | |
| tree | 0dea0df7b9e6833c0e1e057a5085404ead6401aa | |
| parent | 992bb253cd6f08129edcb42b90e6c388ebf605f8 (diff) | |
| download | op-kernel-dev-1c90ea2c7eb3b24a07a2f82164323588fb029bc1.zip op-kernel-dev-1c90ea2c7eb3b24a07a2f82164323588fb029bc1.tar.gz | |
mfd: fix pcap adc locking
Release the lock on error.
Signed-off-by: Daniel Ribeiro <drwyrm@gmail.com>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
| -rw-r--r-- | drivers/mfd/ezx-pcap.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/mfd/ezx-pcap.c b/drivers/mfd/ezx-pcap.c index 671a7ef..c1de4af 100644 --- a/drivers/mfd/ezx-pcap.c +++ b/drivers/mfd/ezx-pcap.c @@ -238,8 +238,10 @@ static irqreturn_t pcap_adc_irq(int irq, void *_pcap) mutex_lock(&pcap->adc_mutex); req = pcap->adc_queue[pcap->adc_head]; - if (WARN(!req, KERN_WARNING "adc irq without pending request\n")) + if (WARN(!req, KERN_WARNING "adc irq without pending request\n")) { + mutex_unlock(&pcap->adc_mutex); return IRQ_HANDLED; + } /* read requested channels results */ ezx_pcap_read(pcap, PCAP_REG_ADC, &tmp); |
