Jon Hunter <jonath...@nvidia.com> writes: > Some IRQ chips may be located in a power domain outside of the CPU > subsystem and hence will require device specific runtime power > management. In order to support such IRQ chips, add a pointer for a > device structure to the irq_chip structure, and if this pointer is > populated by the IRQ chip driver and CONFIG_PM is selected in the kernel > configuration, then the pm_runtime_get/put APIs for this chip will be > called when an IRQ is requested/freed, respectively. > > Signed-off-by: Jon Hunter <jonath...@nvidia.com>
[...] > @@ -1891,10 +1906,18 @@ int setup_percpu_irq(unsigned int irq, struct > irqaction *act) > > if (!desc || !irq_settings_is_per_cpu_devid(desc)) > return -EINVAL; > + > + retval = irq_chip_pm_get(&desc->irq_data); > + if (retval < 0) > + return retval; > + > chip_bus_lock(desc); > retval = __setup_irq(irq, desc, act); > chip_bus_sync_unlock(desc); > > + if (retval) > + irq_chip_pm_get(&desc->irq_data); > + Shouldn't this one be a _put() ? Otherwise, LGTM Reviewed-by: Kevin Hilman <khil...@baylibre.com> Kevin