lkml.org 
[lkml]   [2008]   [Apr]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[patch 26/54] PNP: factor pnp_init_resource_table() and pnp_clean_resource_table()
    Move the common part of pnp_init_resource_table() and
    pnp_clean_resource_table() into a new pnp_init_resource().
    This reduces a little code duplication and will be
    useful later to initialize an individual resource.

    Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com>

    ---
    drivers/pnp/base.h | 2 +
    drivers/pnp/manager.c | 90 ++++++++++++++++++++++++--------------------------
    2 files changed, 46 insertions(+), 46 deletions(-)

    Index: work10/drivers/pnp/base.h
    ===================================================================
    --- work10.orig/drivers/pnp/base.h 2008-04-25 11:14:57.000000000 -0600
    +++ work10/drivers/pnp/base.h 2008-04-25 11:15:00.000000000 -0600
    @@ -18,3 +18,5 @@
    int pnp_check_dma(struct pnp_dev * dev, int idx);

    void dbg_pnp_show_resources(struct pnp_dev *dev, char *desc);
    +
    +void pnp_init_resource(struct resource *res);
    Index: work10/drivers/pnp/manager.c
    ===================================================================
    --- work10.orig/drivers/pnp/manager.c 2008-04-25 11:14:59.000000000 -0600
    +++ work10/drivers/pnp/manager.c 2008-04-25 11:15:00.000000000 -0600
    @@ -234,42 +234,52 @@
    dev_dbg(&dev->dev, " disable dma %d\n", idx);
    }

    +void pnp_init_resource(struct resource *res)
    +{
    + unsigned long type;
    +
    + type = res->flags & (IORESOURCE_IO | IORESOURCE_MEM |
    + IORESOURCE_IRQ | IORESOURCE_DMA);
    +
    + res->name = NULL;
    + res->flags = type | IORESOURCE_AUTO | IORESOURCE_UNSET;
    + if (type == IORESOURCE_IRQ || type == IORESOURCE_DMA) {
    + res->start = -1;
    + res->end = -1;
    + } else {
    + res->start = 0;
    + res->end = 0;
    + }
    +}
    +
    /**
    * pnp_init_resources - Resets a resource table to default values.
    * @table: pointer to the desired resource table
    */
    void pnp_init_resources(struct pnp_dev *dev)
    {
    - struct pnp_resource_table *table = &dev->res;
    + struct resource *res;
    int idx;

    for (idx = 0; idx < PNP_MAX_IRQ; idx++) {
    - table->irq_resource[idx].name = NULL;
    - table->irq_resource[idx].start = -1;
    - table->irq_resource[idx].end = -1;
    - table->irq_resource[idx].flags =
    - IORESOURCE_IRQ | IORESOURCE_AUTO | IORESOURCE_UNSET;
    + res = &dev->res.irq_resource[idx];
    + res->flags = IORESOURCE_IRQ;
    + pnp_init_resource(res);
    }
    for (idx = 0; idx < PNP_MAX_DMA; idx++) {
    - table->dma_resource[idx].name = NULL;
    - table->dma_resource[idx].start = -1;
    - table->dma_resource[idx].end = -1;
    - table->dma_resource[idx].flags =
    - IORESOURCE_DMA | IORESOURCE_AUTO | IORESOURCE_UNSET;
    + res = &dev->res.dma_resource[idx];
    + res->flags = IORESOURCE_DMA;
    + pnp_init_resource(res);
    }
    for (idx = 0; idx < PNP_MAX_PORT; idx++) {
    - table->port_resource[idx].name = NULL;
    - table->port_resource[idx].start = 0;
    - table->port_resource[idx].end = 0;
    - table->port_resource[idx].flags =
    - IORESOURCE_IO | IORESOURCE_AUTO | IORESOURCE_UNSET;
    + res = &dev->res.port_resource[idx];
    + res->flags = IORESOURCE_IO;
    + pnp_init_resource(res);
    }
    for (idx = 0; idx < PNP_MAX_MEM; idx++) {
    - table->mem_resource[idx].name = NULL;
    - table->mem_resource[idx].start = 0;
    - table->mem_resource[idx].end = 0;
    - table->mem_resource[idx].flags =
    - IORESOURCE_MEM | IORESOURCE_AUTO | IORESOURCE_UNSET;
    + res = &dev->res.mem_resource[idx];
    + res->flags = IORESOURCE_MEM;
    + pnp_init_resource(res);
    }
    }

    @@ -279,40 +289,28 @@
    */
    static void pnp_clean_resource_table(struct pnp_dev *dev)
    {
    - struct pnp_resource_table *res = &dev->res;
    + struct resource *res;
    int idx;

    for (idx = 0; idx < PNP_MAX_IRQ; idx++) {
    - if (!(res->irq_resource[idx].flags & IORESOURCE_AUTO))
    - continue;
    - res->irq_resource[idx].start = -1;
    - res->irq_resource[idx].end = -1;
    - res->irq_resource[idx].flags =
    - IORESOURCE_IRQ | IORESOURCE_AUTO | IORESOURCE_UNSET;
    + res = &dev->res.irq_resource[idx];
    + if (res->flags & IORESOURCE_AUTO)
    + pnp_init_resource(res);
    }
    for (idx = 0; idx < PNP_MAX_DMA; idx++) {
    - if (!(res->dma_resource[idx].flags & IORESOURCE_AUTO))
    - continue;
    - res->dma_resource[idx].start = -1;
    - res->dma_resource[idx].end = -1;
    - res->dma_resource[idx].flags =
    - IORESOURCE_DMA | IORESOURCE_AUTO | IORESOURCE_UNSET;
    + res = &dev->res.dma_resource[idx];
    + if (res->flags & IORESOURCE_AUTO)
    + pnp_init_resource(res);
    }
    for (idx = 0; idx < PNP_MAX_PORT; idx++) {
    - if (!(res->port_resource[idx].flags & IORESOURCE_AUTO))
    - continue;
    - res->port_resource[idx].start = 0;
    - res->port_resource[idx].end = 0;
    - res->port_resource[idx].flags =
    - IORESOURCE_IO | IORESOURCE_AUTO | IORESOURCE_UNSET;
    + res = &dev->res.port_resource[idx];
    + if (res->flags & IORESOURCE_AUTO)
    + pnp_init_resource(res);
    }
    for (idx = 0; idx < PNP_MAX_MEM; idx++) {
    - if (!(res->mem_resource[idx].flags & IORESOURCE_AUTO))
    - continue;
    - res->mem_resource[idx].start = 0;
    - res->mem_resource[idx].end = 0;
    - res->mem_resource[idx].flags =
    - IORESOURCE_MEM | IORESOURCE_AUTO | IORESOURCE_UNSET;
    + res = &dev->res.mem_resource[idx];
    + if (res->flags & IORESOURCE_AUTO)
    + pnp_init_resource(res);
    }
    }

    --


    \
     
     \ /
      Last update: 2008-04-25 20:53    [W:4.325 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site