The error_ret label should have been before the mutex_unlock(). It's a typo. Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> diff --git a/drivers/staging/iio/adc/ad7150.c b/drivers/staging/iio/adc/ad7150.c index 8943420..d1767d8 100644 --- a/drivers/staging/iio/adc/ad7150.c +++ b/drivers/staging/iio/adc/ad7150.c @@ -332,8 +332,8 @@ static int ad7150_write_event_value(struct iio_dev *indio_dev, /* write back if active */ ret = ad7150_write_event_params(indio_dev, event_code); - mutex_unlock(&chip->state_lock); error_ret: + mutex_unlock(&chip->state_lock); return ret; } _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel