RE: [PATCH v4 02/14] Documentation: media: description of DMABUF importing in V4L2

2012-04-23 Thread Marek Szyprowski
Hi Mauro, On Friday, April 20, 2012 3:37 PM Mauro Carvalho Chehab wrote: (snipped) So my rough-idea was to remove USERPTR support from kernel drivers (if possible of course) and to provide an emulation layer in the userspace code like libv4l2. Please note that it is only a rough idea.

Re: [PATCH v4 02/14] Documentation: media: description of DMABUF importing in V4L2

2012-04-23 Thread Mauro Carvalho Chehab
Em 23-04-2012 07:50, Marek Szyprowski escreveu: Hi Mauro, On Friday, April 20, 2012 3:37 PM Mauro Carvalho Chehab wrote: (snipped) So my rough-idea was to remove USERPTR support from kernel drivers (if possible of course) and to provide an emulation layer in the userspace code like

Re: [PATCH v4 02/14] Documentation: media: description of DMABUF importing in V4L2

2012-04-21 Thread Laurent Pinchart
Hi Rémi, On Friday 20 April 2012 15:03:17 Rémi Denis-Courmont wrote: On Fri, 20 Apr 2012 14:25:01 +0200, Tomasz Stanislawski wrote: The USERPTR simplifies userspace code but introduce a lot of complexity problems for the kernel drivers and frameworks. It is not only a simplification.

Re: [PATCH v4 02/14] Documentation: media: description of DMABUF importing in V4L2

2012-04-20 Thread Mauro Carvalho Chehab
Em 19-04-2012 11:32, Tomasz Stanislawski escreveu: On 04/17/2012 01:25 AM, Laurent Pinchart wrote: Hi Tomasz, Thanks for the patch. On Friday 13 April 2012 17:47:44 Tomasz Stanislawski wrote: This patch adds description and usage examples for importing DMABUF file descriptor in V4L2.

Re: [PATCH v4 02/14] Documentation: media: description of DMABUF importing in V4L2

2012-04-20 Thread Mauro Carvalho Chehab
Em 19-04-2012 11:32, Tomasz Stanislawski escreveu: Hi Laurent, One may find similar sentences in MMAP, USERPTR and DMABUF. Maybe the common parts like description of STREAMON/OFF, QBUF/DQBUF shuffling should be moved to separate section like Streaming :). Maybe it is worth to introduce

Re: [PATCH v4 02/14] Documentation: media: description of DMABUF importing in V4L2

2012-04-20 Thread Tomasz Stanislawski
Hi Mauro, On 04/19/2012 10:37 PM, Mauro Carvalho Chehab wrote: Em 19-04-2012 11:32, Tomasz Stanislawski escreveu: Hi Laurent, One may find similar sentences in MMAP, USERPTR and DMABUF. Maybe the common parts like description of STREAMON/OFF, QBUF/DQBUF shuffling should be moved to

Re: [PATCH v4 02/14] Documentation: media: description of DMABUF importing in V4L2

2012-04-20 Thread Tomasz Stanislawski
Hi Remi, On 04/20/2012 12:56 PM, Rémi Denis-Courmont wrote: On Fri, 20 Apr 2012 10:41:37 +0200, Tomasz Stanislawski t.stanisl...@samsung.com wrote: Am I understanding wrong or are you saying that you want to drop userptr from V4L2 API in long-term? If so, why? Dropping userptr is just some

Re: [PATCH v4 02/14] Documentation: media: description of DMABUF importing in V4L2

2012-04-20 Thread Rémi Denis-Courmont
On Fri, 20 Apr 2012 10:41:37 +0200, Tomasz Stanislawski t.stanisl...@samsung.com wrote: Am I understanding wrong or are you saying that you want to drop userptr from V4L2 API in long-term? If so, why? Dropping userptr is just some brainstorming idea. It was found out that userptr is not a

Re: [PATCH v4 02/14] Documentation: media: description of DMABUF importing in V4L2

2012-04-20 Thread Rémi Denis-Courmont
On Fri, 20 Apr 2012 14:25:01 +0200, Tomasz Stanislawski t.stanisl...@samsung.com wrote: The USERPTR simplifies userspace code but introduce a lot of complexity problems for the kernel drivers and frameworks. It is not only a simplification. In some cases, USERPTR is the only I/O method that

Re: [PATCH v4 02/14] Documentation: media: description of DMABUF importing in V4L2

2012-04-20 Thread Mauro Carvalho Chehab
Em 20-04-2012 07:56, Rémi Denis-Courmont escreveu: On Fri, 20 Apr 2012 10:41:37 +0200, Tomasz Stanislawski t.stanisl...@samsung.com wrote: Am I understanding wrong or are you saying that you want to drop userptr from V4L2 API in long-term? If so, why? Dropping userptr is just some

Re: [PATCH v4 02/14] Documentation: media: description of DMABUF importing in V4L2

2012-04-20 Thread Mauro Carvalho Chehab
Em 20-04-2012 09:25, Tomasz Stanislawski escreveu: Hi Remi, Now, I do realize that some devices cannot support USERPTR efficiently, then they should not support USERPTR. The problem is not there is *NO* device that can handle USERPTR reliably. The can handle USERPTR generated by malloc or

Re: [PATCH v4 02/14] Documentation: media: description of DMABUF importing in V4L2

2012-04-19 Thread Tomasz Stanislawski
On 04/17/2012 01:25 AM, Laurent Pinchart wrote: Hi Tomasz, Thanks for the patch. On Friday 13 April 2012 17:47:44 Tomasz Stanislawski wrote: This patch adds description and usage examples for importing DMABUF file descriptor in V4L2. Signed-off-by: Tomasz Stanislawski

Re: [PATCH v4 02/14] Documentation: media: description of DMABUF importing in V4L2

2012-04-16 Thread Laurent Pinchart
Hi Tomasz, Thanks for the patch. On Friday 13 April 2012 17:47:44 Tomasz Stanislawski wrote: This patch adds description and usage examples for importing DMABUF file descriptor in V4L2. Signed-off-by: Tomasz Stanislawski t.stanisl...@samsung.com Signed-off-by: Kyungmin Park