Fix: issue with interpretation of the F bit in the ST40 packet #1184
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
awilczyns
requested review from
Sakoram,
DawidWesierski4 and
Falron98
as code owners
DawidWesierski4
changed the title
There was a problem hiding this comment.
Seems to track with https://datatracker.ietf.org/doc/html/rfc8331
F: 2 bits
These two bits relate to signaling the field specified by the
RTP timestamp in an interlaced SDI raster. A value of 0b00
indicates that either the video format is progressive or that
no field is specified. A value of 0b10 indicates that the
timestamp refers to the first field of an interlaced video
signal. A value of 0b11 indicates that the timestamp refers
to the second field of an interlaced video signal. The value
0b01 is not valid. Receivers SHOULD ignore an ANC data
packet with an F field value of 0b01 and SHOULD process any
other ANC data packets with valid F field values that are
present in the RTP payload.
Note that some multi-stream SDI interfaces might use multiple
interlaced signal streams to transmit progressive images, in
which case the "F" bits would refer to the field of the
interlaced stream used for transport of the ANC data packet.
was this tested ?
Sakoram
reviewed
Jun 24, 2025
Yes |
Sakoram
approved these changes
Jun 24, 2025
awilczyns
force-pushed
the
fix-anc-issue-f-bits
branch
from
f8068b6 to
9f7dcdd
Compare
Mionsz
force-pushed
the
fix-anc-issue-f-bits
branch
from
9f7dcdd to
1554e30
Compare
Mionsz
added
bug fix
…ST40 packet Signed-off-by: Wilczynski, Andrzej <andrzej.wilczynski@intel.com>
…if statement for improved readability. Signed-off-by: Wilczynski, Andrzej <andrzej.wilczynski@intel.com>
Mionsz
force-pushed
the
fix-anc-issue-f-bits
branch
from
1554e30 to
b2c3681
Compare
Mionsz
approved these changes
Jul 3, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.