PM / Runtime: Idle devices asynchronously after probe|release
authorUlf Hansson <ulf.hansson@linaro.org>
Wed, 10 Apr 2013 15:00:48 +0000 (17:00 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 11 Apr 2013 19:42:52 +0000 (12:42 -0700)
Putting devices into idle|suspend in a synchronous manner means we are
waiting for each device to become idle|suspended before the probe|release
is fully done.

This patch switch to use the asynchronous runtime PM API:s instead and
thus improves the parallelism since we can move on and handle the next
device in queue in an earlier phase.

Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
Acked-by: Kevin Hilman <khilman@linaro.org>
Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Cc: Alan Stern <stern@rowland.harvard.edu>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/base/dd.c

index bb5645e..35fa368 100644 (file)
@@ -380,7 +380,7 @@ int driver_probe_device(struct device_driver *drv, struct device *dev)
 
        pm_runtime_barrier(dev);
        ret = really_probe(dev, drv);
-       pm_runtime_idle(dev);
+       pm_request_idle(dev);
 
        return ret;
 }
@@ -428,7 +428,7 @@ int device_attach(struct device *dev)
                }
        } else {
                ret = bus_for_each_drv(dev->bus, NULL, dev, __device_attach);
-               pm_runtime_idle(dev);
+               pm_request_idle(dev);
        }
 out_unlock:
        device_unlock(dev);
@@ -499,7 +499,7 @@ static void __device_release_driver(struct device *dev)
                                                     BUS_NOTIFY_UNBIND_DRIVER,
                                                     dev);
 
-               pm_runtime_put_sync(dev);
+               pm_runtime_put(dev);
 
                if (dev->bus && dev->bus->remove)
                        dev->bus->remove(dev);