PNP: make pnp_resource_table private to PNP core

There are no remaining references to the PNP_MAX_* constants or
the pnp_resource_table structure outside of the PNP core.  Make
them private to the PNP core.

Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com>
Signed-off-by: Len Brown <len.brown@intel.com>

8 files changed, 43 insertions, 31 deletions

diff --git a/drivers/pnp/base.h b/drivers/pnp/base.h
index e739d4bba423..b888a5fb6b7f 100644
--- a/drivers/pnp/base.h
+++ b/drivers/pnp/base.h
@@ -20,3 +20,15 @@ int pnp_check_dma(struct pnp_dev *dev, struct resource *res);
 void dbg_pnp_show_resources(struct pnp_dev *dev, char *desc);
 
 void pnp_init_resource(struct resource *res);
+
+#define PNP_MAX_PORT            40
+#define PNP_MAX_MEM             24
+#define PNP_MAX_IRQ              2
+#define PNP_MAX_DMA              2
+
+struct pnp_resource_table {
+        struct resource port_resource[PNP_MAX_PORT];
+        struct resource mem_resource[PNP_MAX_MEM];
+        struct resource dma_resource[PNP_MAX_DMA];
+        struct resource irq_resource[PNP_MAX_IRQ];
+};
diff --git a/drivers/pnp/core.c b/drivers/pnp/core.c
index cf37701a4f9e..20771b7d4482 100644
--- a/drivers/pnp/core.c
+++ b/drivers/pnp/core.c
@@ -106,6 +106,7 @@ static void pnp_release_device(struct device *dmdev)
         pnp_free_option(dev->independent);
         pnp_free_option(dev->dependent);
         pnp_free_ids(dev);
+        kfree(dev->res);
         kfree(dev);
 }
 
