Em Mon, 21 Sep 2015 12:08:43 -0700
Eric Nelson <e...@nelint.com> 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 <e...@nelint.com>
> ---
>  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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to