Hi Sakari, On Friday 02 March 2012 19:58:59 Sakari Ailus wrote: > 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. I'd like to submit the patch for v3.4. If your patch set is ready for v3.4 as well, we can push both through the same tree. -- Regards, Laurent Pinchart -- 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
