diff options
author | Arjan van de Ven <arjan@linux.intel.com> | 2008-09-20 22:09:00 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-10-16 12:24:50 -0400 |
commit | 728f08934b087b96aacb00467f5551e0a5593fca (patch) | |
tree | 94342640e5bca8b8c350fecbd05ff4e9bef0f8d0 | |
parent | e61396627f91abb855ddd8925be9172fb5871944 (diff) |
debug: use dev_WARN() rather than WARN_ON() in device_pm_add()
device_pm_add() has a WARN_ON that is showing relatively high on
kerneloops.org, but unfortunately the WARN_ON is less than useful
in that it doesn't print any information about what device is causing
the issue.
This patch fixes this by turning the WARN_ON() into the newly
introduces dev_WARN() which will print information about the
device in question.
Signed-off-by: Arjan van de Ven <arjan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r-- | drivers/base/power/main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c index 273a944d4040..b0eb6afdd861 100644 --- a/drivers/base/power/main.c +++ b/drivers/base/power/main.c | |||
@@ -83,7 +83,7 @@ void device_pm_add(struct device *dev) | |||
83 | * transition is in progress in order to avoid leaving them | 83 | * transition is in progress in order to avoid leaving them |
84 | * unhandled down the road | 84 | * unhandled down the road |
85 | */ | 85 | */ |
86 | WARN_ON(true); | 86 | dev_WARN(dev, "Parentless device registered during a PM transaction\n"); |
87 | } | 87 | } |
88 | 88 | ||
89 | list_add_tail(&dev->power.entry, &dpm_list); | 89 | list_add_tail(&dev->power.entry, &dpm_list); |