diff options
author | Dan Carpenter <error27@gmail.com> | 2011-07-30 04:45:09 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2011-08-02 19:07:51 -0400 |
commit | 8f89615528b11eabda68faaf2438d09c9565a125 (patch) | |
tree | 9fe0e6b804aedbc05eff1f176c3e9ba73c9d9b22 /drivers/staging/iio/accel/adis16203_core.c | |
parent | c060ae7db4ebf1e5e08199b2aec12f051189041a (diff) |
Staging: iio: add some unlocks to raw_read() functions
This code was cut and pasted in several places. It's missing some
unlocks on error.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/iio/accel/adis16203_core.c')
-rw-r--r-- | drivers/staging/iio/accel/adis16203_core.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/drivers/staging/iio/accel/adis16203_core.c b/drivers/staging/iio/accel/adis16203_core.c index bf1988884e9..cf5d15da76a 100644 --- a/drivers/staging/iio/accel/adis16203_core.c +++ b/drivers/staging/iio/accel/adis16203_core.c | |||
@@ -311,13 +311,17 @@ static int adis16203_read_raw(struct iio_dev *indio_dev, | |||
311 | mutex_lock(&indio_dev->mlock); | 311 | mutex_lock(&indio_dev->mlock); |
312 | addr = adis16203_addresses[chan->address][0]; | 312 | addr = adis16203_addresses[chan->address][0]; |
313 | ret = adis16203_spi_read_reg_16(indio_dev, addr, &val16); | 313 | ret = adis16203_spi_read_reg_16(indio_dev, addr, &val16); |
314 | if (ret) | 314 | if (ret) { |
315 | mutex_unlock(&indio_dev->mlock); | ||
315 | return ret; | 316 | return ret; |
317 | } | ||
316 | 318 | ||
317 | if (val16 & ADIS16203_ERROR_ACTIVE) { | 319 | if (val16 & ADIS16203_ERROR_ACTIVE) { |
318 | ret = adis16203_check_status(indio_dev); | 320 | ret = adis16203_check_status(indio_dev); |
319 | if (ret) | 321 | if (ret) { |
322 | mutex_unlock(&indio_dev->mlock); | ||
320 | return ret; | 323 | return ret; |
324 | } | ||
321 | } | 325 | } |
322 | val16 = val16 & ((1 << chan->scan_type.realbits) - 1); | 326 | val16 = val16 & ((1 << chan->scan_type.realbits) - 1); |
323 | if (chan->scan_type.sign == 's') | 327 | if (chan->scan_type.sign == 's') |