diff options
| author | Hans Verkuil <hans.verkuil@cisco.com> | 2021-03-17 10:24:50 +0100 |
|---|---|---|
| committer | Hans Verkuil <hans.verkuil@cisco.com> | 2021-03-17 10:24:50 +0100 |
| commit | 3439c85d254774bd6debaedcfcb7fbab21d8a360 (patch) | |
| tree | 1186dc8ff1d1f484ef49c288cd0781c17945be1c | |
| parent | 02f295d4c4c1b401cdb589df5bef2645c7f83813 (diff) | |
| download | v4l-utils-3439c85d254774bd6debaedcfcb7fbab21d8a360.tar.gz | |
v4l2-compliance: don't warn about seq counter for metadata
Don't warn about unexpected sequence counters for metadata streams:
the sequence counter is typically linked to the video capture to sync
the metadata with the video data. So the sequence counter would start
at a non-zero value and produce a warning when it shouldn't.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
| -rw-r--r-- | utils/v4l2-compliance/v4l2-test-buffers.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/utils/v4l2-compliance/v4l2-test-buffers.cpp b/utils/v4l2-compliance/v4l2-test-buffers.cpp index e40461bd..131fedc6 100644 --- a/utils/v4l2-compliance/v4l2-test-buffers.cpp +++ b/utils/v4l2-compliance/v4l2-test-buffers.cpp @@ -449,7 +449,10 @@ int buffer::check(unsigned type, unsigned memory, unsigned index, "got sequence number %u, expected %u\n", g_sequence(), seq.last_seq + 1); } - } else if (static_cast<int>(g_sequence()) != seq.last_seq + 1) { + } else if (!v4l_type_is_meta(g_type()) && static_cast<int>(g_sequence()) != seq.last_seq + 1) { + // Don't do this for meta streams: the sequence counter is typically + // linked to the video capture to sync the metadata with the video + // data. So the sequence counter would start at a non-zero value. warn_or_info(is_vivid, "got sequence number %u, expected %u\n", g_sequence(), seq.last_seq + 1); } |