Commit 25d39c39 authored by Bjorn Helgaas's avatar Bjorn Helgaas Committed by Andi Kleen

PNP: remove ratelimit on add resource failures

We used to have a fixed-size resource table.  If a device had
twenty resources when the table only had space for ten, we didn't
need ten warnings, so we added the ratelimit.

Now that we can dynamically allocate new resources, we should
only get failures if the allocation fails.  That should be
rare enough that we don't need to ratelimit the messages.
Signed-off-by: default avatarBjorn Helgaas <bjorn.helgaas@hp.com>
Signed-off-by: default avatarLen Brown <len.brown@intel.com>
Signed-off-by: default avatarAndi Kleen <ak@linux.intel.com>
parent 5acf9141
...@@ -537,15 +537,10 @@ struct pnp_resource *pnp_add_irq_resource(struct pnp_dev *dev, int irq, ...@@ -537,15 +537,10 @@ struct pnp_resource *pnp_add_irq_resource(struct pnp_dev *dev, int irq,
{ {
struct pnp_resource *pnp_res; struct pnp_resource *pnp_res;
struct resource *res; struct resource *res;
static unsigned char warned;
pnp_res = pnp_new_resource(dev); pnp_res = pnp_new_resource(dev);
if (!pnp_res) { if (!pnp_res) {
if (!warned) { dev_err(&dev->dev, "can't add resource for IRQ %d\n", irq);
dev_err(&dev->dev, "can't add resource for IRQ %d\n",
irq);
warned = 1;
}
return NULL; return NULL;
} }
...@@ -563,15 +558,10 @@ struct pnp_resource *pnp_add_dma_resource(struct pnp_dev *dev, int dma, ...@@ -563,15 +558,10 @@ struct pnp_resource *pnp_add_dma_resource(struct pnp_dev *dev, int dma,
{ {
struct pnp_resource *pnp_res; struct pnp_resource *pnp_res;
struct resource *res; struct resource *res;
static unsigned char warned;
pnp_res = pnp_new_resource(dev); pnp_res = pnp_new_resource(dev);
if (!pnp_res) { if (!pnp_res) {
if (!warned) { dev_err(&dev->dev, "can't add resource for DMA %d\n", dma);
dev_err(&dev->dev, "can't add resource for DMA %d\n",
dma);
warned = 1;
}
return NULL; return NULL;
} }
...@@ -590,16 +580,12 @@ struct pnp_resource *pnp_add_io_resource(struct pnp_dev *dev, ...@@ -590,16 +580,12 @@ struct pnp_resource *pnp_add_io_resource(struct pnp_dev *dev,
{ {
struct pnp_resource *pnp_res; struct pnp_resource *pnp_res;
struct resource *res; struct resource *res;
static unsigned char warned;
pnp_res = pnp_new_resource(dev); pnp_res = pnp_new_resource(dev);
if (!pnp_res) { if (!pnp_res) {
if (!warned) { dev_err(&dev->dev, "can't add resource for IO %#llx-%#llx\n",
dev_err(&dev->dev, "can't add resource for IO " (unsigned long long) start,
"%#llx-%#llx\n",(unsigned long long) start,
(unsigned long long) end); (unsigned long long) end);
warned = 1;
}
return NULL; return NULL;
} }
...@@ -619,16 +605,12 @@ struct pnp_resource *pnp_add_mem_resource(struct pnp_dev *dev, ...@@ -619,16 +605,12 @@ struct pnp_resource *pnp_add_mem_resource(struct pnp_dev *dev,
{ {
struct pnp_resource *pnp_res; struct pnp_resource *pnp_res;
struct resource *res; struct resource *res;
static unsigned char warned;
pnp_res = pnp_new_resource(dev); pnp_res = pnp_new_resource(dev);
if (!pnp_res) { if (!pnp_res) {
if (!warned) { dev_err(&dev->dev, "can't add resource for MEM %#llx-%#llx\n",
dev_err(&dev->dev, "can't add resource for MEM " (unsigned long long) start,
"%#llx-%#llx\n",(unsigned long long) start,
(unsigned long long) end); (unsigned long long) end);
warned = 1;
}
return NULL; return NULL;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment