Re: [PATCH 2/3] v4l: events: Define frame start event

2011-07-29 Thread Sakari Ailus
On Fri, Jul 29, 2011 at 11:57:17AM +0200, Laurent Pinchart wrote: > On Friday 29 July 2011 11:54:03 Sakari Ailus wrote: > > On Fri, Jul 29, 2011 at 11:38:16AM +0200, Laurent Pinchart wrote: > > > On Friday 29 July 2011 09:44:46 Sakari Ailus wrote: > > > > On Thu, Jul 28, 2011 at 10:36:57PM +0200, L

Re: [PATCH 2/3] v4l: events: Define frame start event

2011-07-29 Thread Laurent Pinchart
On Friday 29 July 2011 11:54:03 Sakari Ailus wrote: > On Fri, Jul 29, 2011 at 11:38:16AM +0200, Laurent Pinchart wrote: > > On Friday 29 July 2011 09:44:46 Sakari Ailus wrote: > > > On Thu, Jul 28, 2011 at 10:36:57PM +0200, Laurent Pinchart wrote: > > > > On Thursday 28 July 2011 22:28:57 Sakari Ai

Re: [PATCH 2/3] v4l: events: Define frame start event

2011-07-29 Thread Sakari Ailus
On Fri, Jul 29, 2011 at 11:38:16AM +0200, Laurent Pinchart wrote: > Hi Sakari, > > On Friday 29 July 2011 09:44:46 Sakari Ailus wrote: > > On Thu, Jul 28, 2011 at 10:36:57PM +0200, Laurent Pinchart wrote: > > > On Thursday 28 July 2011 22:28:57 Sakari Ailus wrote: > > > > On Thu, Jul 28, 2011 at 0

Re: [PATCH 2/3] v4l: events: Define frame start event

2011-07-29 Thread Laurent Pinchart
Hi Sakari, On Friday 29 July 2011 09:44:46 Sakari Ailus wrote: > On Thu, Jul 28, 2011 at 10:36:57PM +0200, Laurent Pinchart wrote: > > On Thursday 28 July 2011 22:28:57 Sakari Ailus wrote: > > > On Thu, Jul 28, 2011 at 01:52:21PM +0200, Laurent Pinchart wrote: > > > > On Tuesday 26 July 2011 20:49

Re: [PATCH 2/3] v4l: events: Define frame start event

2011-07-29 Thread Sakari Ailus
On Thu, Jul 28, 2011 at 10:36:57PM +0200, Laurent Pinchart wrote: > Hi Sakari, Hi, Laurent! > On Thursday 28 July 2011 22:28:57 Sakari Ailus wrote: > > On Thu, Jul 28, 2011 at 01:52:21PM +0200, Laurent Pinchart wrote: > > > On Tuesday 26 July 2011 20:49:43 Sakari Ailus wrote: > > [snip] > > > >

Re: [PATCH 2/3] v4l: events: Define frame start event

2011-07-28 Thread Laurent Pinchart
Hi Sakari, On Thursday 28 July 2011 22:28:57 Sakari Ailus wrote: > On Thu, Jul 28, 2011 at 01:52:21PM +0200, Laurent Pinchart wrote: > > On Tuesday 26 July 2011 20:49:43 Sakari Ailus wrote: [snip] > > > + > > > + struct > > > v4l2_event_frame_sync + > > cols="3"> > > > + &cs-str;

Re: [PATCH 2/3] v4l: events: Define frame start event

2011-07-28 Thread Sakari Ailus
On Thu, Jul 28, 2011 at 01:52:21PM +0200, Laurent Pinchart wrote: > Hi Sakari, Hi Laurent, Thanks for the comments! > On Tuesday 26 July 2011 20:49:43 Sakari Ailus wrote: > > Define a frame start event to tell user space when the reception of a frame > > starts. > > You might want to rename 'fr

Re: [PATCH 2/3] v4l: events: Define frame start event

2011-07-28 Thread Laurent Pinchart
Hi Sakari, On Tuesday 26 July 2011 20:49:43 Sakari Ailus wrote: > Define a frame start event to tell user space when the reception of a frame > starts. You might want to rename 'frame start' to 'frame sync' in the subject and commit message as well. > Signed-off-by: Sakari Ailus > --- > Docum

[PATCH 2/3] v4l: events: Define frame start event

2011-07-26 Thread Sakari Ailus
Define a frame start event to tell user space when the reception of a frame starts. Signed-off-by: Sakari Ailus --- Documentation/DocBook/media/v4l/vidioc-dqevent.xml | 23 .../DocBook/media/v4l/vidioc-subscribe-event.xml | 18 +++ include/linux/videodev2.h