Commit 63d00be6 authored Jun 08, 2021 by Ulf Hansson Committed by Rafael J. Wysocki Jun 11, 2021

PM: runtime: Allow unassigned ->runtime_suspend|resume callbacks



We are currently allowing ->runtime_idle() callbacks to be unassigned
without returning an error code from rpm_idle(). This has been useful to
avoid boilerplate code in drivers. Let's take this approach a step further,
by allowing also unassigned ->runtime_suspend|resume() callbacks.

In this way, a consumer/supplier device link can be used to let a consumer
device be power managed through its supplier device, without requiring
assigned ->runtime_suspend|resume() callbacks for the consumer device, for
example.

Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
Acked-by: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

parent 5a2bd1b1

drivers/base/power/runtime.c

+3 −5

Original line number	Diff line number	Diff line
		@@ -345,7 +345,7 @@ static void rpm_suspend_suppliers(struct device *dev)
		static int __rpm_callback(int (cb)(struct device ), struct device *dev)
		__releases(&dev->power.lock) __acquires(&dev->power.lock)
		{
		int retval, idx;
		int retval = 0, idx;
		bool use_links = dev->power.links_count > 0;

		if (dev->power.irq_safe) {
		@@ -373,6 +373,7 @@ static int __rpm_callback(int (cb)(struct device ), struct device *dev)
		}
		}

		if (cb)
		retval = cb(dev);

		if (dev->power.irq_safe) {
		@@ -484,9 +485,6 @@ static int rpm_callback(int (cb)(struct device ), struct device *dev)
		{
		int retval;

		if (!cb)
		return -ENOSYS;

		if (dev->power.memalloc_noio) {
		unsigned int noio_flag;