Re: [PATCH V2 1/2] rc-core: define a default timeout for drivers

[Eric Nelson <eric-SeqgQ6RdavfQT0dZR+AlfA@public.gmane.org>]

Thanks Mauro,

On 10/03/2015 07:25 AM, Mauro Carvalho Chehab wrote:
> Em Mon, 21 Sep 2015 12:08:43 -0700
> Eric Nelson <eric-SeqgQ6RdavfQT0dZR+AlfA@public.gmane.org> 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.
> 

Sounds good. I'll submit V3 shortly.

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html