On 10/06/11 07:17, Dan Carpenter wrote:
> The error_ret label should have been before the mutex_unlock().  It's
> a typo.
Good catch thanks!
> 
> Signed-off-by: Dan Carpenter <[email protected]>
Acked-by: Jonathan Cameron <[email protected]>
> 
> 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
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to