On Fri, Sep 11, 2015 at 07:00:24AM -0700, Eric Nelson wrote: > Many decoders require a trailing space (period without IR illumination) > to be delivered before completing a decode. > > Since the gpio-ir-recv driver only delivers events on gpio transitions, > a single IR symbol (caused by a quick touch on an IR remote) will not > be properly decoded without the use of a timer to flush the tail end > state of the IR receiver. This is a problem other IR drivers suffer from too. It might be better to send a IR timeout event like st_rc_send_lirc_timeout() in st_rc.c, with the duration set to what the timeout was. That is what irraw timeouts are for; much better than fake transitions. > This patch adds an optional device tree node "flush-ms" which, if > present, will use a jiffie-based timer to complete the last pulse > stream and allow decode. A common value for this is 100ms, I'm not sure what use it has to have it configurable. It's nice to have it exposed in rc_dev->timeout. Sean -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html