-
Notifications
You must be signed in to change notification settings - Fork 14
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
Disambiguation needed about AJANTV2_BUILD_OPENSOURCE and HEVC in the Linux driver #6
Comments
Which files are you referring to that you are concerned about? There are some closed-source HEVC codec files that are not part of the open source libajantv2 repo, whose inclusion is controlled by the AJANTV2_BUILD_OPENSOURCE variable. The intention behind that variable was to allow gating any proprietary code in the library, present and future. |
It's guarded behind Edit: The whole HEVC part of the Linux driver is MIT |
For more context the Linux driver Makefile have no mention that HEVC part is closed source, unlike the CMake with |
I think this might have been an error in the Linux driver's CMakeLists. I'll check with the rest of the team. I believe those hevc*.h/.c files in the Linux driver are part of the Kona HEVC card driver, are MIT-licensed, and thus probably shouldn't be omitted if However, there are some proprietary HEVC codec files in the ajantv2 component that are closed-source and not included in the libajantv2 open-source repo. These are only built into the ajantv2 library if |
I forgot to mention this, I got the purpose |
It looks like we did have an AJA_HEVC variable in the Makefile that enabled building of the Kona HEVC card driver, so I suppose it makes some sense that we separate that. The Kona HEVC Linux driver itself is open-source, it's just the SDK HEVC codec code that is not made available to open-source users, and is thus gated behind the AJANTV2_BUILD_OPENSOURCE CMake var. I'll see what the team wants to do here and make the necessary code changes, if any. |
I think the notion of HEVC patent and AJANTV2_BUILD_OPENSOURCE needs to be split if the files are really under MIT.
The text was updated successfully, but these errors were encountered: