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<[email protected]>
Signed-off-by: Sungchun Kang<[email protected]>
---
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
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to