Commit ce481895 authored Apr 18, 2022 by Minghao Chi Committed by Thomas Gleixner May 02, 2022

genirq: Use pm_runtime_resume_and_get() instead of pm_runtime_get_sync()



pm_runtime_resume_and_get() achieves the same and simplifies the code.

[ tglx: Simplify it further by presetting retval ]

Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: Minghao Chi <chi.minghao@zte.com.cn>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Link: https://lore.kernel.org/r/20220418110716.2559453-1-chi.minghao@zte.com.cn

parent 1adb4d7a

kernel/irq/chip.c

+4 −9

Original line number	Diff line number	Diff line
		@@ -1573,17 +1573,12 @@ static struct device irq_get_parent_device(struct irq_data data)
		int irq_chip_pm_get(struct irq_data *data)
		{
		struct device *dev = irq_get_parent_device(data);
		int retval;
		int retval = 0;

		if (IS_ENABLED(CONFIG_PM) && dev) {
		retval = pm_runtime_get_sync(dev);
		if (retval < 0) {
		pm_runtime_put_noidle(dev);
		return retval;
		}
		}
		if (IS_ENABLED(CONFIG_PM) && dev)
		retval = pm_runtime_resume_and_get(dev);

		return 0;
		return retval;
		}

		/**