Rework CEC driver for kernel 4.9#58
Merged
jnettlet merged 15 commits intoSolidRun:solidrun-imx_4.9.x_1.0.0_gafrom May 14, 2018
Merged
Rework CEC driver for kernel 4.9#58jnettlet merged 15 commits intoSolidRun:solidrun-imx_4.9.x_1.0.0_gafrom
jnettlet merged 15 commits intoSolidRun:solidrun-imx_4.9.x_1.0.0_gafrom
Conversation
Empty internal queue when userland closes the device Signed-off-by: Rudi <[email protected]>
…ponsible for all this work Since libcec does the complete handling of the state machine and the setup logic by itself, kill any traces of "smartness" in the driver. Signed-off-by: Rudi <[email protected]>
Signed-off-by: Rudi <[email protected]>
This includes a new function to receive hotplug notifications from the hdmi core as well as moving the code for starting the device out of the ioctl handler and using symbolic constants whenever possible. Signed-off-by: Rudi <[email protected]>
Signed-off-by: Rudi <[email protected]>
Don't vmalloc() a new buffer for each event. Instead, get memory in units of pages from the kernel and sub-allocate as needed. Signed-off-by: Rudi <[email protected]>
…nt libs The maximum CEC message length is 16, not 17 as defined before. However, since this number has been made visible to userland, ensure that older clients that specify slightly more than 16 bytes will not be rejected. Signed-off-by: Rudi <[email protected]>
Allow userland to distinguish between a NAKs and real error conditions. Return EAGAIN in case no monitor is plugged in or a previous transmission is still pending. Signed-off-by: Rudi <[email protected]>
Even though there is only one instance of the device and only one client that uses it, access instance data via the private data pointer passed in rather than modifying the global structures directly. Signed-off-by: Rudi <[email protected]>
Inform userland after an outgoing transmission has been completed (ACKed or NAKed). Signed-off-by: Rudi <[email protected]>
Re-send an outgoing message when it was NAKed taking in account the required bus arbitration timing. Signed-off-by: Rudi <[email protected]>
Signed-off-by: Rudi <[email protected]>
If there is no link we need to allow writes otherwise libcec will block on select() in the SocketWrite() call. The write() will still fail with EAGAIN. Signed-off-by: Doug Nazar <[email protected]>
Signed-off-by: Rudi <[email protected]>
Signed-off-by: Rudi <[email protected]>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
This is a major overhaul of the hdmi-cec driver. It's based on the initial work by @wolfgar and is compatible with his libCEC implementation. It's the adaption of #42 to the 4.9 kernel series.