-
Notifications
You must be signed in to change notification settings - Fork 294
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 support for iMX.6 #323
Conversation
Ironically I spend an hour today updating it too before I thought to check if there was a pull request. I only use the basics (on, off, etc) but it works fine for me on several i.MX6 devices and various TVs. Thanks, |
I'll have to check this after doing a minor release with fixes. Thanks :) |
@warped-rudi does this need to be updated to be compatible with iMX.8 (and iMX.7) SoCs? I saw that SolidRun have a new iMX 8 based CuBox as well as a new HummingBoard board. |
This stuff is compatible with SolidRun's 3.x and 4.9.x kernels for iMX.6, which in turn are derived from the chip vendor's non-mainline kernels. It appears that SolidRun is working on a unified 4.9 for all their iMX products and I guess/hope that we we can use this code for iMX.8 as well. But it could also be, that iMX.8 will be using the newer kernel CEC framework. |
@warped-rudi sorry for the delay. If this PR is still relevant with the current iMX hardware/software, could you then rebase it so I can include it in the upcoming release. Also, if you haven't signed this yet, please take a look at https://www.pulse-eight.com/contributors If it's all using the Linux in-kernel implementation now, then this is no longer needed. Thanks |
Requires specific kernel driver. Check http://stephan-rafin.net/blog/2013/09/30/i-mx6-cec/ Author: wolfgar <[email protected]>
…ode() and GetStats()
a9d5af2
to
bb7cab7
Compare
@opdenkamp I rebased to the patch and did a quick test t verify that it works on an older iMX.6 board. I kinda lost track of the current status of the iMX.6 mainline support, but AFAIK it's still not as good as the old vendor kernels with respect to multimedia. Therefore I'd like to see it included. As for the contributors agreement: I'm pretty sure that I already signed it when I submitted the TDA995x stuff many years ago. |
Thanks for the update and for the patch. There are a couple of small cosmetic issues that I'll fix up later. |
Talking about cleanup: It appears that all of the "custom" adapters implement a member function named 'XXXCECAdapterCommunication::HandleLogicalAddressLost()'. I think this is a leftover from earlier days and is not needed anymore. |
Finally... This patch set adds support for the iMX.6 SoC. It is based on the original code by @wolfgar from 2014. Even though it should work with his original kernel driver, it's suggested to use this driver. So far it has only been tested with the 'official' SolidRun 3.14.xx series kernels.