Hi Javier On Mon, 23 Jan 2012, javier Martin wrote: > Hi Guennadi, > thank you for your attention. > > I've recently sent a new patch series on top of this patch: > [PATCH 0/4] media i.MX27 camera: fix buffer handling and videobuf2 > support. (http://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/msg42255.html) > > Among other things, it adds videobuf2 support and adds "stream_stop" > and "stream_start" callbacks which allow to enable/disable capturing > of buffers at the right moment. > This also makes the sequence number trick disappear and a much cleaner > approach is used instead. > > I suggest you hold on this patch until the new series is accepted and > then merge both at the same time. > > What do you think? Ok, I'll be reviewing that patch series hopefully soon, and in principle it is good, that the buffer counting will really be fixed in it, but in an ideal world it would be better to have this your patch merged into patch 2/4 of the series, agree? Would I be asking too much of you if I suggest that? Feel free to explain why this wouldn't work or just reject if you're just too tight on schedule. I'll see ifI can swallow it that way or maybe merge myself :-) Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/ -- 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
