TODO RC: Difference between revisions

From LinuxTVWiki
Jump to navigation Jump to search
No edit summary
(Changes merged for v4.16)
Line 2: Line 2:
* struct ir_raw_event_ctrl member prev_ev. Investigate if this can be removed.
* struct ir_raw_event_ctrl member prev_ev. Investigate if this can be removed.
* if driver depends on RC_CORE, change this to select RC_CORE, depends on INPUT? OR use a patch like https://patchwork.linuxtv.org/patch/40601/ and remove "depends on RC_CORE".
* if driver depends on RC_CORE, change this to select RC_CORE, depends on INPUT? OR use a patch like https://patchwork.linuxtv.org/patch/40601/ and remove "depends on RC_CORE".
* RC device RC_DRIVER_IR_RAW_TX leaks dev->raw on rc_unregister_device()?
* Unplug iguanair with lirc chardev open causes kernel to go bang
* CEC repeat code needs to move into rc-core
* mceusb can do wakeups. Allow this to be configured
* mceusb can do wakeups. Allow this to be configured
* Can sir_ir and/or serial_ir use serdev
* Can sir_ir and/or serial_ir use serdev
Line 15: Line 12:
** Further improvement: make the kernel mapping rc_proto:scancode -> keycode, not just scancode. This should reduce the number of possible conflicts.
** Further improvement: make the kernel mapping rc_proto:scancode -> keycode, not just scancode. This should reduce the number of possible conflicts.
* Compare lirc rc6 decoder. Can it handle multiple keypresses? What repeat/delay times does it use? See https://bugzilla.kernel.org/show_bug.cgi?id=48761
* Compare lirc rc6 decoder. Can it handle multiple keypresses? What repeat/delay times does it use? See https://bugzilla.kernel.org/show_bug.cgi?id=48761
* lirc_zilog rc core driver (Sean Young working on this)
* Once lirc_zilog is removed, remove lirc kapi. Example is here: https://git.linuxtv.org/syoung/media_tree.git/log/?h=lirc_scancodes
* Implement scancode reading and transmitting:
** It's very hard to tell what protocol a remote is using
** Kernel already has encoders, make sure they can be used for tx
** CEC might want to reuse this interface
** Patch series already exist for this
* MS MCE Keyboard keymap needs multiple protocols. Keymap needed to select it. https://www.mail-archive.com/linux-media@vger.kernel.org/msg115706.html
* MS MCE Keyboard keymap needs multiple protocols. Keymap needed to select it. https://www.mail-archive.com/linux-media@vger.kernel.org/msg115706.html
* ir-ctl should list available devices if run without any argument and allow to select device based on rc (based on /sys/class/rc/).
* ir-ctl should list available devices if run without any argument and allow to select device based on rc (based on /sys/class/rc/).

Revision as of 15:25, 14 December 2017

Minor bugs/improvement

  • struct ir_raw_event_ctrl member prev_ev. Investigate if this can be removed.
  • if driver depends on RC_CORE, change this to select RC_CORE, depends on INPUT? OR use a patch like https://patchwork.linuxtv.org/patch/40601/ and remove "depends on RC_CORE".
  • mceusb can do wakeups. Allow this to be configured
  • Can sir_ir and/or serial_ir use serdev
  • iguanair_tx() can be simplified
  • NEC32 mess (again)
  • Rename lirc record -> lirc receive (e.g. ir-ctl).

Larger work

  • Make it possible to load multiple keymaps with ir-keytable. It should detect if the new keymap conflicts with any already loaded keymap, and ensure the right protocols are selected.
    • Further improvement: make the kernel mapping rc_proto:scancode -> keycode, not just scancode. This should reduce the number of possible conflicts.
  • Compare lirc rc6 decoder. Can it handle multiple keypresses? What repeat/delay times does it use? See https://bugzilla.kernel.org/show_bug.cgi?id=48761
  • MS MCE Keyboard keymap needs multiple protocols. Keymap needed to select it. https://www.mail-archive.com/linux-media@vger.kernel.org/msg115706.html
  • ir-ctl should list available devices if run without any argument and allow to select device based on rc (based on /sys/class/rc/).
  • Rework ir raw decoders; register should only happen when protocol enabled (so we don't get an MCE input device). Also if decoders not enabled, #ifdef a load of stuff out.