Thread (5 messages) 5 messages, 2 authors, 2008-05-07

Re: [PATCH][RESEND] gpio-keys debouncing support

From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Date: 2008-05-07 17:20:17

On Wed, May 07, 2008 at 08:21:27PM +0400, Dmitry wrote:
Hi, Dmitry,

2008/5/6, Dmitry Torokhov [off-list ref]:
quoted
Hi Dmitry,


 On Fri, May 02, 2008 at 03:20:48PM +0400, Dmitry Baryshkov wrote:
 > Sometimes gpio line can generate jitter while transitioning from one state
 > to another one. Implement a way to filter such noise during transitions.
 >


I don't think we need to do both count and interval and you don't
 really need to track state... What do you think about the patch below?
I would say it's pretty different from what I meant. You patch only
delays the decision about the status of the pin while mine does really
"average" the status
of the pin. E.g. in my tosa PDA the headphones jack is a bit noisy. So
the input layer shouldn't react to the noise.
The userspace will not see the new event until gpio stabilizes,
that's all that is needed.
And with your patch each
time the GPIO irq is generated the input layer will get input_sync.
That I think I need to fix in input core. I think we used to not reset
sync flag on ignored events.

-- 
Dmitry
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help