Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-04 Thread Sakari Ailus
Guennadi Liakhovetski wrote: > On Wed, 3 Aug 2011, Hans Verkuil wrote: > On Thu, 28 Jul 2011, Hans Verkuil wrote: > On Thursday, July 28, 2011 06:11:38 Pawel Osciak wrote: >> Hi Guennadi, >> >> On Wed, Jul 20, 2011 at 01:43, Guennadi Liakhovetski >> wrote: >>> A

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-03 Thread Guennadi Liakhovetski
On Wed, 3 Aug 2011, Hans Verkuil wrote: > >> On Thu, 28 Jul 2011, Hans Verkuil wrote: > >> > >>> On Thursday, July 28, 2011 06:11:38 Pawel Osciak wrote: > >>> > Hi Guennadi, > >>> > > >>> > On Wed, Jul 20, 2011 at 01:43, Guennadi Liakhovetski > >>> > wrote: > >>> > > A possibility to preallocate

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-03 Thread Hans Verkuil
>> On Thu, 28 Jul 2011, Hans Verkuil wrote: >> >>> On Thursday, July 28, 2011 06:11:38 Pawel Osciak wrote: >>> > Hi Guennadi, >>> > >>> > On Wed, Jul 20, 2011 at 01:43, Guennadi Liakhovetski >>> > wrote: >>> > > A possibility to preallocate and initialise buffers of different >>> sizes >>> > > in

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-02 Thread Sakari Ailus
On Wed, Jul 27, 2011 at 09:11:38PM -0700, Pawel Osciak wrote: > Hi Guennadi, Hi Pawel, > On Wed, Jul 20, 2011 at 01:43, Guennadi Liakhovetski > wrote: > > A possibility to preallocate and initialise buffers of different sizes > > in V4L2 is required for an efficient implementation of asnapshot m

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-02 Thread Guennadi Liakhovetski
On Tue, 2 Aug 2011, Sakari Ailus wrote: > Uh, I'm a bit lost now. Do different buffer types (e.g. capture, overlay > and input) need to be taken into account here, and if so, how? My > understanding was this is not related to preparing buffers. They all should be dealt with correctly. If you find

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-02 Thread Sakari Ailus
Guennadi Liakhovetski wrote: > On Tue, 2 Aug 2011, Sakari Ailus wrote: > >> Guennadi Liakhovetski wrote: >>> On Tue, 2 Aug 2011, Sakari Ailus wrote: >>> Guennadi Liakhovetski wrote: > On Mon, 1 Aug 2011, Sakari Ailus wrote: > >> Guennadi Liakhovetski wrote: >>> On Tue, 26 Jul

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-02 Thread Guennadi Liakhovetski
On Tue, 2 Aug 2011, Sakari Ailus wrote: > Guennadi Liakhovetski wrote: > > On Tue, 2 Aug 2011, Sakari Ailus wrote: > > > >> Guennadi Liakhovetski wrote: > >>> On Mon, 1 Aug 2011, Sakari Ailus wrote: > >>> > Guennadi Liakhovetski wrote: > > On Tue, 26 Jul 2011, Hans Verkuil wrote: > >

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-02 Thread Sakari Ailus
Guennadi Liakhovetski wrote: > On Tue, 2 Aug 2011, Sakari Ailus wrote: > >> Guennadi Liakhovetski wrote: >>> On Mon, 1 Aug 2011, Sakari Ailus wrote: >>> Guennadi Liakhovetski wrote: > On Tue, 26 Jul 2011, Hans Verkuil wrote: > >> On Tuesday, July 26, 2011 13:44:28 Sakari Ailus wro

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-02 Thread Hans Verkuil
> On Thu, 28 Jul 2011, Hans Verkuil wrote: > >> On Thursday, July 28, 2011 06:11:38 Pawel Osciak wrote: >> > Hi Guennadi, >> > >> > On Wed, Jul 20, 2011 at 01:43, Guennadi Liakhovetski >> > wrote: >> > > A possibility to preallocate and initialise buffers of different >> sizes >> > > in V4L2 is re

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-02 Thread Guennadi Liakhovetski
On Tue, 2 Aug 2011, Sakari Ailus wrote: > Guennadi Liakhovetski wrote: > > On Mon, 1 Aug 2011, Sakari Ailus wrote: > > > >> Guennadi Liakhovetski wrote: > >>> On Tue, 26 Jul 2011, Hans Verkuil wrote: > >>> > On Tuesday, July 26, 2011 13:44:28 Sakari Ailus wrote: > > Hi Hans and Guennadi,

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-02 Thread Guennadi Liakhovetski
On Thu, 28 Jul 2011, Hans Verkuil wrote: > On Thursday, July 28, 2011 06:11:38 Pawel Osciak wrote: > > Hi Guennadi, > > > > On Wed, Jul 20, 2011 at 01:43, Guennadi Liakhovetski > > wrote: > > > A possibility to preallocate and initialise buffers of different sizes > > > in V4L2 is required for a

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-02 Thread Sakari Ailus
Guennadi Liakhovetski wrote: > On Mon, 1 Aug 2011, Sakari Ailus wrote: > >> Guennadi Liakhovetski wrote: >>> On Tue, 26 Jul 2011, Hans Verkuil wrote: >>> On Tuesday, July 26, 2011 13:44:28 Sakari Ailus wrote: > Hi Hans and Guennadi, >> I realized that it is not clear f

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-01 Thread Guennadi Liakhovetski
On Mon, 1 Aug 2011, Sakari Ailus wrote: > Guennadi Liakhovetski wrote: > > On Tue, 26 Jul 2011, Hans Verkuil wrote: > > > >> On Tuesday, July 26, 2011 13:44:28 Sakari Ailus wrote: > >>> Hi Hans and Guennadi, > >> > >> > >> > I realized that it is not clear from the documentation whether it

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-01 Thread Sakari Ailus
Guennadi Liakhovetski wrote: > On Tue, 26 Jul 2011, Hans Verkuil wrote: > >> On Tuesday, July 26, 2011 13:44:28 Sakari Ailus wrote: >>> Hi Hans and Guennadi, >> >> >> I realized that it is not clear from the documentation whether it is possible to call VIDIOC_REQBUFS and make addi

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-01 Thread Guennadi Liakhovetski
On Sat, 30 Jul 2011, Hans Verkuil wrote: [snip] > No. We do not have a mechanism (yet) to tie a pipeline configuration to > a queued buffer (to be discussed in next week in Cambourne). > > It is my understanding that you change the current streaming format, and > then queue the large (prepared)

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-01 Thread Guennadi Liakhovetski
On Tue, 26 Jul 2011, Hans Verkuil wrote: > On Tuesday, July 26, 2011 13:44:28 Sakari Ailus wrote: > > Hi Hans and Guennadi, > > > > > > I realized that it is not clear from the documentation whether it is > > > possible to call > > > VIDIOC_REQBUFS and make additional calls to VIDIOC_CREATE_BU

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-30 Thread Guennadi Liakhovetski
On Sat, 30 Jul 2011, Hans Verkuil wrote: > On Saturday, July 30, 2011 06:21:37 Pawel Osciak wrote: [snip] > > So when the driver sees a larger buffer being queued, it is > > to change the current streaming format for the duration of filling > > that buffer and switch back afterwards? > > No. We

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-30 Thread Hans Verkuil
On Saturday, July 30, 2011 06:21:37 Pawel Osciak wrote: > Hi Hans, > > On Wed, Jul 27, 2011 at 23:56, Hans Verkuil wrote: > > On Thursday, July 28, 2011 06:11:38 Pawel Osciak wrote: > >> Hi Guennadi, > >> > >> On Wed, Jul 20, 2011 at 01:43, Guennadi Liakhovetski > >> wrote: > >> > A possibility

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-29 Thread Pawel Osciak
Hi Hans, On Wed, Jul 27, 2011 at 23:56, Hans Verkuil wrote: > On Thursday, July 28, 2011 06:11:38 Pawel Osciak wrote: >> Hi Guennadi, >> >> On Wed, Jul 20, 2011 at 01:43, Guennadi Liakhovetski >> wrote: >> > A possibility to preallocate and initialise buffers of different sizes >> > in V4L2 is r

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-29 Thread Sakari Ailus
On Thu, Jul 28, 2011 at 02:42:52PM +0200, Hans Verkuil wrote: > On Thursday, July 28, 2011 14:29:38 Guennadi Liakhovetski wrote: > > On Thu, 28 Jul 2011, Hans Verkuil wrote: > > > > > On Thursday, July 28, 2011 06:11:38 Pawel Osciak wrote: > > > > Hi Guennadi, > > > > > > > > On Wed, Jul 20, 2011

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-28 Thread Hans Verkuil
On Thursday, July 28, 2011 14:29:38 Guennadi Liakhovetski wrote: > On Thu, 28 Jul 2011, Hans Verkuil wrote: > > > On Thursday, July 28, 2011 06:11:38 Pawel Osciak wrote: > > > Hi Guennadi, > > > > > > On Wed, Jul 20, 2011 at 01:43, Guennadi Liakhovetski > > > wrote: > > > > A possibility to prea

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-28 Thread Guennadi Liakhovetski
On Thu, 28 Jul 2011, Hans Verkuil wrote: > On Thursday, July 28, 2011 06:11:38 Pawel Osciak wrote: > > Hi Guennadi, > > > > On Wed, Jul 20, 2011 at 01:43, Guennadi Liakhovetski > > wrote: > > > A possibility to preallocate and initialise buffers of different sizes > > > in V4L2 is required for a

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-28 Thread Guennadi Liakhovetski
Hi Sylwester On Wed, 27 Jul 2011, Sylwester Nawrocki wrote: > Hi Gueannadi, > > On 07/20/2011 10:43 AM, Guennadi Liakhovetski wrote: > > A possibility to preallocate and initialise buffers of different sizes > > in V4L2 is required for an efficient implementation of asnapshot mode. > > This patc

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-28 Thread Guennadi Liakhovetski
On Tue, 26 Jul 2011, Hans Verkuil wrote: [snip] > I am happy with the API. The only thing that's unclear to me is whether you > can call > CREATE_BUFS after REQBUFS. And if not, then why not? It would also be helpful > to see > the full patch series as the last one was from April. It is interes

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-27 Thread Hans Verkuil
On Thursday, July 28, 2011 06:11:38 Pawel Osciak wrote: > Hi Guennadi, > > On Wed, Jul 20, 2011 at 01:43, Guennadi Liakhovetski > wrote: > > A possibility to preallocate and initialise buffers of different sizes > > in V4L2 is required for an efficient implementation of asnapshot mode. > > This p

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-27 Thread Pawel Osciak
Hi Guennadi, On Wed, Jul 20, 2011 at 01:43, Guennadi Liakhovetski wrote: > A possibility to preallocate and initialise buffers of different sizes > in V4L2 is required for an efficient implementation of asnapshot mode. > This patch adds two new ioctl()s: VIDIOC_CREATE_BUFS and > VIDIOC_PREPARE_BU

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-27 Thread Sylwester Nawrocki
Hi Gueannadi, On 07/20/2011 10:43 AM, Guennadi Liakhovetski wrote: > A possibility to preallocate and initialise buffers of different sizes > in V4L2 is required for an efficient implementation of asnapshot mode. > This patch adds two new ioctl()s: VIDIOC_CREATE_BUFS and > VIDIOC_PREPARE_BUF and d

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-26 Thread Hans Verkuil
On Tuesday, July 26, 2011 13:44:28 Sakari Ailus wrote: > Hi Hans and Guennadi, > > I realized that it is not clear from the documentation whether it is > > possible to call > > VIDIOC_REQBUFS and make additional calls to VIDIOC_CREATE_BUFS afterwards. > > That's actually a must if one wants to

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-26 Thread Sakari Ailus
Hi Hans and Guennadi, On Tue, Jul 26, 2011 at 01:05:29PM +0200, Hans Verkuil wrote: > On Wednesday, July 20, 2011 17:19:46 Sakari Ailus wrote: > > On Wed, Jul 20, 2011 at 04:47:46PM +0200, Guennadi Liakhovetski wrote: > > > On Wed, 20 Jul 2011, Sakari Ailus wrote: > > > > > > > Hi, Guennadi! > >

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-26 Thread Hans Verkuil
On Wednesday, July 20, 2011 17:19:46 Sakari Ailus wrote: > On Wed, Jul 20, 2011 at 04:47:46PM +0200, Guennadi Liakhovetski wrote: > > On Wed, 20 Jul 2011, Sakari Ailus wrote: > > > > > Hi, Guennadi! > > > > > > Thanks for the patch! > > > > > > On Wed, Jul 20, 2011 at 10:43:08AM +0200, Guennadi

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-26 Thread Hans Verkuil
Just some typos/documentation improvements: On Wednesday, July 20, 2011 10:43:08 Guennadi Liakhovetski wrote: > A possibility to preallocate and initialise buffers of different sizes > in V4L2 is required for an efficient implementation of asnapshot mode. > This patch adds two new ioctl()s: VIDIOC

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-20 Thread Sakari Ailus
On Wed, Jul 20, 2011 at 04:47:46PM +0200, Guennadi Liakhovetski wrote: > On Wed, 20 Jul 2011, Sakari Ailus wrote: > > > Hi, Guennadi! > > > > Thanks for the patch! > > > > On Wed, Jul 20, 2011 at 10:43:08AM +0200, Guennadi Liakhovetski wrote: > > > A possibility to preallocate and initialise buf

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-20 Thread Guennadi Liakhovetski
On Wed, 20 Jul 2011, Sakari Ailus wrote: > Hi, Guennadi! > > Thanks for the patch! > > On Wed, Jul 20, 2011 at 10:43:08AM +0200, Guennadi Liakhovetski wrote: > > A possibility to preallocate and initialise buffers of different sizes > > in V4L2 is required for an efficient implementation of asna

Re: [PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-20 Thread Sakari Ailus
Hi, Guennadi! Thanks for the patch! On Wed, Jul 20, 2011 at 10:43:08AM +0200, Guennadi Liakhovetski wrote: > A possibility to preallocate and initialise buffers of different sizes > in V4L2 is required for an efficient implementation of asnapshot mode. > This patch adds two new ioctl()s: VIDIOC_C

[PATCH v3] V4L: add two new ioctl()s for multi-size videobuffer management

2011-07-20 Thread Guennadi Liakhovetski
A possibility to preallocate and initialise buffers of different sizes in V4L2 is required for an efficient implementation of asnapshot mode. This patch adds two new ioctl()s: VIDIOC_CREATE_BUFS and VIDIOC_PREPARE_BUF and defines respective data structures. Signed-off-by: Guennadi Liakhovetski --