diff options
author | Bjorn Helgaas <bjorn.helgaas@hp.com> | 2008-04-28 18:34:23 -0400 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2008-04-29 03:22:26 -0400 |
commit | be81b4a4838ce329b9f3978c7fc007b047c23722 (patch) | |
tree | 69ddf0e4ef6fe0c38388d3d7b7ea18ad0344d579 /drivers | |
parent | f5d94ff014cb7e6212f40fc6644f3fd68507df33 (diff) |
PNP: convert resource checks to use pnp_get_resource(), not pnp_resource_table
This removes more direct references to pnp_resource_table.
Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com>
Acked-By: Rene Herman <rene.herman@gmail.com>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/pnp/resource.c | 32 |
1 files changed, 16 insertions, 16 deletions
diff --git a/drivers/pnp/resource.c b/drivers/pnp/resource.c index 93bf45e01f2c..b2516d62fcf6 100644 --- a/drivers/pnp/resource.c +++ b/drivers/pnp/resource.c | |||
@@ -270,8 +270,8 @@ int pnp_check_port(struct pnp_dev *dev, struct resource *res) | |||
270 | 270 | ||
271 | /* check for internal conflicts */ | 271 | /* check for internal conflicts */ |
272 | for (i = 0; i < PNP_MAX_PORT; i++) { | 272 | for (i = 0; i < PNP_MAX_PORT; i++) { |
273 | tres = &dev->res.port_resource[i]; | 273 | tres = pnp_get_resource(dev, IORESOURCE_IO, i); |
274 | if (tres != res && tres->flags & IORESOURCE_IO) { | 274 | if (tres && tres != res && tres->flags & IORESOURCE_IO) { |
275 | tport = &tres->start; | 275 | tport = &tres->start; |
276 | tend = &tres->end; | 276 | tend = &tres->end; |
277 | if (ranged_conflict(port, end, tport, tend)) | 277 | if (ranged_conflict(port, end, tport, tend)) |
@@ -284,8 +284,8 @@ int pnp_check_port(struct pnp_dev *dev, struct resource *res) | |||
284 | if (tdev == dev) | 284 | if (tdev == dev) |
285 | continue; | 285 | continue; |
286 | for (i = 0; i < PNP_MAX_PORT; i++) { | 286 | for (i = 0; i < PNP_MAX_PORT; i++) { |
287 | tres = &tdev->res.port_resource[i]; | 287 | tres = pnp_get_resource(tdev, IORESOURCE_IO, i); |
288 | if (tres->flags & IORESOURCE_IO) { | 288 | if (tres && tres->flags & IORESOURCE_IO) { |
289 | if (cannot_compare(tres->flags)) | 289 | if (cannot_compare(tres->flags)) |
290 | continue; | 290 | continue; |
291 | tport = &tres->start; | 291 | tport = &tres->start; |
@@ -330,8 +330,8 @@ int pnp_check_mem(struct pnp_dev *dev, struct resource *res) | |||
330 | 330 | ||
331 | /* check for internal conflicts */ | 331 | /* check for internal conflicts */ |
332 | for (i = 0; i < PNP_MAX_MEM; i++) { | 332 | for (i = 0; i < PNP_MAX_MEM; i++) { |
333 | tres = &dev->res.mem_resource[i]; | 333 | tres = pnp_get_resource(dev, IORESOURCE_MEM, i); |
334 | if (tres != res && tres->flags & IORESOURCE_MEM) { | 334 | if (tres && tres != res && tres->flags & IORESOURCE_MEM) { |
335 | taddr = &tres->start; | 335 | taddr = &tres->start; |
336 | tend = &tres->end; | 336 | tend = &tres->end; |
337 | if (ranged_conflict(addr, end, taddr, tend)) | 337 | if (ranged_conflict(addr, end, taddr, tend)) |
@@ -344,8 +344,8 @@ int pnp_check_mem(struct pnp_dev *dev, struct resource *res) | |||
344 | if (tdev == dev) | 344 | if (tdev == dev) |
345 | continue; | 345 | continue; |
346 | for (i = 0; i < PNP_MAX_MEM; i++) { | 346 | for (i = 0; i < PNP_MAX_MEM; i++) { |
347 | tres = &tdev->res.mem_resource[i]; | 347 | tres = pnp_get_resource(tdev, IORESOURCE_MEM, i); |
348 | if (tres->flags & IORESOURCE_MEM) { | 348 | if (tres && tres->flags & IORESOURCE_MEM) { |
349 | if (cannot_compare(tres->flags)) | 349 | if (cannot_compare(tres->flags)) |
350 | continue; | 350 | continue; |
351 | taddr = &tres->start; | 351 | taddr = &tres->start; |
@@ -389,8 +389,8 @@ int pnp_check_irq(struct pnp_dev *dev, struct resource *res) | |||
389 | 389 | ||
390 | /* check for internal conflicts */ | 390 | /* check for internal conflicts */ |
391 | for (i = 0; i < PNP_MAX_IRQ; i++) { | 391 | for (i = 0; i < PNP_MAX_IRQ; i++) { |
392 | tres = &dev->res.irq_resource[i]; | 392 | tres = pnp_get_resource(dev, IORESOURCE_IRQ, i); |
393 | if (tres != res && tres->flags & IORESOURCE_IRQ) { | 393 | if (tres && tres != res && tres->flags & IORESOURCE_IRQ) { |
394 | if (tres->start == *irq) | 394 | if (tres->start == *irq) |
395 | return 0; | 395 | return 0; |
396 | } | 396 | } |
@@ -423,8 +423,8 @@ int pnp_check_irq(struct pnp_dev *dev, struct resource *res) | |||
423 | if (tdev == dev) | 423 | if (tdev == dev) |
424 | continue; | 424 | continue; |
425 | for (i = 0; i < PNP_MAX_IRQ; i++) { | 425 | for (i = 0; i < PNP_MAX_IRQ; i++) { |
426 | tres = &tdev->res.irq_resource[i]; | 426 | tres = pnp_get_resource(tdev, IORESOURCE_IRQ, i); |
427 | if (tres->flags & IORESOURCE_IRQ) { | 427 | if (tres && tres->flags & IORESOURCE_IRQ) { |
428 | if (cannot_compare(tres->flags)) | 428 | if (cannot_compare(tres->flags)) |
429 | continue; | 429 | continue; |
430 | if (tres->start == *irq) | 430 | if (tres->start == *irq) |
@@ -462,8 +462,8 @@ int pnp_check_dma(struct pnp_dev *dev, struct resource *res) | |||
462 | 462 | ||
463 | /* check for internal conflicts */ | 463 | /* check for internal conflicts */ |
464 | for (i = 0; i < PNP_MAX_DMA; i++) { | 464 | for (i = 0; i < PNP_MAX_DMA; i++) { |
465 | tres = &dev->res.dma_resource[i]; | 465 | tres = pnp_get_resource(dev, IORESOURCE_DMA, i); |
466 | if (tres != res && tres->flags & IORESOURCE_DMA) { | 466 | if (tres && tres != res && tres->flags & IORESOURCE_DMA) { |
467 | if (tres->start == *dma) | 467 | if (tres->start == *dma) |
468 | return 0; | 468 | return 0; |
469 | } | 469 | } |
@@ -482,8 +482,8 @@ int pnp_check_dma(struct pnp_dev *dev, struct resource *res) | |||
482 | if (tdev == dev) | 482 | if (tdev == dev) |
483 | continue; | 483 | continue; |
484 | for (i = 0; i < PNP_MAX_DMA; i++) { | 484 | for (i = 0; i < PNP_MAX_DMA; i++) { |
485 | tres = &tdev->res.dma_resource[i]; | 485 | tres = pnp_get_resource(tdev, IORESOURCE_DMA, i); |
486 | if (tres->flags & IORESOURCE_DMA) { | 486 | if (tres && tres->flags & IORESOURCE_DMA) { |
487 | if (cannot_compare(tres->flags)) | 487 | if (cannot_compare(tres->flags)) |
488 | continue; | 488 | continue; |
489 | if (tres->start == *dma) | 489 | if (tres->start == *dma) |