Re: [PATCH] V4L: mt9m032: fix two dead-locks

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

Hi Guennadi,

Thanks for the patch.

On Sunday 08 April 2012 22:31:24 Guennadi Liakhovetski wrote:
> Fix a copy-paste typo and a nested locking function call in mt9m032.
> 
> Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

and applied to my tree.

> ---
>  drivers/media/video/mt9m032.c |    5 +++--
>  1 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/media/video/mt9m032.c b/drivers/media/video/mt9m032.c
> index 7636672..645973c 100644
> --- a/drivers/media/video/mt9m032.c
> +++ b/drivers/media/video/mt9m032.c
> @@ -392,10 +392,11 @@ static int mt9m032_set_pad_format(struct v4l2_subdev
> *subdev, }
> 
>  	/* Scaling is not supported, the format is thus fixed. */
> -	ret = mt9m032_get_pad_format(subdev, fh, fmt);
> +	fmt->format = *__mt9m032_get_pad_format(sensor, fh, fmt->which);
> +	ret = 0;
> 
>  done:
> -	mutex_lock(&sensor->lock);
> +	mutex_unlock(&sensor->lock);
>  	return ret;
>  }
-- 
Regards,

Laurent Pinchart