summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBjorn Helgaas <bjorn.helgaas@hp.com>2008-06-28 00:57:07 +0200
committerAndi Kleen <andi@basil.nowhere.org>2008-07-16 23:27:06 +0200
commitfcfb7ce3d688d5c15fc9bc0a2a48e1ededdb046f (patch)
treebc0b01e17601ba76b5af45573595ab8f53b6441e
parentPNP: increase I/O port & memory option address sizes (diff)
downloadlinux-fcfb7ce3d688d5c15fc9bc0a2a48e1ededdb046f.tar.xz
linux-fcfb7ce3d688d5c15fc9bc0a2a48e1ededdb046f.zip
PNP: improve resource assignment debug
When we fail to assign an I/O or MEM resource, include the min/max in the debug output to help match it with the options. 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>
-rw-r--r--drivers/pnp/manager.c10
1 files changed, 8 insertions, 2 deletions
diff --git a/drivers/pnp/manager.c b/drivers/pnp/manager.c
index e758dd225576..c706dd2ddb02 100644
--- a/drivers/pnp/manager.c
+++ b/drivers/pnp/manager.c
@@ -47,7 +47,10 @@ static int pnp_assign_port(struct pnp_dev *dev, struct pnp_port *rule, int idx)
res->start += rule->align;
res->end = res->start + rule->size - 1;
if (res->start > rule->max || !rule->align) {
- dev_dbg(&dev->dev, " couldn't assign io %d\n", idx);
+ dev_dbg(&dev->dev, " couldn't assign io %d "
+ "(min %#llx max %#llx)\n", idx,
+ (unsigned long long) rule->min,
+ (unsigned long long) rule->max);
return 0;
}
}
@@ -96,7 +99,10 @@ static int pnp_assign_mem(struct pnp_dev *dev, struct pnp_mem *rule, int idx)
res->start += rule->align;
res->end = res->start + rule->size - 1;
if (res->start > rule->max || !rule->align) {
- dev_dbg(&dev->dev, " couldn't assign mem %d\n", idx);
+ dev_dbg(&dev->dev, " couldn't assign mem %d "
+ "(min %#llx max %#llx)\n", idx,
+ (unsigned long long) rule->min,
+ (unsigned long long) rule->max);
return 0;
}
}