gpio/omap: move bank width into struct gpio_bank
authorKevin Hilman <khilman@ti.com>
Thu, 21 Apr 2011 16:23:00 +0000 (09:23 -0700)
committerKevin Hilman <khilman@ti.com>
Thu, 16 Jun 2011 18:13:13 +0000 (11:13 -0700)
Rather than having a file-global bank_width variable, move it into
struct gpio_bank so it can be bank-specific.   Note the bank width
is already passed per-bank via platform_data, so current code would
be incorrect if any banks had different width.

Signed-off-by: Kevin Hilman <khilman@ti.com>
drivers/gpio/gpio-omap.c

index 7135810a77bc7cc4c09862b27a200af75dfde717..3f436c03361180bc42f43e06ceab60d1409883dd 100644 (file)
@@ -54,6 +54,7 @@ struct gpio_bank {
        struct device *dev;
        bool dbck_flag;
        int stride;
+       u32 width;
 };
 
 #ifdef CONFIG_ARCH_OMAP3
@@ -79,8 +80,6 @@ static struct omap3_gpio_regs gpio_context[OMAP34XX_NR_GPIOS];
  */
 static struct gpio_bank *gpio_bank;
 
-static int bank_width;
-
 /* TODO: Analyze removing gpio_bank_count usage from driver code */
 int gpio_bank_count;
 
@@ -870,7 +869,7 @@ static int _set_gpio_wakeup(struct gpio_bank *bank, int gpio, int enable)
                if (bank->non_wakeup_gpios & (1 << gpio)) {
                        printk(KERN_ERR "Unable to modify wakeup on "
                                        "non-wakeup GPIO%d\n",
-                                       (bank - gpio_bank) * 32 + gpio);
+                              (bank - gpio_bank) * bank->width + gpio);
                        return -EINVAL;
                }
                spin_lock_irqsave(&bank->lock, flags);
@@ -1545,14 +1544,14 @@ static void __devinit omap_gpio_chip_init(struct gpio_bank *bank)
        } else {
                bank->chip.label = "gpio";
                bank->chip.base = gpio;
-               gpio += bank_width;
+               gpio += bank->width;
        }
-       bank->chip.ngpio = bank_width;
+       bank->chip.ngpio = bank->width;
 
        gpiochip_add(&bank->chip);
 
        for (j = bank->virtual_irq_start;
-                    j < bank->virtual_irq_start + bank_width; j++) {
+                    j < bank->virtual_irq_start + bank->width; j++) {
                irq_set_lockdep_class(j, &gpio_lock_class);
                irq_set_chip_data(j, bank);
                if (bank_is_mpuio(bank))
@@ -1602,7 +1601,7 @@ static int __devinit omap_gpio_probe(struct platform_device *pdev)
        bank->dev = &pdev->dev;
        bank->dbck_flag = pdata->dbck_flag;
        bank->stride = pdata->bank_stride;
-       bank_width = pdata->bank_width;
+       bank->width = pdata->bank_width;
 
        spin_lock_init(&bank->lock);