RE: [RFC] [PATCH 1/3] OMAP4: Keyboard Controller Support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----Original Message-----
> From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap-owner@xxxxxxxxxxxxxxx] On Behalf Of Arce,
> Abraham
> Sent: Wednesday, May 12, 2010 11:10 AM
> To: Dmitry Torokhov
> Cc: linux-input@xxxxxxxxxxxxxxx; linux-omap@xxxxxxxxxxxxxxx
> Subject: RE: [RFC] [PATCH 1/3] OMAP4: Keyboard Controller Support
> 
> Dmitry,
> 
> 2 comments + one question before sending next version...
> 
> [...]
> 
> > > > > > +static irqreturn_t omap_keypad_threaded(int irq, void *dev_id)
> > > > > > +{
> > > > >
> > > > > Why is iti threaded? I fo not see anything that will sleep.
> > >
> > >
> > > It was implemented based on previous comments...
> > >
> >
> > Would you point me to that comment? Like I said, I do not see anything
> > that would possibly sleep in this routine so you don't need to use
> > threaded interrupt.
> 
> Using now request_irq based on your comments. In same omap_keypad_interrupt disable/clear/enable
> interrupts will be executed
> 
> [...]
>
Sorry for jumping into the comments late. Thought this was sorted out. Key scanning
and debounce timeouts etc still there. Having all these things in ISR itself isn't good
idea.

Dmitry,
Don't you think its optimal to push the key-scanning and debounce timeout code
part of bottom half ??

Regards,
Santosh
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux