PM / QoS: Resume device before exposing/hiding PM QoS flags
authorLan Tianyu <tianyu.lan@intel.com>
Thu, 8 Nov 2012 03:14:08 +0000 (11:14 +0800)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Sat, 10 Nov 2012 21:56:19 +0000 (22:56 +0100)
Since dev_pm_qos_add_request(), dev_pm_qos_update_request() and
dev_pm_qos_remove_request() for PM QoS flags should not be invoked
when device in RPM_SUSPENDED, add pm_runtime_get_sync() and pm_runtime_put()
around these functions in dev_pm_qos_expose_flags() and
dev_pm_qos_hide_flags().

[rjw: Modified the subject and changelog to better reflect the code
 changes made.]

Signed-off-by: Lan Tianyu <tianyu.lan@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/base/power/qos.c

index 081db2d25daadbc37fe667dd52ec92ce7192cc2d..fdc3894bc33afe357060bd4061e6afcb8e0b4575 100644 (file)
@@ -633,15 +633,18 @@ int dev_pm_qos_expose_flags(struct device *dev, s32 val)
        if (!req)
                return -ENOMEM;
 
+       pm_runtime_get_sync(dev);
        ret = dev_pm_qos_add_request(dev, req, DEV_PM_QOS_FLAGS, val);
        if (ret < 0)
-               return ret;
+               goto fail;
 
        dev->power.qos->flags_req = req;
        ret = pm_qos_sysfs_add_flags(dev);
        if (ret)
                __dev_pm_qos_drop_user_request(dev, DEV_PM_QOS_FLAGS);
 
+fail:
+       pm_runtime_put(dev);
        return ret;
 }
 EXPORT_SYMBOL_GPL(dev_pm_qos_expose_flags);
@@ -654,7 +657,9 @@ void dev_pm_qos_hide_flags(struct device *dev)
 {
        if (dev->power.qos && dev->power.qos->flags_req) {
                pm_qos_sysfs_remove_flags(dev);
+               pm_runtime_get_sync(dev);
                __dev_pm_qos_drop_user_request(dev, DEV_PM_QOS_FLAGS);
+               pm_runtime_put(dev);
        }
 }
 EXPORT_SYMBOL_GPL(dev_pm_qos_hide_flags);