@@ -118,6 +119,12 @@ struct pnp_dev *pnp_alloc_dev(struct pnp_protocol *protocol, int id, char *pnpid
         if (!dev)
                 return NULL;
 
+        dev->res = kzalloc(sizeof(struct pnp_resource_table), GFP_KERNEL);
+        if (!dev->res) {
+                kfree(dev);
+                return NULL;
+        }
+
         dev->protocol = protocol;
         dev->number = id;
         dev->dma_mask = DMA_24BIT_MASK;
@@ -133,6 +140,7 @@ struct pnp_dev *pnp_alloc_dev(struct pnp_protocol *protocol, int id, char *pnpid
 
         dev_id = pnp_add_id(dev, pnpid);
         if (!dev_id) {
+                kfree(dev->res);
                 kfree(dev);
                 return NULL;
         }
diff --git a/drivers/pnp/isapnp/core.c b/drivers/pnp/isapnp/core.c
index 990d8cd6295c..4407e844b5ea 100644
--- a/drivers/pnp/isapnp/core.c
+++ b/drivers/pnp/isapnp/core.c
@@ -931,7 +931,7 @@ EXPORT_SYMBOL(isapnp_write_byte);
 
 static int isapnp_read_resources(struct pnp_dev *dev)
 {
-        struct pnp_resource_table *res = &dev->res;
+        struct pnp_resource_table *res = dev->res;
         int tmp, ret;
 
         dev->active = isapnp_read_byte(ISAPNP_CFG_ACTIVATE);
@@ -987,7 +987,7 @@ static int isapnp_get_resources(struct pnp_dev *dev)
 
 static int isapnp_set_resources(struct pnp_dev *dev)
 {
-        struct pnp_resource_table *res = &dev->res;
+        struct pnp_resource_table *res = dev->res;
         int tmp;
 
         dev_dbg(&dev->dev, "set resources\n");
diff --git a/drivers/pnp/manager.c b/drivers/pnp/manager.c
index 7c5ebddfc6af..46a5e0e90d9a 100644
--- a/drivers/pnp/manager.c
+++ b/drivers/pnp/manager.c
@@ -247,22 +247,22 @@ void pnp_init_resources(struct pnp_dev *dev)
         int idx;
 
         for (idx = 0; idx < PNP_MAX_IRQ; idx++) {
-                res = &dev->res.irq_resource[idx];
+                res = &dev->res->irq_resource[idx];
                 res->flags = IORESOURCE_IRQ;
                 pnp_init_resource(res);
         }
         for (idx = 0; idx < PNP_MAX_DMA; idx++) {
-                res = &dev->res.dma_resource[idx];
+                res = &dev->res->dma_resource[idx];
                 res->flags = IORESOURCE_DMA;
                 pnp_init_resource(res);
         }
         for (idx = 0; idx < PNP_MAX_PORT; idx++) {
-                res = &dev->res.port_resource[idx];
+                res = &dev->res->port_resource[idx];
                 res->flags = IORESOURCE_IO;
                 pnp_init_resource(res);
         }
         for (idx = 0; idx < PNP_MAX_MEM; idx++) {
-                res = &dev->res.mem_resource[idx];
+                res = &dev->res->mem_resource[idx];
                 res->flags = IORESOURCE_MEM;
                 pnp_init_resource(res);
         }
@@ -278,28 +278,28 @@ static void pnp_clean_resource_table(struct pnp_dev *dev)
         int idx;
 
         for (idx = 0; idx < PNP_MAX_IRQ; idx++) {
-                res = &dev->res.irq_resource[idx];
+                res = &dev->res->irq_resource[idx];
                 if (res->flags & IORESOURCE_AUTO) {
                         res->flags = IORESOURCE_IRQ;
                         pnp_init_resource(res);
                 }
         }
         for (idx = 0; idx < PNP_MAX_DMA; idx++) {
-                res = &dev->res.dma_resource[idx];
+                res = &dev->res->dma_resource[idx];
                 if (res->flags & IORESOURCE_AUTO) {
                         res->flags = IORESOURCE_DMA;
                         pnp_init_resource(res);
                 }
         }
         for (idx = 0; idx < PNP_MAX_PORT; idx++) {
-                res = &dev->res.port_resource[idx];
+                res = &dev->res->port_resource[idx];
                 if (res->flags & IORESOURCE_AUTO) {
                         res->flags = IORESOURCE_IO;
                         pnp_init_resource(res);
                 }
         }
         for (idx = 0; idx < PNP_MAX_MEM; idx++) {
-                res = &dev->res.mem_resource[idx];
+                res = &dev->res->mem_resource[idx];
                 if (res->flags & IORESOURCE_AUTO) {
                         res->flags = IORESOURCE_MEM;
                         pnp_init_resource(res);
diff --git a/drivers/pnp/pnpacpi/rsparser.c b/drivers/pnp/pnpacpi/rsparser.c
index 000a1b39f0b6..2669518b4795 100644
--- a/drivers/pnp/pnpacpi/rsparser.c
+++ b/drivers/pnp/pnpacpi/rsparser.c
@@ -21,6 +21,8 @@
 #include <linux/kernel.h>
 #include <linux/acpi.h>
 #include <linux/pci.h>
+#include <linux/pnp.h>
+#include "../base.h"
 #include "pnpacpi.h"
 
 #ifdef CONFIG_IA64
@@ -80,7 +82,7 @@ static void pnpacpi_parse_allocated_irqresource(struct pnp_dev *dev,
                                                 u32 gsi, int triggering,
                                                 int polarity, int shareable)
 {
-        struct pnp_resource_table *res = &dev->res;
+        struct pnp_resource_table *res = dev->res;
         int i = 0;
         int irq;
         int p, t;
@@ -176,7 +178,7 @@ static int dma_flags(int type, int bus_master, int transfer)
 static void pnpacpi_parse_allocated_dmaresource(struct pnp_dev *dev,
                                                 u32 dma, int flags)
 {
-        struct pnp_resource_table *res = &dev->res;
+        struct pnp_resource_table *res = dev->res;
         int i = 0;
         static unsigned char warned;
 
@@ -202,7 +204,7 @@ static void pnpacpi_parse_allocated_dmaresource(struct pnp_dev *dev,
 static void pnpacpi_parse_allocated_ioresource(struct pnp_dev *dev,
                                                u64 io, u64 len, int io_decode)
 {
-        struct pnp_resource_table *res = &dev->res;
+        struct pnp_resource_table *res = dev->res;
         int i = 0;
         static unsigned char warned;
 
@@ -230,7 +232,7 @@ static void pnpacpi_parse_allocated_memresource(struct pnp_dev *dev,
                                                 u64 mem, u64 len,
                                                 int write_protect)
 {
-        struct pnp_resource_table *res = &dev->res;
+        struct pnp_resource_table *res = dev->res;
         int i = 0;
         static unsigned char warned;
 
diff --git a/drivers/pnp/pnpbios/rsparser.c b/drivers/pnp/pnpbios/rsparser.c
index c1f9e162d2c5..9f0538af0321 100644
--- a/drivers/pnp/pnpbios/rsparser.c
+++ b/drivers/pnp/pnpbios/rsparser.c
@@ -56,7 +56,7 @@ inline void pcibios_penalize_isa_irq(int irq, int active)
 
 static void pnpbios_parse_allocated_irqresource(struct pnp_dev *dev, int irq)
 {
-        struct pnp_resource_table *res = &dev->res;
+        struct pnp_resource_table *res = dev->res;
         int i = 0;
 
         while (!(res->irq_resource[i].flags & IORESOURCE_UNSET)
@@ -76,7 +76,7 @@ static void pnpbios_parse_allocated_irqresource(struct pnp_dev *dev, int irq)
 
 static void pnpbios_parse_allocated_dmaresource(struct pnp_dev *dev, int dma)
 {
-        struct pnp_resource_table *res = &dev->res;
+        struct pnp_resource_table *res = dev->res;
         int i = 0;
 
         while (i < PNP_MAX_DMA &&
@@ -96,7 +96,7 @@ static void pnpbios_parse_allocated_dmaresource(struct pnp_dev *dev, int dma)
 static void pnpbios_parse_allocated_ioresource(struct pnp_dev *dev,
                                                int io, int len)
 {
-        struct pnp_resource_table *res = &dev->res;
+        struct pnp_resource_table *res = dev->res;
         int i = 0;
 
         while (!(res->port_resource[i].flags & IORESOURCE_UNSET)
@@ -116,7 +116,7 @@ static void pnpbios_parse_allocated_ioresource(struct pnp_dev *dev,
 static void pnpbios_parse_allocated_memresource(struct pnp_dev *dev,
                                                 int mem, int len)
 {
-        struct pnp_resource_table *res = &dev->res;
+        struct pnp_resource_table *res = dev->res;
         int i = 0;
 
         while (!(res->mem_resource[i].flags & IORESOURCE_UNSET)
diff --git a/drivers/pnp/resource.c b/drivers/pnp/resource.c
index 84362818fa8b..f7adc7eefbf8 100644
--- a/drivers/pnp/resource.c
+++ b/drivers/pnp/resource.c
@@ -502,7 +502,7 @@ int pnp_check_dma(struct pnp_dev *dev, struct resource *res)
 struct resource *pnp_get_resource(struct pnp_dev *dev,
                                   unsigned int type, unsigned int num)
 {
-        struct pnp_resource_table *res = &dev->res;
+        struct pnp_resource_table *res = dev->res;
 
         switch (type) {
         case IORESOURCE_IO:
diff --git a/include/linux/pnp.h b/include/linux/pnp.h
index 1640562f3ebc..a5487b6a4e57 100644
--- a/include/linux/pnp.h
+++ b/include/linux/pnp.h
@@ -13,14 +13,11 @@
 #include <linux/errno.h>
 #include <linux/mod_devicetable.h>
 
-#define PNP_MAX_PORT            40
-#define PNP_MAX_MEM             24
-#define PNP_MAX_IRQ             2
-#define PNP_MAX_DMA             2
 #define PNP_NAME_LEN            50
 
 struct pnp_protocol;
 struct pnp_dev;
+struct pnp_resource_table;
 
 /*
  * Resource Management
@@ -184,13 +181,6 @@ struct pnp_option {
         struct pnp_option *next;        /* used to chain dependent resources */
 };
 
-struct pnp_resource_table {
-        struct resource port_resource[PNP_MAX_PORT];
-        struct resource mem_resource[PNP_MAX_MEM];
-        struct resource dma_resource[PNP_MAX_DMA];
-        struct resource irq_resource[PNP_MAX_IRQ];
-};
-
 /*
  * Device Management
  */
@@ -260,7 +250,7 @@ struct pnp_dev {
         int capabilities;
         struct pnp_option *independent;
         struct pnp_option *dependent;
-        struct pnp_resource_table res;
+        struct pnp_resource_table *res;
 
         char name[PNP_NAME_LEN];        /* contains a human-readable name */
         unsigned short regs;            /* ISAPnP: supported registers */