summaryrefslogtreecommitdiffstats
path: root/drivers/pnp/manager.c
diff options
context:
space:
mode:
authorBjorn Helgaas <bjorn.helgaas@hp.com>2008-04-28 16:34:25 -0600
committerLen Brown <len.brown@intel.com>2008-04-29 03:22:26 -0400
commitf6505fef18644557f732468c1f22f84560d8a819 (patch)
tree942bf646b502369e3f8e8f398265d0872e8d3a0c /drivers/pnp/manager.c
parent7e2cf31f1c97ac14b6d9dc5f1ce9e1e01aef9c18 (diff)
downloadop-kernel-dev-f6505fef18644557f732468c1f22f84560d8a819.zip
op-kernel-dev-f6505fef18644557f732468c1f22f84560d8a819.tar.gz
PNP: convert assign, interface to use pnp_get_resource(), not pnp_resource_table
This removes more direct references to pnp_resource_table from the pnp_assign_resources() path and the /sys user interface path. Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com> Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/pnp/manager.c')
-rw-r--r--drivers/pnp/manager.c20
1 files changed, 8 insertions, 12 deletions
diff --git a/drivers/pnp/manager.c b/drivers/pnp/manager.c
index 0886529..7c5ebdd 100644
--- a/drivers/pnp/manager.c
+++ b/drivers/pnp/manager.c
@@ -21,14 +21,13 @@ static int pnp_assign_port(struct pnp_dev *dev, struct pnp_port *rule, int idx)
{
struct resource *res;
- if (idx >= PNP_MAX_PORT) {
+ res = pnp_get_resource(dev, IORESOURCE_IO, idx);
+ if (!res) {
dev_err(&dev->dev, "too many I/O port resources\n");
/* pretend we were successful so at least the manager won't try again */
return 1;
}
- res = &dev->res.port_resource[idx];
-
/* check if this resource has been manually set, if so skip */
if (!(res->flags & IORESOURCE_AUTO)) {
dev_dbg(&dev->dev, " io %d already set to %#llx-%#llx "
@@ -68,14 +67,13 @@ static int pnp_assign_mem(struct pnp_dev *dev, struct pnp_mem *rule, int idx)
{
struct resource *res;
- if (idx >= PNP_MAX_MEM) {
+ res = pnp_get_resource(dev, IORESOURCE_MEM, idx);
+ if (!res) {
dev_err(&dev->dev, "too many memory resources\n");
/* pretend we were successful so at least the manager won't try again */
return 1;
}
- res = &dev->res.mem_resource[idx];
-
/* check if this resource has been manually set, if so skip */
if (!(res->flags & IORESOURCE_AUTO)) {
dev_dbg(&dev->dev, " mem %d already set to %#llx-%#llx "
@@ -131,14 +129,13 @@ static int pnp_assign_irq(struct pnp_dev *dev, struct pnp_irq *rule, int idx)
5, 10, 11, 12, 9, 14, 15, 7, 3, 4, 13, 0, 1, 6, 8, 2
};
- if (idx >= PNP_MAX_IRQ) {
+ res = pnp_get_resource(dev, IORESOURCE_IRQ, idx);
+ if (!res) {
dev_err(&dev->dev, "too many IRQ resources\n");
/* pretend we were successful so at least the manager won't try again */
return 1;
}
- res = &dev->res.irq_resource[idx];
-
/* check if this resource has been manually set, if so skip */
if (!(res->flags & IORESOURCE_AUTO)) {
dev_dbg(&dev->dev, " irq %d already set to %d flags %#lx\n",
@@ -188,13 +185,12 @@ static void pnp_assign_dma(struct pnp_dev *dev, struct pnp_dma *rule, int idx)
1, 3, 5, 6, 7, 0, 2, 4
};
- if (idx >= PNP_MAX_DMA) {
+ res = pnp_get_resource(dev, IORESOURCE_DMA, idx);
+ if (!res) {
dev_err(&dev->dev, "too many DMA resources\n");
return;
}
- res = &dev->res.dma_resource[idx];
-
/* check if this resource has been manually set, if so skip */
if (!(res->flags & IORESOURCE_AUTO)) {
dev_dbg(&dev->dev, " dma %d already set to %d flags %#lx\n",
OpenPOWER on IntegriCloud