Re: [PATCH] gpio: gpio-omap: take pm_runtime usage while IRQs are claimed

[Tony Lindgren <tony@xxxxxxxxxxx>]

* Grygorii Strashko <grygorii.strashko@xxxxxx> [190410 18:17]:
> I'm very sorry, but what was the regression exactly?
> Can't enter RTC+DDR state? some crash?

AFAIK fails to enter RTC+DDR suspend because of conditional
PM runtime handling.

> We have in driver:
>  irqc->parent_device = dev;
> 
> which means:
>  request_threaded_irq()
>    irq_chip_pm_get()
> 	if (IS_ENABLED(CONFIG_PM) && data->chip->parent_device) {
> 		retval = pm_runtime_get_sync(data->chip->parent_device);
> 
> and power.usage_count will be incremented every time GPIO irq is requested.
> 
> only in free_irq() (or in case of error) power.usage_count is decremented.
> 
> Now above change will introduce just another incrementation of power.usage_count
> How is it helping?

Oh OK, that means we can simplify things further and drop those
changes then. I'll post v2 version shortly with updated subject
and description.

Regards,

Tony