Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1609582
| From | Hans Verkuil <hverkuil@xs4all.nl> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH v7 5/9] media: venus: vdec: add video decoder files |
| Date | 2017-03-27 11:00 +0200 |
| Message-ID | <tpyFj-5la-3@gated-at.bofh.it> (permalink) |
| References | <tkBaN-3bi-5@gated-at.bofh.it> <tkBaO-3bi-19@gated-at.bofh.it> <toyHp-2WH-31@gated-at.bofh.it> <tp2Fr-7mu-17@gated-at.bofh.it> <tpszT-Wf-3@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On 27/03/17 04:18, Nicolas Dufresne wrote: > Le dimanche 26 mars 2017 à 00:30 +0200, Stanimir Varbanov a écrit : >>>> + vb->planes[0].data_offset = data_offset; >>>> + vb->timestamp = timestamp_us * NSEC_PER_USEC; >>>> + vbuf->sequence = inst->sequence++; >>> >>> 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. > > Neither GStreamer or Chromium seems to use it. What does that number > means for a m2m driver ? Does it really means something ? It can be used to detect dropped frame (the sequence counter will skip in that case). Unlikely to happen for m2m devices, and most apps ignore it as well. But you still need to fill it in, it's a V4L2 requirement. Regards, Hans
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
Re: [PATCH v7 5/9] media: venus: vdec: add video decoder files Hans Verkuil <hverkuil@xs4all.nl> - 2017-03-24 15:50 +0100
Re: [PATCH v7 5/9] media: venus: vdec: add video decoder files Nicolas Dufresne <nicolas@ndufresne.ca> - 2017-03-24 19:30 +0100
Re: [PATCH v7 5/9] media: venus: vdec: add video decoder files Stanimir Varbanov <stanimir.varbanov@linaro.org> - 2017-03-26 00:20 +0100
Re: [PATCH v7 5/9] media: venus: vdec: add video decoder files Stanimir Varbanov <stanimir.varbanov@linaro.org> - 2017-03-25 23:50 +0100
Re: [PATCH v7 5/9] media: venus: vdec: add video decoder files Nicolas Dufresne <nicolas@ndufresne.ca> - 2017-03-27 04:30 +0200
Re: [PATCH v7 5/9] media: venus: vdec: add video decoder files Hans Verkuil <hverkuil@xs4all.nl> - 2017-03-27 11:00 +0200
Re: [PATCH v7 5/9] media: venus: vdec: add video decoder files Hans Verkuil <hverkuil@xs4all.nl> - 2017-03-27 10:50 +0200
Re: [PATCH v7 5/9] media: venus: vdec: add video decoder files Hans Verkuil <hverkuil@xs4all.nl> - 2017-03-27 10:50 +0200
Re: [PATCH v7 5/9] media: venus: vdec: add video decoder files Stanimir Varbanov <stanimir.varbanov@linaro.org> - 2017-03-27 14:00 +0200
Re: [PATCH v7 5/9] media: venus: vdec: add video decoder files Nicolas Dufresne <nicolas@ndufresne.ca> - 2017-03-27 17:10 +0200
csiph-web