* Javier Martinez Canillas [off-list ref] [130923 10:09]:

On 09/23/2013 06:45 PM, Tony Lindgren wrote:

quoted

Hmm does this still work for legacy platform data based
drivers that are doing gpio_request() first?

Yes it still work when booting using board files. I tested on my OMAP3 board and
it worked in both DT and legacy booting mode.

OK great.
 

quoted

And what's the path for clearing things for PM when free_irq()
gets called? It seems that this would leave the GPIO bank
enabled causing a PM regression?

Indeed, I did set bank->mod_usage |= 1 << offset so the bank is enabled if the
device goes to suspended and then resumed but I completely forget about the
clearing path when the IRQ is freed.

Which makes me think that we should probably maintain two usage variables, one
for GPIO and another one for IRQ and check both of them on the suspend/resume pm
functions.

Yes that it seems that they should be treated separately.

Regards,

Tony