Re: [PATCH v2] media i.MX27 camera: properly detect frame loss.

2012-01-23 Thread javier Martin
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@vger.kernel.org/msg42255.html) Among other things, it adds videobuf2 support

Re: [PATCH v2] media i.MX27 camera: properly detect frame loss.

2012-01-23 Thread Guennadi Liakhovetski
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.

Re: [PATCH v2] media i.MX27 camera: properly detect frame loss.

2012-01-23 Thread javier Martin
Hi Guennadi, 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

Re: [PATCH v2] media i.MX27 camera: properly detect frame loss.

2012-01-22 Thread Guennadi Liakhovetski
A small addendum On Sun, 22 Jan 2012, Guennadi Liakhovetski wrote: Hi Javier Please, excuse my curiosity and bear with my lack of understanding :-) On Wed, 11 Jan 2012, Javier Martin wrote: As V4L2 specification states, frame_count must also regard lost frames so that the user can

Re: [PATCH v2] media i.MX27 camera: properly detect frame loss.

2012-01-21 Thread Guennadi Liakhovetski
Hi Javier Please, excuse my curiosity and bear with my lack of understanding :-) On Wed, 11 Jan 2012, Javier Martin wrote: As V4L2 specification states, frame_count must also regard lost frames so that the user can handle that case properly. This patch adds a mechanism to increment the