-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add RVF support #35
Add RVF support #35
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR, @adrianf0!
Looks really good, just a few comments =D
@edersondisouza Thank you for the review. I applied all your suggestions except the one regarding formatting. Please check my comment on it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking great! Just a typo to fix.
The tab thing is your call, unless someone else complains.
Copied from linux kernel (added Macros alignment). Helps to keep code formatting. Signed-off-by: Adrian Fiergolski <adrian.fiergolski@fastree3d.com>
Signed-off-by: Adrian Fiergolski <adrian.fiergolski@fastree3d.com>
This patch adds support for AVTP Raw Video Format (RVF) specified in chapter 12 of the IEEE 1722-2016 spec. RVF is based on AVTP Stream PDU, so it reuses `struct avtp_stream_pdu` defined in avtp.h. However, as the Raw header has two 32-bit fields more than that struct, it also introduces a "struct avtp_rvf_payload" that is used to access the fields defined on the "avtp_payload" field. RVF API is exposed via "avtp_rvf.h" header, including raw specific bits. Signed-off-by: Adrian Fiergolski <adrian.fiergolski@fastree3d.com>
Adds unit tests "test-rvf.c" that implements several RVF test cases. The tests should cover all APIs defined in "avtp_rvf.h". Signed-off-by: Adrian Fiergolski <adrian.fiergolski@fastree3d.com>
lgtm! |
@edersondisouza : I have added support to Gstreamer (link). Feel free to take a look. |
Great! I'll take a look in the next few days =D |
This MR request adds RVF support (#33).