sm501: gpio dynamic registration for PCI devices
authorArnaud Patard <apatard@mandriva.com>
Fri, 25 Jul 2008 08:46:00 +0000 (01:46 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 25 Jul 2008 17:53:30 +0000 (10:53 -0700)
The SM501 PCI card requires a dyanmic gpio allocation as the number of
cards is not known at compile time.  Fixup the platform data and
registration to deal with this.

Acked-by: Ben Dooks <ben-linux@fluff.org>
Signed-off-by: Arnaud Patard <apatard@mandriva.com>
Cc: David Brownell <david-b@pacbell.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/mfd/sm501.c
include/linux/sm501.h

index be87139081255ad6375cf8b473c8e48eccb7ad61..c3e5a48f6148d912d009dd009453d2e13130068f 100644 (file)
@@ -996,12 +996,13 @@ static int __devinit sm501_gpio_register_chip(struct sm501_devdata *sm,
 {
        struct sm501_platdata *pdata = sm->platdata;
        struct gpio_chip *gchip = &chip->gpio;
-       unsigned base = pdata->gpio_base;
+       int base = pdata->gpio_base;
 
        memcpy(chip, &gpio_chip_template, sizeof(struct gpio_chip));
 
        if (chip == &gpio->high) {
-               base += 32;
+               if (base > 0)
+                       base += 32;
                chip->regbase = gpio->regs + SM501_GPIO_DATA_HIGH;
                gchip->label  = "SM501-HIGH";
        } else {
@@ -1452,6 +1453,7 @@ static struct sm501_platdata_fb sm501_fb_pdata = {
 static struct sm501_platdata sm501_pci_platdata = {
        .init           = &sm501_pci_initdata,
        .fb             = &sm501_fb_pdata,
+       .gpio_base      = -1,
 };
 
 static int sm501_pci_probe(struct pci_dev *dev,
index 6ea39007c8a3ef0a96bb923152f67ad6e83ceee1..a8d02f36ad3263485f4afccdeba32e5a5041f5c6 100644 (file)
@@ -156,7 +156,7 @@ struct sm501_platdata {
        struct sm501_platdata_fb        *fb;
 
        int                              flags;
-       unsigned                         gpio_base;
+       int                              gpio_base;
 
        int     (*get_power)(struct device *dev);
        int     (*set_power)(struct device *dev, unsigned int on);