Hi Laurent, On Fri, Mar 02, 2012 at 11:49:02AM +0100, Laurent Pinchart wrote: > When propagating the frame number through the pipeline, the frame number > must be incremented at frame start by the appropriate IRQ handler. This > was properly handled for the CSI2 and CCP2 receivers, but not when the > CCDC parallel interface is used. > > ADD frame number incrementation to the HS/VS interrupt handler. As the > HS/VS interrupt is also generated for frames received by the CSI2 and > CCP2 receivers, remove explicit propagation handling from the serial > receivers. > > Reported-by: Kruno Mrak <kruno.mrak@xxxxxxxxxxxxxxxx> > Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > drivers/media/video/omap3isp/isp.c | 8 -------- > drivers/media/video/omap3isp/ispccdc.c | 3 +++ > drivers/media/video/omap3isp/ispccp2.c | 23 ----------------------- > drivers/media/video/omap3isp/ispcsi2.c | 20 +++----------------- > drivers/media/video/omap3isp/ispcsi2.h | 1 - > 5 files changed, 6 insertions(+), 49 deletions(-) Thanks for the patch, Laurent! Also, this patch simplifies frame numbering a lot. Acked-by: Sakari Ailus <sakari.ailus@xxxxxx> Tested-by: Sakari Ailus <sakari.ailus@xxxxxx> Using CSI-2 receiver writing straight to memory, that is. There's a slight dependency to my patches; are you going to submit this first or how shall we proceed? No conflicts but there's some fuzz nonetheless. Regards, -- Sakari Ailus e-mail: sakari.ailus@xxxxxx jabber/XMPP/Gmail: sailus@xxxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html