Re: [PATCH] [media] s5p-fimc: fimc_stop_capture bug fix

[Sylwester Nawrocki <snjw23@gmail.com>]

On 12/30/2010 06:35 AM, Sungchun Kang wrote:
> When is called fimc_stop_capture, it seems that wait_event_timeout
> used improperly. It should be wake up by irq handler.
>
> Reviewed-by Jonghun Han<jonghun.han@samsung.com>
> Signed-off-by: Sungchun Kang<sungchun.kang@samsung.com>
> ---
> This patch is depended on:
> http://git.infradead.org/users/kmpark/linux-2.6-samsung/shortlog/refs/heads/vb2-mfc-fimc
>
>   drivers/media/video/s5p-fimc/fimc-capture.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/media/video/s5p-fimc/fimc-capture.c b/drivers/media/video/s5p-fimc/fimc-capture.c
> index 4e4441f..821f927 100644
> --- a/drivers/media/video/s5p-fimc/fimc-capture.c
> +++ b/drivers/media/video/s5p-fimc/fimc-capture.c
> @@ -187,7 +187,7 @@ static int fimc_stop_capture(struct fimc_dev *fimc)
>   	spin_unlock_irqrestore(&fimc->slock, flags);
>
>   	wait_event_timeout(fimc->irq_queue,
> -			   test_bit(ST_CAPT_SHUT,&fimc->state),

just wondering, how this one sneaked in...it looks like a need to take
a long holiday.. ;)

Starting from the new year I am going to setup a fixed branch
v4l/fimc-for-next at git://git.infradead.org/users/kmpark/linux-2.6-samsung
so there is a common up to date tree available.
Until then please base your work on
git://git.infradead.org/users/kmpark/linux-2.6-samsung vb2

I will apply this patch in the next week.

> +			   !test_bit(ST_CAPT_SHUT,&fimc->state),
>   			   FIMC_SHUTDOWN_TIMEOUT);
>
>   	ret = v4l2_subdev_call(cap->sd, video, s_stream, 0);

Thanks,
Sylwester