Messages in this thread | | | Subject | Re: [PATCH v7 5/9] media: venus: vdec: add video decoder files | From | Nicolas Dufresne <> | Date | Mon, 27 Mar 2017 10:58:03 -0400 |
| |
Le lundi 27 mars 2017 à 10:45 +0200, Hans Verkuil a écrit : > > > timestamp and sequence are only set for CAPTURE, not OUTPUT. Is > > > that correct? > > > > Correct. I can add sequence for the OUTPUT queue too, but I have no > > idea how that sequence is used by userspace. > > You set V4L2_BUF_FLAG_TIMESTAMP_COPY, so you have to copy the > timestamp from the output buffer > to the capture buffer, if that makes sense for this codec. If not, > then you shouldn't use that > V4L2_BUF_FLAG and just generate new timestamps whenever a capture > buffer is ready. > > For sequence numbering just give the output queue its own sequence > counter.
Btw, GStreamer and Chromium only supports TIMESTAMP_COPY, and will most likely leak frames if you craft timestamp.
Nicolas[unhandled content-type:application/pgp-signature] | |