Em Mon, 21 Sep 2015 12:08:43 -0700 Eric Nelson <eric@xxxxxxxxxx> escreveu: > A default timeout value of 100ms is workable for most decoders. > Declare a constant to help standardize its' use. I guess the worse case scenario is the NEC protocol: http://www.sbprojects.com/knowledge/ir/nec.php with allows a repeat message to be sent on every 110ms. As the repeat message is 11.25 ms, that would mean a maximum time without data for 98.75 ms. So, in thesis, 100 ms would be ok. However, IR timings are not always precise and may affected by the battery charge. So, I think that a timeout of 100ms is too close to 98.75 and may cause troubles. S, IMHO, it is safer to define the default timeout as 125ms. Regards, Mauro > > Signed-off-by: Eric Nelson <eric@xxxxxxxxxx> > --- > include/media/rc-core.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/media/rc-core.h b/include/media/rc-core.h > index ec921f6..62c64bd 100644 > --- a/include/media/rc-core.h > +++ b/include/media/rc-core.h > @@ -239,6 +239,7 @@ static inline void init_ir_raw_event(struct ir_raw_event *ev) > memset(ev, 0, sizeof(*ev)); > } > > +#define IR_DEFAULT_TIMEOUT MS_TO_NS(100) > #define IR_MAX_DURATION 500000000 /* 500 ms */ > #define US_TO_NS(usec) ((usec) * 1000) > #define MS_TO_US(msec) ((msec) * 1000) -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
