Em Mon, 21 Sep 2015 12:08:43 -0700
Eric Nelson <[email protected]> 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 <[email protected]>
> ---
> 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 devicetree" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
