[media] omap3isp: ccdc: Restart the CCDC immediately after an underrun in BT.656
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Tue, 10 Jun 2014 13:28:48 +0000 (10:28 -0300)
committerMauro Carvalho Chehab <m.chehab@samsung.com>
Thu, 21 Aug 2014 20:25:17 +0000 (15:25 -0500)
As the CCDC doesn't generate interrupts when stopped in BT.656 mode,
restart it immediately when the next buffer after an underrun is queued
instead of relying on the interrupt handler to restart the CCDC.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Tested-by: Enrico Butera <ebutera@users.sourceforge.net>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
drivers/media/platform/omap3isp/ispccdc.c

index ec0a0e884c79ad48e05eddcad9f794e18b146b5a..cabf46b4b64549b6b7377389e418bfbaef78e3a1 100644 (file)
@@ -1789,6 +1789,8 @@ int omap3isp_ccdc_isr(struct isp_ccdc_device *ccdc, u32 events)
 static int ccdc_video_queue(struct isp_video *video, struct isp_buffer *buffer)
 {
        struct isp_ccdc_device *ccdc = &video->isp->isp_ccdc;
+       unsigned long flags;
+       bool restart = false;
 
        if (!(ccdc->output & CCDC_OUTPUT_MEMORY))
                return -ENODEV;
@@ -1797,9 +1799,20 @@ static int ccdc_video_queue(struct isp_video *video, struct isp_buffer *buffer)
 
        /* We now have a buffer queued on the output, restart the pipeline
         * on the next CCDC interrupt if running in continuous mode (or when
-        * starting the stream).
+        * starting the stream) in external sync mode, or immediately in BT.656
+        * sync mode as no CCDC interrupt is generated when the CCDC is stopped
+        * in that case.
         */
-       ccdc->underrun = 1;
+       spin_lock_irqsave(&ccdc->lock, flags);
+       if (ccdc->state == ISP_PIPELINE_STREAM_CONTINUOUS && !ccdc->running &&
+           ccdc->bt656)
+               restart = 1;
+       else
+               ccdc->underrun = 1;
+       spin_unlock_irqrestore(&ccdc->lock, flags);
+
+       if (restart)
+               ccdc_enable(ccdc);
 
        return 0;
 }