Skip to content
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

Backporting lirc_odroid driver from hardkernel #113

Open
wants to merge 1 commit into
base: amlogic-3.14.y
Choose a base branch
from

Conversation

synchrone
Copy link

@synchrone synchrone commented Apr 15, 2018

Backporting the GPIO-based implementation of Lirc receiver/transmitter driver, initally contributed at hardkernel/linux#29, but later reworked in hardkernel/linux@ca7d83c#diff-a8d20779ea0c3115b28e31876ff93685

Tested this with LibreELEC 8.2.5.

@JimmyS83
Copy link

JimmyS83 commented Jan 3, 2019

Nice!
What about these changes/insertions?

CONFIG_LIRC=y
CONFIG_IR_LIRC_CODEC=y
CONFIG_LIRC_STAGING=y
CONFIG_LIRC_ODROID=m

2013 version (for X/X2) is transmitter/receiver, unfortunatelly, rework done 2016/2017 change this to only transmitter.

@synchrone
Copy link
Author

Unfortunately I do not have the X or X2 boards to try it with. I made this patch for the C2.

@JimmyS83
Copy link

JimmyS83 commented Jan 9, 2019

Ok.. I just pointed out that the lirc_odroid as it is now, is only for transmitter.. For the receiver function gpioplug-ir-recv+gpio-ir-recv is needed :(

@synchrone
Copy link
Author

But that's exactly what I added. The GPIO Receiver functionality.

Btw, it seems like gpioplug-ir-recv was just added in a fork https://github.com/CoreELEC/CoreELEC/releases/tag/8.95.7

@JimmyS83
Copy link

But that's exactly what I added. The GPIO Receiver functionality.

I could be wrong, but submitted lirc_odroid.c:

MODULE_DESCRIPTION("GPIO based IR Transmitter driver for Odroid C/C2");
MODULE_PARM_DESC(gpio_out_pin, "GPIO output/transmitter pin number");

is transmitter, not receiver.. This works for you as receiver?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants