Re: [PATCHv7 03/15] v4l: vb2: add support for shared buffer (dma_buf)

2012-06-26 Thread Dima Zavin
Hans and Laurent, Thanks for the feedback. On Tue, Jun 26, 2012 at 2:40 AM, Hans Verkuil wrote: > On Tue 26 June 2012 11:11:06 Laurent Pinchart wrote: >> Hi Dima and Tomasz, >> >> Sorry for the late reply. >> >> On Tuesday 26 June 2012 10:40:44 Tomasz Stani

[PATCHv7 03/15] v4l: vb2: add support for shared buffer (dma_buf)

2012-06-26 Thread Dima Zavin
Hans and Laurent, Thanks for the feedback. On Tue, Jun 26, 2012 at 2:40 AM, Hans Verkuil wrote: > On Tue 26 June 2012 11:11:06 Laurent Pinchart wrote: >> Hi Dima and Tomasz, >> >> Sorry for the late reply. >> >> On Tuesday 26 June 2012 10:40:44 Tomasz Stani

Re: [PATCHv7 03/15] v4l: vb2: add support for shared buffer (dma_buf)

2012-06-26 Thread Dima Zavin
On Tue, Jun 26, 2012 at 1:40 AM, Tomasz Stanislawski wrote: > Hi Dima Zavin, > Thank you for the patch and for a ping remainder :). > > You are right. The unmap is missing in __vb2_queue_cancel. > I will apply your fix into next version of V4L2 support for dmabuf. > >

[PATCHv7 03/15] v4l: vb2: add support for shared buffer (dma_buf)

2012-06-26 Thread Dima Zavin
On Tue, Jun 26, 2012 at 1:40 AM, Tomasz Stanislawski wrote: > Hi Dima Zavin, > Thank you for the patch and for a ping remainder :). > > You are right. The unmap is missing in __vb2_queue_cancel. > I will apply your fix into next version of V4L2 support for dmabuf. > >

Re: [PATCHv7 03/15] v4l: vb2: add support for shared buffer (dma_buf)

2012-06-25 Thread Dima Zavin
ping? On Tue, Jun 19, 2012 at 11:12 PM, Dima Zavin wrote: > Tomasz, > > I've encountered an issue with this patch when userspace does several > stream_on/stream_off cycles. When the user tries to qbuf a buffer > after doing stream_off, we trigger the "dmabuf already p

[PATCHv7 03/15] v4l: vb2: add support for shared buffer (dma_buf)

2012-06-25 Thread Dima Zavin
ping? On Tue, Jun 19, 2012 at 11:12 PM, Dima Zavin wrote: > Tomasz, > > I've encountered an issue with this patch when userspace does several > stream_on/stream_off cycles. When the user tries to qbuf a buffer > after doing stream_off, we trigger the "dmabuf already p

Re: [PATCHv7 03/15] v4l: vb2: add support for shared buffer (dma_buf)

2012-06-20 Thread Dima Zavin
Stanislawski Signed-off-by: Dima Zavin --- drivers/media/video/videobuf2-core.c | 22 -- 1 files changed, 20 insertions(+), 2 deletions(-) diff --git a/drivers/media/video/videobuf2-core.c b/drivers/media/video/videobuf2-core.c index b431dc6..e2a8f12 100644 --- a/drivers/m

[PATCHv7 03/15] v4l: vb2: add support for shared buffer (dma_buf)

2012-06-19 Thread Dima Zavin
Stanislawski Signed-off-by: Dima Zavin --- drivers/media/video/videobuf2-core.c | 22 -- 1 files changed, 20 insertions(+), 2 deletions(-) diff --git a/drivers/media/video/videobuf2-core.c b/drivers/media/video/videobuf2-core.c index b431dc6..e2a8f12 100644 --- a/drivers/m