Hi Hans, Thanks for your comments. On Mon, Feb 17, 2014 at 09:54:53AM +0100, Hans Verkuil wrote: ... > > @@ -1119,6 +1113,31 @@ in which case caches have not been used.</entry> > > <entry>The CAPTURE buffer timestamp has been taken from the > > corresponding OUTPUT buffer. This flag applies only to mem2mem devices.</entry> > > </row> > > + <row> > > + <entry><constant>V4L2_BUF_FLAG_TSTAMP_SRC_MASK</constant></entry> > > + <entry>0x00070000</entry> > > + <entry>Mask for timestamp sources below. The timestamp source > > + defines the point of time the timestamp is taken in relation to > > + the frame. Logical and operation between the > > + <structfield>flags</structfield> field and > > + <constant>V4L2_BUF_FLAG_TSTAMP_SRC_MASK</constant> produces the > > + value of the timestamp source.</entry> > > + </row> > > + <row> > > + <entry><constant>V4L2_BUF_FLAG_TSTAMP_SRC_EOF</constant></entry> > > + <entry>0x00000000</entry> > > + <entry>"End of frame." The buffer timestamp has been taken > > More a typographical thing than anything else: I prefer this: > > "End Of Frame": the buffer... > > The capitalization links back to the EOF abbreviation more directly. Fixed, same for the similar one below. > > + when the last pixel of the frame has been received or the > > I would say: "after the last pixel of the frame has been received or after the" > > The "when" word suggests that it is exactly "when", which is not true in > practice. That's the intent nonetheless: to take the timestamp at the end of the frame, not an unspecified time after the event has taken place. I'd rather add a note that there's a level of impreciseness in taking the timestamp, such as: "In practice, software generated timestamp will typically be read from the clock a small amount of time after the last pixel has been received, depending on the system and other activity in it." That would probably be best put somewhere else in the document, though. -- Kind regards, Sakari Ailus e-mail: sakari.ailus@xxxxxx XMPP: sailus@xxxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
