Re: Interrupt issue in twl4030_keypad

2011-12-15 Thread Felipe Contreras
On Mon, Dec 12, 2011 at 11:34 PM, Felipe Balbi wrote: > On Mon, Dec 12, 2011 at 10:55:11PM +0200, Felipe Contreras wrote: >> > Samuel assumed it was ok and, like I said above, it worked for my simple >> > GPIO >> > usecase with beagleboard. >> >> Well, for 3.2 I think the situation is fine, but t

Re: Interrupt issue in twl4030_keypad

2011-12-13 Thread Grazvydas Ignotas
On Mon, Dec 12, 2011 at 10:55 PM, Felipe Contreras wrote: > On Mon, Dec 12, 2011 at 9:12 PM, Felipe Balbi wrote: >> On Mon, Dec 12, 2011 at 08:30:49PM +0200, Felipe Contreras wrote: >>> The short version is this: either we revert this patch[1], or we apply >>> this patch series[2], as well as its

Re: Interrupt issue in twl4030_keypad

2011-12-12 Thread Felipe Balbi
Hi, On Mon, Dec 12, 2011 at 10:55:11PM +0200, Felipe Contreras wrote: > > Samuel assumed it was ok and, like I said above, it worked for my simple > > GPIO > > usecase with beagleboard. > > Well, for 3.2 I think the situation is fine, but that's not what I'm > talking about. About your GPIO test

Re: Interrupt issue in twl4030_keypad

2011-12-12 Thread Greg KH
On Mon, Dec 12, 2011 at 11:04:35PM +0200, Felipe Contreras wrote: > On Mon, Dec 12, 2011 at 9:23 PM, Greg KH wrote: > > On Mon, Dec 12, 2011 at 08:30:49PM +0200, Felipe Contreras wrote: > >> The short version is this: either we revert this patch[1], or we apply > >> this patch series[2], as well a

Re: Interrupt issue in twl4030_keypad

2011-12-12 Thread Felipe Contreras
On Mon, Dec 12, 2011 at 9:23 PM, Greg KH wrote: > On Mon, Dec 12, 2011 at 08:30:49PM +0200, Felipe Contreras wrote: >> The short version is this: either we revert this patch[1], or we apply >> this patch series[2], as well as its essential fixes[3]. >> >> The long version is this. There's a synchr

Re: Interrupt issue in twl4030_keypad

2011-12-12 Thread Felipe Contreras
On Mon, Dec 12, 2011 at 9:12 PM, Felipe Balbi wrote: > On Mon, Dec 12, 2011 at 08:30:49PM +0200, Felipe Contreras wrote: >> The short version is this: either we revert this patch[1], or we apply >> this patch series[2], as well as its essential fixes[3]. >> >> The long version is this. There's a s

Re: Interrupt issue in twl4030_keypad

2011-12-12 Thread Greg KH
On Mon, Dec 12, 2011 at 08:30:49PM +0200, Felipe Contreras wrote: > Hi, > > The short version is this: either we revert this patch[1], or we apply > this patch series[2], as well as its essential fixes[3]. > > The long version is this. There's a synchronization issue with the > current keypad dri

Re: Interrupt issue in twl4030_keypad

2011-12-12 Thread Felipe Balbi
Hi, On Mon, Dec 12, 2011 at 08:30:49PM +0200, Felipe Contreras wrote: > The short version is this: either we revert this patch[1], or we apply > this patch series[2], as well as its essential fixes[3]. > > The long version is this. There's a synchronization issue with the > current keypad driver

Interrupt issue in twl4030_keypad

2011-12-12 Thread Felipe Contreras
Hi, The short version is this: either we revert this patch[1], or we apply this patch series[2], as well as its essential fixes[3]. The long version is this. There's a synchronization issue with the current keypad driver and twl core; the irq is marked as handled even though the thread that is su