| Message ID | cover.1624006513.git.sean@mess.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | IR driver for USB-UIRT device | expand |
On Fri, Jun 18, 2021 at 11:18:45AM +0100, Sean Young wrote: > This is a new rc-core driver for the USB-UIRT which you can see here > http://www.usbuirt.com/ > > This device is supported in lirc, via the usb serial kernel driver. This > driver is both for rc-core, which means it can use kernel/BPF decoding > ec. Also this implement is superior because it can: > - support learning mode > - setting transmit carrier > - larger transmits using streaming tx command > - Much better latency since it is a kernel driver A ball-park number to back this claim up would be good here. Your driver sets the ftdi latency timer to 50 ms which adds quite a bit of latency for short packets (e.g. a single key press?) to begin with. > Changes since v4: > - Fixed clang warning/uninitialized variable usage > - Some cosmetic changes > > Changes since v3: > - Review comments from Johan Hovold In the future, please include some details on what changed also when addressing review feedback. > - Do not move the ftdi_sio.h file an copy FTDI_* definitions instead > > Changes since v2: > - Fixed race condition is disconnect > - Removed superfluous kmalloc in short tx > > Changes since v1: > - Review comments from Oliver Neukum > - Simplified wideband read function > > Sean Young (2): > media: rc: new driver for USB-UIRT device > USB: serial: blacklist USB-UIRT when driver is selected > > drivers/media/rc/Kconfig | 11 + > drivers/media/rc/Makefile | 1 + > drivers/media/rc/uirt.c | 744 ++++++++++++++++++++++++++++++++++ > drivers/usb/serial/ftdi_sio.c | 6 +- > 4 files changed, 760 insertions(+), 2 deletions(-) > create mode 100644 drivers/media/rc/uirt.c Johan
On Fri, Jul 02, 2021 at 12:01:59PM +0200, Johan Hovold wrote: > On Fri, Jun 18, 2021 at 11:18:45AM +0100, Sean Young wrote: > > This is a new rc-core driver for the USB-UIRT which you can see here > > http://www.usbuirt.com/ > > > > This device is supported in lirc, via the usb serial kernel driver. This > > driver is both for rc-core, which means it can use kernel/BPF decoding > > ec. Also this implement is superior because it can: > > - support learning mode > > - setting transmit carrier > > - larger transmits using streaming tx command > > - Much better latency since it is a kernel driver > > A ball-park number to back this claim up would be good here. Your driver > sets the ftdi latency timer to 50 ms which adds quite a bit of latency > for short packets (e.g. a single key press?) to begin with. Yes, I'll do some measurements. This is based on observations with other in kernel drivers. > > Changes since v4: > > - Fixed clang warning/uninitialized variable usage > > - Some cosmetic changes > > > > Changes since v3: > > - Review comments from Johan Hovold > > In the future, please include some details on what changed also when > addressing review feedback. Right, good point. Will do. > > - Do not move the ftdi_sio.h file an copy FTDI_* definitions instead > > > > Changes since v2: > > - Fixed race condition is disconnect > > - Removed superfluous kmalloc in short tx > > > > Changes since v1: > > - Review comments from Oliver Neukum > > - Simplified wideband read function > > > > Sean Young (2): > > media: rc: new driver for USB-UIRT device > > USB: serial: blacklist USB-UIRT when driver is selected > > > > drivers/media/rc/Kconfig | 11 + > > drivers/media/rc/Makefile | 1 + > > drivers/media/rc/uirt.c | 744 ++++++++++++++++++++++++++++++++++ > > drivers/usb/serial/ftdi_sio.c | 6 +- > > 4 files changed, 760 insertions(+), 2 deletions(-) > > create mode 100644 drivers/media/rc/uirt.c > > Johan Thanks Sean