diff options
author | Bjorn Helgaas <bjorn.helgaas@hp.com> | 2008-06-27 18:57:08 -0400 |
---|---|---|
committer | Andi Kleen <andi@basil.nowhere.org> | 2008-07-16 17:27:06 -0400 |
commit | 819beac3806a5e986d81f476b999b7fffce1a233 (patch) | |
tree | e8e153c3c4b0fc58cc5e85654a59975d2222c38d /drivers/pnp | |
parent | fcfb7ce3d688d5c15fc9bc0a2a48e1ededdb046f (diff) |
PNP: in debug resource dump, make empty list obvious
If the resource list is empty, say that explicitly. Previously,
it was confusing because often the heading was followed by zero
resource lines, then some "add resource" lines from auto-assignment,
so the "add" lines looked like current resources.
Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com>
Signed-off-by: Andi Kleen <ak@linux.intel.com>
Acked-by: Rene Herman <rene.herman@gmail.com>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/pnp')
-rw-r--r-- | drivers/pnp/support.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/drivers/pnp/support.c b/drivers/pnp/support.c index 1566e4a73849..0ad42db94884 100644 --- a/drivers/pnp/support.c +++ b/drivers/pnp/support.c | |||
@@ -79,7 +79,12 @@ void dbg_pnp_show_resources(struct pnp_dev *dev, char *desc) | |||
79 | struct pnp_resource *pnp_res; | 79 | struct pnp_resource *pnp_res; |
80 | struct resource *res; | 80 | struct resource *res; |
81 | 81 | ||
82 | dev_dbg(&dev->dev, "current resources: %s\n", desc); | 82 | if (list_empty(&dev->resources)) { |
83 | dev_dbg(&dev->dev, "%s: no current resources\n", desc); | ||
84 | return; | ||
85 | } | ||
86 | |||
87 | dev_dbg(&dev->dev, "%s: current resources:\n", desc); | ||
83 | list_for_each_entry(pnp_res, &dev->resources, list) { | 88 | list_for_each_entry(pnp_res, &dev->resources, list) { |
84 | res = &pnp_res->res; | 89 | res = &pnp_res->res; |
85 | 90 | ||