Merge tag 'pm-fixes-4.15-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm

Pull two power management fixes from Rafael Wysocki:
 "This is the change making /proc/cpuinfo on x86 report current CPU
  frequency in "cpu MHz" again in all cases and an additional one
  dealing with an overzealous check in one of the helper routines in the
  runtime PM framework"

* tag 'pm-fixes-4.15-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm:
  PM / runtime: Drop children check from __pm_runtime_set_status()
  x86 / CPU: Always show current CPU frequency in /proc/cpuinfo

1 files changed, 11 insertions, 20 deletions

diff --git a/drivers/base/power/runtime.c b/drivers/base/power/runtime.c
index 2362b9e9701e..027d159ac381 100644
--- a/drivers/base/power/runtime.c
+++ b/drivers/base/power/runtime.c
@@ -1101,29 +1101,13 @@ int __pm_runtime_set_status(struct device *dev, unsigned int status)
                 goto out;
         }
 
-        if (dev->power.runtime_status == status)
+        if (dev->power.runtime_status == status || !parent)
                 goto out_set;
 
         if (status == RPM_SUSPENDED) {
-                /*
-                 * It is invalid to suspend a device with an active child,
-                 * unless it has been set to ignore its children.
-                 */
-                if (!dev->power.ignore_children &&
-                        atomic_read(&dev->power.child_count)) {
-                        dev_err(dev, "runtime PM trying to suspend device but active child\n");
-                        error = -EBUSY;
-                        goto out;
-                }
-
-                if (parent) {
-                        atomic_add_unless(&parent->power.child_count, -1, 0);
-                        notify_parent = !parent->power.ignore_children;
-                }
-                goto out_set;
-        }
-
-        if (parent) {
+                atomic_add_unless(&parent->power.child_count, -1, 0);
+                notify_parent = !parent->power.ignore_children;
+        } else {
                 spin_lock_nested(&parent->power.lock, SINGLE_DEPTH_NESTING);
 
                 /*
@@ -1307,6 +1291,13 @@ void pm_runtime_enable(struct device *dev)
         else
                 dev_warn(dev, "Unbalanced %s!\n", __func__);
 
+        WARN(!dev->power.disable_depth &&
+             dev->power.runtime_status == RPM_SUSPENDED &&
+             !dev->power.ignore_children &&
+             atomic_read(&dev->power.child_count) > 0,
+             "Enabling runtime PM for inactive device (%s) with active children\n",
+             dev_name(dev));
+
         spin_unlock_irqrestore(&dev->power.lock, flags);
 }
 EXPORT_SYMBOL_GPL(pm_runtime_enable);