On 02/24/2014 04:34 PM, Sakari Ailus wrote: > Hans Verkuil wrote: >> On 02/15/2014 09:53 PM, Sakari Ailus wrote: >>> The buffer flags field is 32 bits but the defined only used 16. This is >>> fine, but as more than 16 bits will be used in the very near future, define >>> them as 32-bit numbers for consistency. >>> >>> Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxx> >>> --- >>> Documentation/DocBook/media/v4l/io.xml | 30 ++++++++++++------------- >>> include/uapi/linux/videodev2.h | 38 +++++++++++++++++++------------- >>> 2 files changed, 38 insertions(+), 30 deletions(-) >>> >>> diff --git a/Documentation/DocBook/media/v4l/io.xml b/Documentation/DocBook/media/v4l/io.xml >>> index 8facac4..46d24b3 100644 >>> --- a/Documentation/DocBook/media/v4l/io.xml >>> +++ b/Documentation/DocBook/media/v4l/io.xml >> >> <snip> >> >>> @@ -1115,7 +1115,7 @@ in which case caches have not been used.</entry> >>> </row> >>> <row> >>> <entry><constant>V4L2_BUF_FLAG_TIMESTAMP_COPY</constant></entry> >>> - <entry>0x4000</entry> >>> + <entry>0x00004000</entry> >>> <entry>The CAPTURE buffer timestamp has been taken from the >>> corresponding OUTPUT buffer. This flag applies only to mem2mem devices.</entry> >>> </row> >> >> Should we add here that if TIMESTAMP_COPY is set and the TIMECODE flag is set, >> then drivers should copy the TIMECODE struct as well? This is happening already >> in various drivers and I think that is appropriate. Although to be honest nobody >> is actually using the timecode struct, but we plan to hijack that for hardware >> timestamps in the future anyway. > > Is there a single driver which uses the timecode field? The fact is that > many m2m drivers copy it but that's probably mostly copying what one of > them happened to do by accident. :-) > No, there are no drivers that use this at the moment (other than for copying). However, it is part of the API and I'd like to close these little holes and define clearly what should be done. I think given the purpose of the timecode field it makes sense to copy it. Note that it is the application that might be providing that data, it doesn't have to come from a driver at all. I've been doing a lot of testing over the weekend and this is one of those little things that are not clearly defined. Regards, Hans -- 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
