Re: [RFCv2 API PATCH 14/28] DocBook: clarify that sequence is also set for output devices.
On Thu 13 September 2012 04:28:41 Laurent Pinchart wrote: > Hi Hans, > > Thanks for the patch. > > On Friday 07 September 2012 15:29:14 Hans Verkuil wrote: > > From: Hans Verkuil > > > > It was not entirely obvious that the sequence count should also > > be set for output devices. Also made it more explicit that this > > sequence counter counts frames, not fields. > > > > Signed-off-by: Hans Verkuil > > --- > > Documentation/DocBook/media/v4l/io.xml |4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/Documentation/DocBook/media/v4l/io.xml > > b/Documentation/DocBook/media/v4l/io.xml index b680d66..d1c2369 100644 > > --- a/Documentation/DocBook/media/v4l/io.xml > > +++ b/Documentation/DocBook/media/v4l/io.xml > > @@ -617,8 +617,8 @@ field is independent of the > > timestamp and __u32 > > sequence > > > > - Set by the driver, counting the frames in the > > -sequence. > > + Set by the driver, counting the frames (not fields!) in the > > +sequence. This field is set for both input and output devices. > > Nitpicking, s/in the sequence/in sequence/ ? Will fix. Regards, Hans > > > > > > > In > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [RFCv2 API PATCH 14/28] DocBook: clarify that sequence is also set for output devices.
Hi Hans, Thanks for the patch. On Friday 07 September 2012 15:29:14 Hans Verkuil wrote: > From: Hans Verkuil > > It was not entirely obvious that the sequence count should also > be set for output devices. Also made it more explicit that this > sequence counter counts frames, not fields. > > Signed-off-by: Hans Verkuil > --- > Documentation/DocBook/media/v4l/io.xml |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/Documentation/DocBook/media/v4l/io.xml > b/Documentation/DocBook/media/v4l/io.xml index b680d66..d1c2369 100644 > --- a/Documentation/DocBook/media/v4l/io.xml > +++ b/Documentation/DocBook/media/v4l/io.xml > @@ -617,8 +617,8 @@ field is independent of the > timestamp and __u32 > sequence > > - Set by the driver, counting the frames in the > -sequence. > + Set by the driver, counting the frames (not fields!) in the > +sequence. This field is set for both input and output devices. Nitpicking, s/in the sequence/in sequence/ ? > > > In http://vger.kernel.org/majordomo-info.html
Re: [RFCv2 API PATCH 14/28] DocBook: clarify that sequence is also set for output devices.
On 09/07/2012 03:29 PM, Hans Verkuil wrote: From: Hans Verkuil It was not entirely obvious that the sequence count should also be set for output devices. Also made it more explicit that this sequence counter counts frames, not fields. Signed-off-by: Hans Verkuil Reviewed-by: Sylwester Nawrocki -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[RFCv2 API PATCH 14/28] DocBook: clarify that sequence is also set for output devices.
From: Hans Verkuil It was not entirely obvious that the sequence count should also be set for output devices. Also made it more explicit that this sequence counter counts frames, not fields. Signed-off-by: Hans Verkuil --- Documentation/DocBook/media/v4l/io.xml |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/DocBook/media/v4l/io.xml b/Documentation/DocBook/media/v4l/io.xml index b680d66..d1c2369 100644 --- a/Documentation/DocBook/media/v4l/io.xml +++ b/Documentation/DocBook/media/v4l/io.xml @@ -617,8 +617,8 @@ field is independent of the timestamp and __u32 sequence - Set by the driver, counting the frames in the -sequence. + Set by the driver, counting the frames (not fields!) in the +sequence. This field is set for both input and output devices. In http://vger.kernel.org/majordomo-info.html