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; } -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html