Commit 482d35c4 authored by Jonathan Corbet's avatar Jonathan Corbet Committed by Mauro Carvalho Chehab

[media] marvell-cam: Increase the DMA shutdown timeout

Experience shows that, on the Armada platform, it can take as much as 120ms
for the DMA engine to actually shut down after it has been told to.  So a
40ms timeout is not adequate; use 150ms instead.  Also make sure we don't
leave the DMA_ACTIVE flag set once things are down.
Signed-off-by: default avatarJonathan Corbet <corbet@lwn.net>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent 0770d07f
...@@ -742,7 +742,14 @@ static void mcam_ctlr_stop_dma(struct mcam_camera *cam) ...@@ -742,7 +742,14 @@ static void mcam_ctlr_stop_dma(struct mcam_camera *cam)
mcam_ctlr_stop(cam); mcam_ctlr_stop(cam);
cam->state = S_IDLE; cam->state = S_IDLE;
spin_unlock_irqrestore(&cam->dev_lock, flags); spin_unlock_irqrestore(&cam->dev_lock, flags);
msleep(40); /*
* This is a brutally long sleep, but experience shows that
* it can take the controller a while to get the message that
* it needs to stop grabbing frames. In particular, we can
* sometimes (on mmp) get a frame at the end WITHOUT the
* start-of-frame indication.
*/
msleep(150);
if (test_bit(CF_DMA_ACTIVE, &cam->flags)) if (test_bit(CF_DMA_ACTIVE, &cam->flags))
cam_err(cam, "Timeout waiting for DMA to end\n"); cam_err(cam, "Timeout waiting for DMA to end\n");
/* This would be bad news - what now? */ /* This would be bad news - what now? */
...@@ -885,6 +892,7 @@ static int mcam_read_setup(struct mcam_camera *cam) ...@@ -885,6 +892,7 @@ static int mcam_read_setup(struct mcam_camera *cam)
* Turn it loose. * Turn it loose.
*/ */
spin_lock_irqsave(&cam->dev_lock, flags); spin_lock_irqsave(&cam->dev_lock, flags);
clear_bit(CF_DMA_ACTIVE, &cam->flags);
mcam_reset_buffers(cam); mcam_reset_buffers(cam);
mcam_ctlr_irq_enable(cam); mcam_ctlr_irq_enable(cam);
cam->state = S_STREAMING; cam->state = S_STREAMING;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment