PM / wakeup: Use pm_pr_dbg() instead of pr_debug()
authorStephen Boyd <swboyd@chromium.org>
Mon, 25 Mar 2019 17:24:56 +0000 (10:24 -0700)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 1 Apr 2019 21:54:47 +0000 (23:54 +0200)
These prints are useful if we're doing PM suspend debugging. Having them
at pr_debug() level means that we need to either enable DEBUG in this
file, or compile the kernel with dynamic debug capabilities. Both of
these options have drawbacks like custom compilation or opting into all
debug statements being included into the kernel image. Given that we
already have infrastructure to collect PM debugging information with
CONFIG_PM_DEBUG and friends, let's change the pr_debug usage here to be
pm_pr_dbg() instead so we can collect the wakeup information in the
kernel logs.

Signed-off-by: Stephen Boyd <swboyd@chromium.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/base/power/wakeup.c

index bb1ae175fae136a57cd85e7566af617706e29232..23c243a4c6754d73a7e51e42c8f671fbd8dac774 100644 (file)
@@ -804,7 +804,7 @@ void pm_print_active_wakeup_sources(void)
        srcuidx = srcu_read_lock(&wakeup_srcu);
        list_for_each_entry_rcu(ws, &wakeup_sources, entry) {
                if (ws->active) {
-                       pr_debug("active wakeup source: %s\n", ws->name);
+                       pm_pr_dbg("active wakeup source: %s\n", ws->name);
                        active = 1;
                } else if (!active &&
                           (!last_activity_ws ||
@@ -815,7 +815,7 @@ void pm_print_active_wakeup_sources(void)
        }
 
        if (!active && last_activity_ws)
-               pr_debug("last active wakeup source: %s\n",
+               pm_pr_dbg("last active wakeup source: %s\n",
                        last_activity_ws->name);
        srcu_read_unlock(&wakeup_srcu, srcuidx);
 }
@@ -845,7 +845,7 @@ bool pm_wakeup_pending(void)
        raw_spin_unlock_irqrestore(&events_lock, flags);
 
        if (ret) {
-               pr_debug("Wakeup pending, aborting suspend\n");
+               pm_pr_dbg("Wakeup pending, aborting suspend\n");
                pm_print_active_wakeup_sources();
        }