Merge branch 'gpio/next' of git://git.secretlab.ca/git/linux-2.6
authorLinus Torvalds <torvalds@linux-foundation.org>
Fri, 22 Jul 2011 21:50:57 +0000 (14:50 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 22 Jul 2011 21:50:57 +0000 (14:50 -0700)
* 'gpio/next' of git://git.secretlab.ca/git/linux-2.6: (61 commits)
  gpio/mxc/mxs: fix build error introduced by the irq_gc_ack() renaming
  mcp23s08: add i2c support
  mcp23s08: isolate spi specific parts
  mcp23s08: get rid of setup/teardown callbacks
  gpio/tegra: dt: add binding for gpio polarity
  mcp23s08: remove unused work queue
  gpio/da9052: remove a redundant assignment for gpio->da9052
  gpio/mxc: add device tree probe support
  ARM: mxc: use ARCH_NR_GPIOS to define gpio number
  gpio/mxc: get rid of the uses of cpu_is_mx()
  gpio/mxc: add missing initialization of basic_mmio_gpio shadow variables
  gpio: Move mpc5200 gpio driver to drivers/gpio
  GPIO: DA9052 GPIO module v3
  gpio/tegra: Use engineering names in DT compatible property
  of/gpio: Add new method for getting gpios under different property names
  gpio/dt: Refine GPIO device tree binding
  gpio/ml-ioh: fix off-by-one for displaying variable i in dev_err
  gpio/pca953x: Deprecate meaningless device-tree bindings
  gpio/pca953x: Remove dynamic platform data pointer
  gpio/pca953x: Fix IRQ support.
  ...

1  2 
arch/arm/mach-ep93xx/core.c
arch/arm/mach-imx/mach-mx31_3ds.c
arch/arm/mach-omap1/gpio15xx.c
arch/arm/mach-omap1/gpio16xx.c
arch/arm/mach-omap1/gpio7xx.c
drivers/gpio/gpio-langwell.c
drivers/gpio/gpio-omap.c
drivers/gpio/gpio-tps65910.c
drivers/gpio/gpio-wm831x.c
drivers/leds/Kconfig

@@@ -174,14 -174,10 +174,10 @@@ struct sys_timer ep93xx_timer = 
  /*************************************************************************
   * EP93xx IRQ handling
   *************************************************************************/
- extern void ep93xx_gpio_init_irq(void);
  void __init ep93xx_init_irq(void)
  {
        vic_init(EP93XX_VIC1_BASE, 0, EP93XX_VIC1_VALID_IRQ_MASK, 0);
        vic_init(EP93XX_VIC2_BASE, 32, EP93XX_VIC2_VALID_IRQ_MASK, 0);
-       ep93xx_gpio_init_irq();
  }
  
  
@@@ -240,6 -236,24 +236,24 @@@ unsigned int ep93xx_chip_revision(void
        return v;
  }
  
+ /*************************************************************************
+  * EP93xx GPIO
+  *************************************************************************/
+ static struct resource ep93xx_gpio_resource[] = {
+       {
+               .start          = EP93XX_GPIO_PHYS_BASE,
+               .end            = EP93XX_GPIO_PHYS_BASE + 0xcc - 1,
+               .flags          = IORESOURCE_MEM,
+       },
+ };
+ static struct platform_device ep93xx_gpio_device = {
+       .name           = "gpio-ep93xx",
+       .id             = -1,
+       .num_resources  = ARRAY_SIZE(ep93xx_gpio_resource),
+       .resource       = ep93xx_gpio_resource,
+ };
  /*************************************************************************
   * EP93xx peripheral handling
   *************************************************************************/
@@@ -251,9 -265,9 +265,9 @@@ static void ep93xx_uart_set_mctrl(struc
        unsigned int mcr;
  
        mcr = 0;
 -      if (!(mctrl & TIOCM_RTS))
 +      if (mctrl & TIOCM_RTS)
                mcr |= 2;
 -      if (!(mctrl & TIOCM_DTR))
 +      if (mctrl & TIOCM_DTR)
                mcr |= 1;
  
        __raw_writel(mcr, base + EP93XX_UART_MCR_OFFSET);
@@@ -870,14 -884,13 +884,13 @@@ void __init ep93xx_register_ac97(void
        platform_device_register(&ep93xx_pcm_device);
  }
  
- extern void ep93xx_gpio_init(void);
  void __init ep93xx_init_devices(void)
  {
        /* Disallow access to MaverickCrunch initially */
        ep93xx_devcfg_clear_bits(EP93XX_SYSCON_DEVCFG_CPENA);
  
-       ep93xx_gpio_init();
+       /* Get the GPIO working early, other devices need it */
+       platform_device_register(&ep93xx_gpio_device);
  
        amba_device_register(&uart1_device, &iomem_resource);
        amba_device_register(&uart2_device, &iomem_resource);
@@@ -13,7 -13,6 +13,7 @@@
   */
  
  #include <linux/delay.h>
 +#include <linux/dma-mapping.h>
  #include <linux/types.h>
  #include <linux/init.h>
  #include <linux/clk.h>
@@@ -690,6 -689,8 +690,8 @@@ static void __init mx31_3ds_init(void
  {
        int ret;
  
+       imx31_soc_init();
        mxc_iomux_setup_multiple_pins(mx31_3ds_pins, ARRAY_SIZE(mx31_3ds_pins),
                                      "mx31_3ds");
  
@@@ -34,14 -34,25 +34,25 @@@ static struct __initdata resource omap1
        },
  };
  
+ static struct omap_gpio_reg_offs omap15xx_mpuio_regs = {
+       .revision       = USHRT_MAX,
+       .direction      = OMAP_MPUIO_IO_CNTL,
+       .datain         = OMAP_MPUIO_INPUT_LATCH,
+       .dataout        = OMAP_MPUIO_OUTPUT,
+       .irqstatus      = OMAP_MPUIO_GPIO_INT,
+       .irqenable      = OMAP_MPUIO_GPIO_MASKIT,
+       .irqenable_inv  = true,
+ };
  static struct __initdata omap_gpio_platform_data omap15xx_mpu_gpio_config = {
        .virtual_irq_start      = IH_MPUIO_BASE,
        .bank_type              = METHOD_MPUIO,
        .bank_width             = 16,
        .bank_stride            = 1,
+       .regs                   = &omap15xx_mpuio_regs,
  };
  
 -static struct __initdata platform_device omap15xx_mpu_gpio = {
 +static struct platform_device omap15xx_mpu_gpio = {
        .name           = "omap_gpio",
        .id             = 0,
        .dev            = {
@@@ -64,13 -75,24 +75,24 @@@ static struct __initdata resource omap1
        },
  };
  
+ static struct omap_gpio_reg_offs omap15xx_gpio_regs = {
+       .revision       = USHRT_MAX,
+       .direction      = OMAP1510_GPIO_DIR_CONTROL,
+       .datain         = OMAP1510_GPIO_DATA_INPUT,
+       .dataout        = OMAP1510_GPIO_DATA_OUTPUT,
+       .irqstatus      = OMAP1510_GPIO_INT_STATUS,
+       .irqenable      = OMAP1510_GPIO_INT_MASK,
+       .irqenable_inv  = true,
+ };
  static struct __initdata omap_gpio_platform_data omap15xx_gpio_config = {
        .virtual_irq_start      = IH_GPIO_BASE,
        .bank_type              = METHOD_GPIO_1510,
        .bank_width             = 16,
+       .regs                   = &omap15xx_gpio_regs,
  };
  
 -static struct __initdata platform_device omap15xx_gpio = {
 +static struct platform_device omap15xx_gpio = {
        .name           = "omap_gpio",
        .id             = 1,
        .dev            = {
@@@ -37,14 -37,25 +37,25 @@@ static struct __initdata resource omap1
        },
  };
  
+ static struct omap_gpio_reg_offs omap16xx_mpuio_regs = {
+       .revision       = USHRT_MAX,
+       .direction      = OMAP_MPUIO_IO_CNTL,
+       .datain         = OMAP_MPUIO_INPUT_LATCH,
+       .dataout        = OMAP_MPUIO_OUTPUT,
+       .irqstatus      = OMAP_MPUIO_GPIO_INT,
+       .irqenable      = OMAP_MPUIO_GPIO_MASKIT,
+       .irqenable_inv  = true,
+ };
  static struct __initdata omap_gpio_platform_data omap16xx_mpu_gpio_config = {
        .virtual_irq_start      = IH_MPUIO_BASE,
        .bank_type              = METHOD_MPUIO,
        .bank_width             = 16,
        .bank_stride            = 1,
+       .regs                   = &omap16xx_mpuio_regs,
  };
  
 -static struct __initdata platform_device omap16xx_mpu_gpio = {
 +static struct platform_device omap16xx_mpu_gpio = {
        .name           = "omap_gpio",
        .id             = 0,
        .dev            = {
@@@ -67,13 -78,27 +78,27 @@@ static struct __initdata resource omap1
        },
  };
  
+ static struct omap_gpio_reg_offs omap16xx_gpio_regs = {
+       .revision       = OMAP1610_GPIO_REVISION,
+       .direction      = OMAP1610_GPIO_DIRECTION,
+       .set_dataout    = OMAP1610_GPIO_SET_DATAOUT,
+       .clr_dataout    = OMAP1610_GPIO_CLEAR_DATAOUT,
+       .datain         = OMAP1610_GPIO_DATAIN,
+       .dataout        = OMAP1610_GPIO_DATAOUT,
+       .irqstatus      = OMAP1610_GPIO_IRQSTATUS1,
+       .irqenable      = OMAP1610_GPIO_IRQENABLE1,
+       .set_irqenable  = OMAP1610_GPIO_SET_IRQENABLE1,
+       .clr_irqenable  = OMAP1610_GPIO_CLEAR_IRQENABLE1,
+ };
  static struct __initdata omap_gpio_platform_data omap16xx_gpio1_config = {
        .virtual_irq_start      = IH_GPIO_BASE,
        .bank_type              = METHOD_GPIO_1610,
        .bank_width             = 16,
+       .regs                   = &omap16xx_gpio_regs,
  };
  
 -static struct __initdata platform_device omap16xx_gpio1 = {
 +static struct platform_device omap16xx_gpio1 = {
        .name           = "omap_gpio",
        .id             = 1,
        .dev            = {
@@@ -100,9 -125,10 +125,10 @@@ static struct __initdata omap_gpio_plat
        .virtual_irq_start      = IH_GPIO_BASE + 16,
        .bank_type              = METHOD_GPIO_1610,
        .bank_width             = 16,
+       .regs                   = &omap16xx_gpio_regs,
  };
  
 -static struct __initdata platform_device omap16xx_gpio2 = {
 +static struct platform_device omap16xx_gpio2 = {
        .name           = "omap_gpio",
        .id             = 2,
        .dev            = {
@@@ -129,9 -155,10 +155,10 @@@ static struct __initdata omap_gpio_plat
        .virtual_irq_start      = IH_GPIO_BASE + 32,
        .bank_type              = METHOD_GPIO_1610,
        .bank_width             = 16,
+       .regs                   = &omap16xx_gpio_regs,
  };
  
 -static struct __initdata platform_device omap16xx_gpio3 = {
 +static struct platform_device omap16xx_gpio3 = {
        .name           = "omap_gpio",
        .id             = 3,
        .dev            = {
@@@ -158,9 -185,10 +185,10 @@@ static struct __initdata omap_gpio_plat
        .virtual_irq_start      = IH_GPIO_BASE + 48,
        .bank_type              = METHOD_GPIO_1610,
        .bank_width             = 16,
+       .regs                   = &omap16xx_gpio_regs,
  };
  
 -static struct __initdata platform_device omap16xx_gpio4 = {
 +static struct platform_device omap16xx_gpio4 = {
        .name           = "omap_gpio",
        .id             = 4,
        .dev            = {
@@@ -39,14 -39,25 +39,25 @@@ static struct __initdata resource omap7
        },
  };
  
+ static struct omap_gpio_reg_offs omap7xx_mpuio_regs = {
+       .revision       = USHRT_MAX,
+       .direction      = OMAP_MPUIO_IO_CNTL / 2,
+       .datain         = OMAP_MPUIO_INPUT_LATCH / 2,
+       .dataout        = OMAP_MPUIO_OUTPUT / 2,
+       .irqstatus      = OMAP_MPUIO_GPIO_INT / 2,
+       .irqenable      = OMAP_MPUIO_GPIO_MASKIT / 2,
+       .irqenable_inv  = true,
+ };
  static struct __initdata omap_gpio_platform_data omap7xx_mpu_gpio_config = {
        .virtual_irq_start      = IH_MPUIO_BASE,
        .bank_type              = METHOD_MPUIO,
        .bank_width             = 32,
        .bank_stride            = 2,
+       .regs                   = &omap7xx_mpuio_regs,
  };
  
 -static struct __initdata platform_device omap7xx_mpu_gpio = {
 +static struct platform_device omap7xx_mpu_gpio = {
        .name           = "omap_gpio",
        .id             = 0,
        .dev            = {
@@@ -69,13 -80,24 +80,24 @@@ static struct __initdata resource omap7
        },
  };
  
+ static struct omap_gpio_reg_offs omap7xx_gpio_regs = {
+       .revision       = USHRT_MAX,
+       .direction      = OMAP7XX_GPIO_DIR_CONTROL,
+       .datain         = OMAP7XX_GPIO_DATA_INPUT,
+       .dataout        = OMAP7XX_GPIO_DATA_OUTPUT,
+       .irqstatus      = OMAP7XX_GPIO_INT_STATUS,
+       .irqenable      = OMAP7XX_GPIO_INT_MASK,
+       .irqenable_inv  = true,
+ };
  static struct __initdata omap_gpio_platform_data omap7xx_gpio1_config = {
        .virtual_irq_start      = IH_GPIO_BASE,
        .bank_type              = METHOD_GPIO_7XX,
        .bank_width             = 32,
+       .regs                   = &omap7xx_gpio_regs,
  };
  
 -static struct __initdata platform_device omap7xx_gpio1 = {
 +static struct platform_device omap7xx_gpio1 = {
        .name           = "omap_gpio",
        .id             = 1,
        .dev            = {
@@@ -102,9 -124,10 +124,10 @@@ static struct __initdata omap_gpio_plat
        .virtual_irq_start      = IH_GPIO_BASE + 32,
        .bank_type              = METHOD_GPIO_7XX,
        .bank_width             = 32,
+       .regs                   = &omap7xx_gpio_regs,
  };
  
 -static struct __initdata platform_device omap7xx_gpio2 = {
 +static struct platform_device omap7xx_gpio2 = {
        .name           = "omap_gpio",
        .id             = 2,
        .dev            = {
@@@ -131,9 -154,10 +154,10 @@@ static struct __initdata omap_gpio_plat
        .virtual_irq_start      = IH_GPIO_BASE + 64,
        .bank_type              = METHOD_GPIO_7XX,
        .bank_width             = 32,
+       .regs                   = &omap7xx_gpio_regs,
  };
  
 -static struct __initdata platform_device omap7xx_gpio3 = {
 +static struct platform_device omap7xx_gpio3 = {
        .name           = "omap_gpio",
        .id             = 3,
        .dev            = {
@@@ -160,9 -184,10 +184,10 @@@ static struct __initdata omap_gpio_plat
        .virtual_irq_start      = IH_GPIO_BASE + 96,
        .bank_type              = METHOD_GPIO_7XX,
        .bank_width             = 32,
+       .regs                   = &omap7xx_gpio_regs,
  };
  
 -static struct __initdata platform_device omap7xx_gpio4 = {
 +static struct platform_device omap7xx_gpio4 = {
        .name           = "omap_gpio",
        .id             = 4,
        .dev            = {
@@@ -189,9 -214,10 +214,10 @@@ static struct __initdata omap_gpio_plat
        .virtual_irq_start      = IH_GPIO_BASE + 128,
        .bank_type              = METHOD_GPIO_7XX,
        .bank_width             = 32,
+       .regs                   = &omap7xx_gpio_regs,
  };
  
 -static struct __initdata platform_device omap7xx_gpio5 = {
 +static struct platform_device omap7xx_gpio5 = {
        .name           = "omap_gpio",
        .id             = 5,
        .dev            = {
@@@ -218,9 -244,10 +244,10 @@@ static struct __initdata omap_gpio_plat
        .virtual_irq_start      = IH_GPIO_BASE + 160,
        .bank_type              = METHOD_GPIO_7XX,
        .bank_width             = 32,
+       .regs                   = &omap7xx_gpio_regs,
  };
  
 -static struct __initdata platform_device omap7xx_gpio6 = {
 +static struct platform_device omap7xx_gpio6 = {
        .name           = "omap_gpio",
        .id             = 6,
        .dev            = {
@@@ -1,4 -1,6 +1,6 @@@
- /* langwell_gpio.c Moorestown platform Langwell chip GPIO driver
+ /*
+  * Moorestown platform Langwell chip GPIO driver
+  *
   * Copyright (c) 2008 - 2009,  Intel Corporation.
   *
   * This program is free software; you can redistribute it and/or modify
@@@ -223,7 -225,7 +225,7 @@@ static void lnw_irq_handler(unsigned ir
                gedr = gpio_reg(&lnw->chip, base, GEDR);
                pending = readl(gedr);
                while (pending) {
 -                      gpio = __ffs(pending) - 1;
 +                      gpio = __ffs(pending);
                        mask = BIT(gpio);
                        pending &= ~mask;
                        /* Clear before handling so we can't lose an edge */
diff --combined drivers/gpio/gpio-omap.c
@@@ -54,6 -54,11 +54,11 @@@ struct gpio_bank 
        struct device *dev;
        bool dbck_flag;
        int stride;
+       u32 width;
+       void (*set_dataout)(struct gpio_bank *bank, int gpio, int enable);
+       struct omap_gpio_reg_offs *regs;
  };
  
  #ifdef CONFIG_ARCH_OMAP3
@@@ -79,121 -84,18 +84,18 @@@ static struct omap3_gpio_regs gpio_cont
   */
  static struct gpio_bank *gpio_bank;
  
- static int bank_width;
  /* TODO: Analyze removing gpio_bank_count usage from driver code */
  int gpio_bank_count;
  
- static inline struct gpio_bank *get_gpio_bank(int gpio)
- {
-       if (cpu_is_omap15xx()) {
-               if (OMAP_GPIO_IS_MPUIO(gpio))
-                       return &gpio_bank[0];
-               return &gpio_bank[1];
-       }
-       if (cpu_is_omap16xx()) {
-               if (OMAP_GPIO_IS_MPUIO(gpio))
-                       return &gpio_bank[0];
-               return &gpio_bank[1 + (gpio >> 4)];
-       }
-       if (cpu_is_omap7xx()) {
-               if (OMAP_GPIO_IS_MPUIO(gpio))
-                       return &gpio_bank[0];
-               return &gpio_bank[1 + (gpio >> 5)];
-       }
-       if (cpu_is_omap24xx())
-               return &gpio_bank[gpio >> 5];
-       if (cpu_is_omap34xx() || cpu_is_omap44xx())
-               return &gpio_bank[gpio >> 5];
-       BUG();
-       return NULL;
- }
- static inline int get_gpio_index(int gpio)
- {
-       if (cpu_is_omap7xx())
-               return gpio & 0x1f;
-       if (cpu_is_omap24xx())
-               return gpio & 0x1f;
-       if (cpu_is_omap34xx() || cpu_is_omap44xx())
-               return gpio & 0x1f;
-       return gpio & 0x0f;
- }
- static inline int gpio_valid(int gpio)
- {
-       if (gpio < 0)
-               return -1;
-       if (cpu_class_is_omap1() && OMAP_GPIO_IS_MPUIO(gpio)) {
-               if (gpio >= OMAP_MAX_GPIO_LINES + 16)
-                       return -1;
-               return 0;
-       }
-       if (cpu_is_omap15xx() && gpio < 16)
-               return 0;
-       if ((cpu_is_omap16xx()) && gpio < 64)
-               return 0;
-       if (cpu_is_omap7xx() && gpio < 192)
-               return 0;
-       if (cpu_is_omap2420() && gpio < 128)
-               return 0;
-       if (cpu_is_omap2430() && gpio < 160)
-               return 0;
-       if ((cpu_is_omap34xx() || cpu_is_omap44xx()) && gpio < 192)
-               return 0;
-       return -1;
- }
- static int check_gpio(int gpio)
- {
-       if (unlikely(gpio_valid(gpio) < 0)) {
-               printk(KERN_ERR "omap-gpio: invalid GPIO %d\n", gpio);
-               dump_stack();
-               return -1;
-       }
-       return 0;
- }
+ #define GPIO_INDEX(bank, gpio) (gpio % bank->width)
+ #define GPIO_BIT(bank, gpio) (1 << GPIO_INDEX(bank, gpio))
  
  static void _set_gpio_direction(struct gpio_bank *bank, int gpio, int is_input)
  {
        void __iomem *reg = bank->base;
        u32 l;
  
-       switch (bank->method) {
- #ifdef CONFIG_ARCH_OMAP1
-       case METHOD_MPUIO:
-               reg += OMAP_MPUIO_IO_CNTL / bank->stride;
-               break;
- #endif
- #ifdef CONFIG_ARCH_OMAP15XX
-       case METHOD_GPIO_1510:
-               reg += OMAP1510_GPIO_DIR_CONTROL;
-               break;
- #endif
- #ifdef CONFIG_ARCH_OMAP16XX
-       case METHOD_GPIO_1610:
-               reg += OMAP1610_GPIO_DIRECTION;
-               break;
- #endif
- #if defined(CONFIG_ARCH_OMAP730) || defined(CONFIG_ARCH_OMAP850)
-       case METHOD_GPIO_7XX:
-               reg += OMAP7XX_GPIO_DIR_CONTROL;
-               break;
- #endif
- #if defined(CONFIG_ARCH_OMAP2) || defined(CONFIG_ARCH_OMAP3)
-       case METHOD_GPIO_24XX:
-               reg += OMAP24XX_GPIO_OE;
-               break;
- #endif
- #if defined(CONFIG_ARCH_OMAP4)
-       case METHOD_GPIO_44XX:
-               reg += OMAP4_GPIO_OE;
-               break;
- #endif
-       default:
-               WARN_ON(1);
-               return;
-       }
+       reg += bank->regs->direction;
        l = __raw_readl(reg);
        if (is_input)
                l |= 1 << gpio;
        __raw_writel(l, reg);
  }
  
- static void _set_gpio_dataout(struct gpio_bank *bank, int gpio, int enable)
+ /* set data out value using dedicate set/clear register */
+ static void _set_gpio_dataout_reg(struct gpio_bank *bank, int gpio, int enable)
  {
        void __iomem *reg = bank->base;
-       u32 l = 0;
+       u32 l = GPIO_BIT(bank, gpio);
+       if (enable)
+               reg += bank->regs->set_dataout;
+       else
+               reg += bank->regs->clr_dataout;
  
-       switch (bank->method) {
- #ifdef CONFIG_ARCH_OMAP1
-       case METHOD_MPUIO:
-               reg += OMAP_MPUIO_OUTPUT / bank->stride;
-               l = __raw_readl(reg);
-               if (enable)
-                       l |= 1 << gpio;
-               else
-                       l &= ~(1 << gpio);
-               break;
- #endif
- #ifdef CONFIG_ARCH_OMAP15XX
-       case METHOD_GPIO_1510:
-               reg += OMAP1510_GPIO_DATA_OUTPUT;
-               l = __raw_readl(reg);
-               if (enable)
-                       l |= 1 << gpio;
-               else
-                       l &= ~(1 << gpio);
-               break;
- #endif
- #ifdef CONFIG_ARCH_OMAP16XX
-       case METHOD_GPIO_1610:
-               if (enable)
-                       reg += OMAP1610_GPIO_SET_DATAOUT;
-               else
-                       reg += OMAP1610_GPIO_CLEAR_DATAOUT;
-               l = 1 << gpio;
-               break;
- #endif
- #if defined(CONFIG_ARCH_OMAP730) || defined(CONFIG_ARCH_OMAP850)
-       case METHOD_GPIO_7XX:
-               reg += OMAP7XX_GPIO_DATA_OUTPUT;
-               l = __raw_readl(reg);
-               if (enable)
-                       l |= 1 << gpio;
-               else
-                       l &= ~(1 << gpio);
-               break;
- #endif
- #if defined(CONFIG_ARCH_OMAP2) || defined(CONFIG_ARCH_OMAP3)
-       case METHOD_GPIO_24XX:
-               if (enable)
-                       reg += OMAP24XX_GPIO_SETDATAOUT;
-               else
-                       reg += OMAP24XX_GPIO_CLEARDATAOUT;
-               l = 1 << gpio;
-               break;
- #endif
- #ifdef CONFIG_ARCH_OMAP4
-       case METHOD_GPIO_44XX:
-               if (enable)
-                       reg += OMAP4_GPIO_SETDATAOUT;
-               else
-                       reg += OMAP4_GPIO_CLEARDATAOUT;
-               l = 1 << gpio;
-               break;
- #endif
-       default:
-               WARN_ON(1);
-               return;
-       }
        __raw_writel(l, reg);
  }
  
- static int _get_gpio_datain(struct gpio_bank *bank, int gpio)
+ /* set data out value using mask register */
+ static void _set_gpio_dataout_mask(struct gpio_bank *bank, int gpio, int enable)
  {
-       void __iomem *reg;
+       void __iomem *reg = bank->base + bank->regs->dataout;
+       u32 gpio_bit = GPIO_BIT(bank, gpio);
+       u32 l;
  
-       if (check_gpio(gpio) < 0)
-               return -EINVAL;
-       reg = bank->base;
-       switch (bank->method) {
- #ifdef CONFIG_ARCH_OMAP1
-       case METHOD_MPUIO:
-               reg += OMAP_MPUIO_INPUT_LATCH / bank->stride;
-               break;
- #endif
- #ifdef CONFIG_ARCH_OMAP15XX
-       case METHOD_GPIO_1510:
-               reg += OMAP1510_GPIO_DATA_INPUT;
-               break;
- #endif
- #ifdef CONFIG_ARCH_OMAP16XX
-       case METHOD_GPIO_1610:
-               reg += OMAP1610_GPIO_DATAIN;
-               break;
- #endif
- #if defined(CONFIG_ARCH_OMAP730) || defined(CONFIG_ARCH_OMAP850)
-       case METHOD_GPIO_7XX:
-               reg += OMAP7XX_GPIO_DATA_INPUT;
-               break;
- #endif
- #if defined(CONFIG_ARCH_OMAP2) || defined(CONFIG_ARCH_OMAP3)
-       case METHOD_GPIO_24XX:
-               reg += OMAP24XX_GPIO_DATAIN;
-               break;
- #endif
- #ifdef CONFIG_ARCH_OMAP4
-       case METHOD_GPIO_44XX:
-               reg += OMAP4_GPIO_DATAIN;
-               break;
- #endif
-       default:
-               return -EINVAL;
-       }
-       return (__raw_readl(reg)
-                       & (1 << get_gpio_index(gpio))) != 0;
+       l = __raw_readl(reg);
+       if (enable)
+               l |= gpio_bit;
+       else
+               l &= ~gpio_bit;
+       __raw_writel(l, reg);
  }
  
- static int _get_gpio_dataout(struct gpio_bank *bank, int gpio)
+ static int _get_gpio_datain(struct gpio_bank *bank, int gpio)
  {
-       void __iomem *reg;
+       void __iomem *reg = bank->base + bank->regs->datain;
  
-       if (check_gpio(gpio) < 0)
-               return -EINVAL;
-       reg = bank->base;
+       return (__raw_readl(reg) & GPIO_BIT(bank, gpio)) != 0;
+ }
  
-       switch (bank->method) {
- #ifdef CONFIG_ARCH_OMAP1
-       case METHOD_MPUIO:
-               reg += OMAP_MPUIO_OUTPUT / bank->stride;
-               break;
- #endif
- #ifdef CONFIG_ARCH_OMAP15XX
-       case METHOD_GPIO_1510:
-               reg += OMAP1510_GPIO_DATA_OUTPUT;
-               break;
- #endif
- #ifdef CONFIG_ARCH_OMAP16XX
-       case METHOD_GPIO_1610:
-               reg += OMAP1610_GPIO_DATAOUT;
-               break;
- #endif
- #if defined(CONFIG_ARCH_OMAP730) || defined(CONFIG_ARCH_OMAP850)
-       case METHOD_GPIO_7XX:
-               reg += OMAP7XX_GPIO_DATA_OUTPUT;
-               break;
- #endif
- #if defined(CONFIG_ARCH_OMAP2) || defined(CONFIG_ARCH_OMAP3)
-       case METHOD_GPIO_24XX:
-               reg += OMAP24XX_GPIO_DATAOUT;
-               break;
- #endif
- #ifdef CONFIG_ARCH_OMAP4
-       case METHOD_GPIO_44XX:
-               reg += OMAP4_GPIO_DATAOUT;
-               break;
- #endif
-       default:
-               return -EINVAL;
-       }
+ static int _get_gpio_dataout(struct gpio_bank *bank, int gpio)
+ {
+       void __iomem *reg = bank->base + bank->regs->dataout;
  
-       return (__raw_readl(reg) & (1 << get_gpio_index(gpio))) != 0;
+       return (__raw_readl(reg) & GPIO_BIT(bank, gpio)) != 0;
  }
  
  #define MOD_REG_BIT(reg, bit_mask, set)       \
@@@ -383,7 -168,7 +168,7 @@@ do {       
  static void _set_gpio_debounce(struct gpio_bank *bank, unsigned gpio,
                unsigned debounce)
  {
-       void __iomem            *reg = bank->base;
+       void __iomem            *reg;
        u32                     val;
        u32                     l;
  
        else
                debounce = (debounce / 0x1f) - 1;
  
-       l = 1 << get_gpio_index(gpio);
-       if (bank->method == METHOD_GPIO_44XX)
-               reg += OMAP4_GPIO_DEBOUNCINGTIME;
-       else
-               reg += OMAP24XX_GPIO_DEBOUNCE_VAL;
+       l = GPIO_BIT(bank, gpio);
  
+       reg = bank->base + bank->regs->debounce;
        __raw_writel(debounce, reg);
  
-       reg = bank->base;
-       if (bank->method == METHOD_GPIO_44XX)
-               reg += OMAP4_GPIO_DEBOUNCENABLE;
-       else
-               reg += OMAP24XX_GPIO_DEBOUNCE_EN;
+       reg = bank->base + bank->regs->debounce_en;
        val = __raw_readl(reg);
  
        if (debounce) {
@@@ -469,9 -245,8 +245,9 @@@ static inline void set_24xx_gpio_trigge
                                        + OMAP24XX_GPIO_CLEARWKUENA);
                }
        }
 -      /* This part needs to be executed always for OMAP34xx */
 -      if (cpu_is_omap34xx() || (bank->non_wakeup_gpios & gpio_bit)) {
 +      /* This part needs to be executed always for OMAP{34xx, 44xx} */
 +      if (cpu_is_omap34xx() || cpu_is_omap44xx() ||
 +                      (bank->non_wakeup_gpios & gpio_bit)) {
                /*
                 * Log the edge gpio and manually trigger the IRQ
                 * after resume if the input level changes
@@@ -629,9 -404,6 +405,6 @@@ static int gpio_irq_type(struct irq_dat
        else
                gpio = d->irq - IH_GPIO_BASE;
  
-       if (check_gpio(gpio) < 0)
-               return -EINVAL;
        if (type & ~IRQ_TYPE_SENSE_MASK)
                return -EINVAL;
  
  
        bank = irq_data_get_irq_chip_data(d);
        spin_lock_irqsave(&bank->lock, flags);
-       retval = _set_gpio_triggering(bank, get_gpio_index(gpio), type);
+       retval = _set_gpio_triggering(bank, GPIO_INDEX(bank, gpio), type);
        spin_unlock_irqrestore(&bank->lock, flags);
  
        if (type & (IRQ_TYPE_LEVEL_LOW | IRQ_TYPE_LEVEL_HIGH))
@@@ -657,195 -429,81 +430,81 @@@ static void _clear_gpio_irqbank(struct 
  {
        void __iomem *reg = bank->base;
  
-       switch (bank->method) {
- #ifdef CONFIG_ARCH_OMAP1
-       case METHOD_MPUIO:
-               /* MPUIO irqstatus is reset by reading the status register,
-                * so do nothing here */
-               return;
- #endif
- #ifdef CONFIG_ARCH_OMAP15XX
-       case METHOD_GPIO_1510:
-               reg += OMAP1510_GPIO_INT_STATUS;
-               break;
- #endif
- #ifdef CONFIG_ARCH_OMAP16XX
-       case METHOD_GPIO_1610:
-               reg += OMAP1610_GPIO_IRQSTATUS1;
-               break;
- #endif
- #if defined(CONFIG_ARCH_OMAP730) || defined(CONFIG_ARCH_OMAP850)
-       case METHOD_GPIO_7XX:
-               reg += OMAP7XX_GPIO_INT_STATUS;
-               break;
- #endif
- #if defined(CONFIG_ARCH_OMAP2) || defined(CONFIG_ARCH_OMAP3)
-       case METHOD_GPIO_24XX:
-               reg += OMAP24XX_GPIO_IRQSTATUS1;
-               break;
- #endif
- #if defined(CONFIG_ARCH_OMAP4)
-       case METHOD_GPIO_44XX:
-               reg += OMAP4_GPIO_IRQSTATUS0;
-               break;
- #endif
-       default:
-               WARN_ON(1);
-               return;
-       }
+       reg += bank->regs->irqstatus;
        __raw_writel(gpio_mask, reg);
  
        /* Workaround for clearing DSP GPIO interrupts to allow retention */
-       if (cpu_is_omap24xx() || cpu_is_omap34xx())
-               reg = bank->base + OMAP24XX_GPIO_IRQSTATUS2;
-       else if (cpu_is_omap44xx())
-               reg = bank->base + OMAP4_GPIO_IRQSTATUS1;
-       if (cpu_is_omap24xx() || cpu_is_omap34xx() || cpu_is_omap44xx()) {
+       if (bank->regs->irqstatus2) {
+               reg = bank->base + bank->regs->irqstatus2;
                __raw_writel(gpio_mask, reg);
+       }
  
        /* Flush posted write for the irq status to avoid spurious interrupts */
        __raw_readl(reg);
-       }
  }
  
  static inline void _clear_gpio_irqstatus(struct gpio_bank *bank, int gpio)
  {
-       _clear_gpio_irqbank(bank, 1 << get_gpio_index(gpio));
+       _clear_gpio_irqbank(bank, GPIO_BIT(bank, gpio));
  }
  
  static u32 _get_gpio_irqbank_mask(struct gpio_bank *bank)
  {
        void __iomem *reg = bank->base;
-       int inv = 0;
        u32 l;
-       u32 mask;
-       switch (bank->method) {
- #ifdef CONFIG_ARCH_OMAP1
-       case METHOD_MPUIO:
-               reg += OMAP_MPUIO_GPIO_MASKIT / bank->stride;
-               mask = 0xffff;
-               inv = 1;
-               break;
- #endif
- #ifdef CONFIG_ARCH_OMAP15XX
-       case METHOD_GPIO_1510:
-               reg += OMAP1510_GPIO_INT_MASK;
-               mask = 0xffff;
-               inv = 1;
-               break;
- #endif
- #ifdef CONFIG_ARCH_OMAP16XX
-       case METHOD_GPIO_1610:
-               reg += OMAP1610_GPIO_IRQENABLE1;
-               mask = 0xffff;
-               break;
- #endif
- #if defined(CONFIG_ARCH_OMAP730) || defined(CONFIG_ARCH_OMAP850)
-       case METHOD_GPIO_7XX:
-               reg += OMAP7XX_GPIO_INT_MASK;
-               mask = 0xffffffff;
-               inv = 1;
-               break;
- #endif
- #if defined(CONFIG_ARCH_OMAP2) || defined(CONFIG_ARCH_OMAP3)
-       case METHOD_GPIO_24XX:
-               reg += OMAP24XX_GPIO_IRQENABLE1;
-               mask = 0xffffffff;
-               break;
- #endif
- #if defined(CONFIG_ARCH_OMAP4)
-       case METHOD_GPIO_44XX:
-               reg += OMAP4_GPIO_IRQSTATUSSET0;
-               mask = 0xffffffff;
-               break;
- #endif
-       default:
-               WARN_ON(1);
-               return 0;
-       }
+       u32 mask = (1 << bank->width) - 1;
  
+       reg += bank->regs->irqenable;
        l = __raw_readl(reg);
-       if (inv)
+       if (bank->regs->irqenable_inv)
                l = ~l;
        l &= mask;
        return l;
  }
  
- static void _enable_gpio_irqbank(struct gpio_bank *bank, int gpio_mask, int enable)
+ static void _enable_gpio_irqbank(struct gpio_bank *bank, int gpio_mask)
  {
        void __iomem *reg = bank->base;
        u32 l;
  
-       switch (bank->method) {
- #ifdef CONFIG_ARCH_OMAP1
-       case METHOD_MPUIO:
-               reg += OMAP_MPUIO_GPIO_MASKIT / bank->stride;
-               l = __raw_readl(reg);
-               if (enable)
-                       l &= ~(gpio_mask);
-               else
-                       l |= gpio_mask;
-               break;
- #endif
- #ifdef CONFIG_ARCH_OMAP15XX
-       case METHOD_GPIO_1510:
-               reg += OMAP1510_GPIO_INT_MASK;
+       if (bank->regs->set_irqenable) {
+               reg += bank->regs->set_irqenable;
+               l = gpio_mask;
+       } else {
+               reg += bank->regs->irqenable;
                l = __raw_readl(reg);
-               if (enable)
-                       l &= ~(gpio_mask);
+               if (bank->regs->irqenable_inv)
+                       l &= ~gpio_mask;
                else
                        l |= gpio_mask;
-               break;
- #endif
- #ifdef CONFIG_ARCH_OMAP16XX
-       case METHOD_GPIO_1610:
-               if (enable)
-                       reg += OMAP1610_GPIO_SET_IRQENABLE1;
-               else
-                       reg += OMAP1610_GPIO_CLEAR_IRQENABLE1;
+       }
+       __raw_writel(l, reg);
+ }
+ static void _disable_gpio_irqbank(struct gpio_bank *bank, int gpio_mask)
+ {
+       void __iomem *reg = bank->base;
+       u32 l;
+       if (bank->regs->clr_irqenable) {
+               reg += bank->regs->clr_irqenable;
                l = gpio_mask;
-               break;
- #endif
- #if defined(CONFIG_ARCH_OMAP730) || defined(CONFIG_ARCH_OMAP850)
-       case METHOD_GPIO_7XX:
-               reg += OMAP7XX_GPIO_INT_MASK;
+       } else {
+               reg += bank->regs->irqenable;
                l = __raw_readl(reg);
-               if (enable)
-                       l &= ~(gpio_mask);
-               else
+               if (bank->regs->irqenable_inv)
                        l |= gpio_mask;
-               break;
- #endif
- #if defined(CONFIG_ARCH_OMAP2) || defined(CONFIG_ARCH_OMAP3)
-       case METHOD_GPIO_24XX:
-               if (enable)
-                       reg += OMAP24XX_GPIO_SETIRQENABLE1;
                else
-                       reg += OMAP24XX_GPIO_CLEARIRQENABLE1;
-               l = gpio_mask;
-               break;
- #endif
- #ifdef CONFIG_ARCH_OMAP4
-       case METHOD_GPIO_44XX:
-               if (enable)
-                       reg += OMAP4_GPIO_IRQSTATUSSET0;
-               else
-                       reg += OMAP4_GPIO_IRQSTATUSCLR0;
-               l = gpio_mask;
-               break;
- #endif
-       default:
-               WARN_ON(1);
-               return;
+                       l &= ~gpio_mask;
        }
        __raw_writel(l, reg);
  }
  
  static inline void _set_gpio_irqenable(struct gpio_bank *bank, int gpio, int enable)
  {
-       _enable_gpio_irqbank(bank, 1 << get_gpio_index(gpio), enable);
+       _enable_gpio_irqbank(bank, GPIO_BIT(bank, gpio));
  }
  
  /*
   */
  static int _set_gpio_wakeup(struct gpio_bank *bank, int gpio, int enable)
  {
-       unsigned long uninitialized_var(flags);
+       u32 gpio_bit = GPIO_BIT(bank, gpio);
+       unsigned long flags;
  
-       switch (bank->method) {
- #ifdef CONFIG_ARCH_OMAP16XX
-       case METHOD_MPUIO:
-       case METHOD_GPIO_1610:
-               spin_lock_irqsave(&bank->lock, flags);
-               if (enable)
-                       bank->suspend_wakeup |= (1 << gpio);
-               else
-                       bank->suspend_wakeup &= ~(1 << gpio);
-               spin_unlock_irqrestore(&bank->lock, flags);
-               return 0;
- #endif
- #ifdef CONFIG_ARCH_OMAP2PLUS
-       case METHOD_GPIO_24XX:
-       case METHOD_GPIO_44XX:
-               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);
-                       return -EINVAL;
-               }
-               spin_lock_irqsave(&bank->lock, flags);
-               if (enable)
-                       bank->suspend_wakeup |= (1 << gpio);
-               else
-                       bank->suspend_wakeup &= ~(1 << gpio);
-               spin_unlock_irqrestore(&bank->lock, flags);
-               return 0;
- #endif
-       default:
-               printk(KERN_ERR "Can't enable GPIO wakeup for method %i\n",
-                      bank->method);
+       if (bank->non_wakeup_gpios & gpio_bit) {
+               dev_err(bank->dev, 
+                       "Unable to modify wakeup on non-wakeup GPIO%d\n", gpio);
                return -EINVAL;
        }
+       spin_lock_irqsave(&bank->lock, flags);
+       if (enable)
+               bank->suspend_wakeup |= gpio_bit;
+       else
+               bank->suspend_wakeup &= ~gpio_bit;
+       spin_unlock_irqrestore(&bank->lock, flags);
+       return 0;
  }
  
  static void _reset_gpio(struct gpio_bank *bank, int gpio)
  {
-       _set_gpio_direction(bank, get_gpio_index(gpio), 1);
+       _set_gpio_direction(bank, GPIO_INDEX(bank, gpio), 1);
        _set_gpio_irqenable(bank, gpio, 0);
        _clear_gpio_irqstatus(bank, gpio);
-       _set_gpio_triggering(bank, get_gpio_index(gpio), IRQ_TYPE_NONE);
+       _set_gpio_triggering(bank, GPIO_INDEX(bank, gpio), IRQ_TYPE_NONE);
  }
  
  /* Use disable_irq_wake() and enable_irq_wake() functions from drivers */
@@@ -911,10 -551,8 +552,8 @@@ static int gpio_wake_enable(struct irq_
        struct gpio_bank *bank;
        int retval;
  
-       if (check_gpio(gpio) < 0)
-               return -ENODEV;
        bank = irq_data_get_irq_chip_data(d);
-       retval = _set_gpio_wakeup(bank, get_gpio_index(gpio), enable);
+       retval = _set_gpio_wakeup(bank, gpio, enable);
  
        return retval;
  }
@@@ -1030,31 -668,7 +669,7 @@@ static void gpio_irq_handler(unsigned i
        chained_irq_enter(chip, desc);
  
        bank = irq_get_handler_data(irq);
- #ifdef CONFIG_ARCH_OMAP1
-       if (bank->method == METHOD_MPUIO)
-               isr_reg = bank->base +
-                               OMAP_MPUIO_GPIO_INT / bank->stride;
- #endif
- #ifdef CONFIG_ARCH_OMAP15XX
-       if (bank->method == METHOD_GPIO_1510)
-               isr_reg = bank->base + OMAP1510_GPIO_INT_STATUS;
- #endif
- #if defined(CONFIG_ARCH_OMAP16XX)
-       if (bank->method == METHOD_GPIO_1610)
-               isr_reg = bank->base + OMAP1610_GPIO_IRQSTATUS1;
- #endif
- #if defined(CONFIG_ARCH_OMAP730) || defined(CONFIG_ARCH_OMAP850)
-       if (bank->method == METHOD_GPIO_7XX)
-               isr_reg = bank->base + OMAP7XX_GPIO_INT_STATUS;
- #endif
- #if defined(CONFIG_ARCH_OMAP2) || defined(CONFIG_ARCH_OMAP3)
-       if (bank->method == METHOD_GPIO_24XX)
-               isr_reg = bank->base + OMAP24XX_GPIO_IRQSTATUS1;
- #endif
- #if defined(CONFIG_ARCH_OMAP4)
-       if (bank->method == METHOD_GPIO_44XX)
-               isr_reg = bank->base + OMAP4_GPIO_IRQSTATUS0;
- #endif
+       isr_reg = bank->base + bank->regs->irqstatus;
  
        if (WARN_ON(!isr_reg))
                goto exit;
                /* clear edge sensitive interrupts before handler(s) are
                called so that we don't miss any interrupt occurred while
                executing them */
-               _enable_gpio_irqbank(bank, isr_saved & ~level_mask, 0);
+               _disable_gpio_irqbank(bank, isr_saved & ~level_mask);
                _clear_gpio_irqbank(bank, isr_saved & ~level_mask);
-               _enable_gpio_irqbank(bank, isr_saved & ~level_mask, 1);
+               _enable_gpio_irqbank(bank, isr_saved & ~level_mask);
  
                /* if there is only edge sensitive GPIO pin interrupts
                configured, we could unmask GPIO bank interrupt immediately */
  
                gpio_irq = bank->virtual_irq_start;
                for (; isr != 0; isr >>= 1, gpio_irq++) {
-                       gpio_index = get_gpio_index(irq_to_gpio(gpio_irq));
+                       gpio_index = GPIO_INDEX(bank, irq_to_gpio(gpio_irq));
  
                        if (!(isr & 1))
                                continue;
@@@ -1150,7 -764,7 +765,7 @@@ static void gpio_mask_irq(struct irq_da
  
        spin_lock_irqsave(&bank->lock, flags);
        _set_gpio_irqenable(bank, gpio, 0);
-       _set_gpio_triggering(bank, get_gpio_index(gpio), IRQ_TYPE_NONE);
+       _set_gpio_triggering(bank, GPIO_INDEX(bank, gpio), IRQ_TYPE_NONE);
        spin_unlock_irqrestore(&bank->lock, flags);
  }
  
@@@ -1158,13 -772,13 +773,13 @@@ static void gpio_unmask_irq(struct irq_
  {
        unsigned int gpio = d->irq - IH_GPIO_BASE;
        struct gpio_bank *bank = irq_data_get_irq_chip_data(d);
-       unsigned int irq_mask = 1 << get_gpio_index(gpio);
+       unsigned int irq_mask = GPIO_BIT(bank, gpio);
        u32 trigger = irqd_get_trigger_type(d);
        unsigned long flags;
  
        spin_lock_irqsave(&bank->lock, flags);
        if (trigger)
-               _set_gpio_triggering(bank, get_gpio_index(gpio), trigger);
+               _set_gpio_triggering(bank, GPIO_INDEX(bank, gpio), trigger);
  
        /* For level-triggered GPIOs, the clearing must be done after
         * the HW source is cleared, thus after the handler has run */
@@@ -1191,45 -805,8 +806,8 @@@ static struct irq_chip gpio_irq_chip = 
  
  #ifdef CONFIG_ARCH_OMAP1
  
- /* MPUIO uses the always-on 32k clock */
- static void mpuio_ack_irq(struct irq_data *d)
- {
-       /* The ISR is reset automatically, so do nothing here. */
- }
- static void mpuio_mask_irq(struct irq_data *d)
- {
-       unsigned int gpio = OMAP_MPUIO(d->irq - IH_MPUIO_BASE);
-       struct gpio_bank *bank = irq_data_get_irq_chip_data(d);
-       _set_gpio_irqenable(bank, gpio, 0);
- }
- static void mpuio_unmask_irq(struct irq_data *d)
- {
-       unsigned int gpio = OMAP_MPUIO(d->irq - IH_MPUIO_BASE);
-       struct gpio_bank *bank = irq_data_get_irq_chip_data(d);
-       _set_gpio_irqenable(bank, gpio, 1);
- }
- static struct irq_chip mpuio_irq_chip = {
-       .name           = "MPUIO",
-       .irq_ack        = mpuio_ack_irq,
-       .irq_mask       = mpuio_mask_irq,
-       .irq_unmask     = mpuio_unmask_irq,
-       .irq_set_type   = gpio_irq_type,
- #ifdef CONFIG_ARCH_OMAP16XX
-       /* REVISIT: assuming only 16xx supports MPUIO wake events */
-       .irq_set_wake   = gpio_wake_enable,
- #endif
- };
  #define bank_is_mpuio(bank)   ((bank)->method == METHOD_MPUIO)
  
  #ifdef CONFIG_ARCH_OMAP16XX
  
  #include <linux/platform_device.h>
@@@ -1289,7 -866,7 +867,7 @@@ static struct platform_device omap_mpui
  
  static inline void mpuio_init(void)
  {
-       struct gpio_bank *bank = get_gpio_bank(OMAP_MPUIO(0));
+       struct gpio_bank *bank = &gpio_bank[0];
        platform_set_drvdata(&omap_mpuio_device, bank);
  
        if (platform_driver_register(&omap_mpuio_driver) == 0)
@@@ -1302,8 -879,6 +880,6 @@@ static inline void mpuio_init(void) {
  
  #else
  
- extern struct irq_chip mpuio_irq_chip;
  #define bank_is_mpuio(bank)   0
  static inline void mpuio_init(void) {}
  
@@@ -1329,31 -904,8 +905,8 @@@ static int gpio_input(struct gpio_chip 
  
  static int gpio_is_input(struct gpio_bank *bank, int mask)
  {
-       void __iomem *reg = bank->base;
+       void __iomem *reg = bank->base + bank->regs->direction;
  
-       switch (bank->method) {
-       case METHOD_MPUIO:
-               reg += OMAP_MPUIO_IO_CNTL / bank->stride;
-               break;
-       case METHOD_GPIO_1510:
-               reg += OMAP1510_GPIO_DIR_CONTROL;
-               break;
-       case METHOD_GPIO_1610:
-               reg += OMAP1610_GPIO_DIRECTION;
-               break;
-       case METHOD_GPIO_7XX:
-               reg += OMAP7XX_GPIO_DIR_CONTROL;
-               break;
-       case METHOD_GPIO_24XX:
-               reg += OMAP24XX_GPIO_OE;
-               break;
-       case METHOD_GPIO_44XX:
-               reg += OMAP4_GPIO_OE;
-               break;
-       default:
-               WARN_ONCE(1, "gpio_is_input: incorrect OMAP GPIO method");
-               return -EINVAL;
-       }
        return __raw_readl(reg) & mask;
  }
  
@@@ -1365,9 -917,9 +918,9 @@@ static int gpio_get(struct gpio_chip *c
        u32 mask;
  
        gpio = chip->base + offset;
-       bank = get_gpio_bank(gpio);
+       bank = container_of(chip, struct gpio_bank, chip);
        reg = bank->base;
-       mask = 1 << get_gpio_index(gpio);
+       mask = GPIO_BIT(bank, gpio);
  
        if (gpio_is_input(bank, mask))
                return _get_gpio_datain(bank, gpio);
@@@ -1382,7 -934,7 +935,7 @@@ static int gpio_output(struct gpio_chi
  
        bank = container_of(chip, struct gpio_bank, chip);
        spin_lock_irqsave(&bank->lock, flags);
-       _set_gpio_dataout(bank, offset, value);
+       bank->set_dataout(bank, offset, value);
        _set_gpio_direction(bank, offset, 0);
        spin_unlock_irqrestore(&bank->lock, flags);
        return 0;
@@@ -1416,7 -968,7 +969,7 @@@ static void gpio_set(struct gpio_chip *
  
        bank = container_of(chip, struct gpio_bank, chip);
        spin_lock_irqsave(&bank->lock, flags);
-       _set_gpio_dataout(bank, offset, value);
+       bank->set_dataout(bank, offset, value);
        spin_unlock_irqrestore(&bank->lock, flags);
  }
  
@@@ -1432,19 -984,17 +985,17 @@@ static int gpio_2irq(struct gpio_chip *
  
  static void __init omap_gpio_show_rev(struct gpio_bank *bank)
  {
+       static bool called;
        u32 rev;
  
-       if (cpu_is_omap16xx() && !(bank->method != METHOD_MPUIO))
-               rev = __raw_readw(bank->base + OMAP1610_GPIO_REVISION);
-       else if (cpu_is_omap24xx() || cpu_is_omap34xx())
-               rev = __raw_readl(bank->base + OMAP24XX_GPIO_REVISION);
-       else if (cpu_is_omap44xx())
-               rev = __raw_readl(bank->base + OMAP4_GPIO_REVISION);
-       else
+       if (called || bank->regs->revision == USHRT_MAX)
                return;
  
-       printk(KERN_INFO "OMAP GPIO hardware version %d.%d\n",
+       rev = __raw_readw(bank->base + bank->regs->revision);
+       pr_info("OMAP GPIO hardware version %d.%d\n",
                (rev >> 4) & 0x0f, rev & 0x0f);
+       called = true;
  }
  
  /* This lock class tells lockdep that GPIO irqs are in a different
@@@ -1526,6 -1076,30 +1077,30 @@@ static void omap_gpio_mod_init(struct g
        }
  }
  
+ static __init void
+ omap_mpuio_alloc_gc(struct gpio_bank *bank, unsigned int irq_start,
+                   unsigned int num)
+ {
+       struct irq_chip_generic *gc;
+       struct irq_chip_type *ct;
+       gc = irq_alloc_generic_chip("MPUIO", 1, irq_start, bank->base,
+                                   handle_simple_irq);
+       ct = gc->chip_types;
+       /* NOTE: No ack required, reading IRQ status clears it. */
+       ct->chip.irq_mask = irq_gc_mask_set_bit;
+       ct->chip.irq_unmask = irq_gc_mask_clr_bit;
+       ct->chip.irq_set_type = gpio_irq_type;
+       /* REVISIT: assuming only 16xx supports MPUIO wake events */
+       if (cpu_is_omap16xx())
+               ct->chip.irq_set_wake = gpio_wake_enable,
+       ct->regs.mask = OMAP_MPUIO_GPIO_INT / bank->stride;
+       irq_setup_generic_chip(gc, IRQ_MSK(num), IRQ_GC_INIT_MASK_CACHE,
+                              IRQ_NOREQUEST | IRQ_NOPROBE, 0);
+ }
  static void __devinit omap_gpio_chip_init(struct gpio_bank *bank)
  {
        int j;
        } 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))
-                       irq_set_chip(j, &mpuio_irq_chip);
-               else
+               if (bank_is_mpuio(bank)) {
+                       omap_mpuio_alloc_gc(bank, j, bank->width);
+               } else {
                        irq_set_chip(j, &gpio_irq_chip);
-               irq_set_handler(j, handle_simple_irq);
-               set_irq_flags(j, IRQF_VALID);
+                       irq_set_handler(j, handle_simple_irq);
+                       set_irq_flags(j, IRQF_VALID);
+               }
        }
        irq_set_chained_handler(bank->irq, gpio_irq_handler);
        irq_set_handler_data(bank->irq, bank);
@@@ -1610,7 -1185,14 +1186,14 @@@ static int __devinit omap_gpio_probe(st
        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;
+       bank->regs = pdata->regs;
+       if (bank->regs->set_dataout && bank->regs->clr_dataout)
+               bank->set_dataout = _set_gpio_dataout_reg;
+       else
+               bank->set_dataout = _set_gpio_dataout_mask;
  
        spin_lock_init(&bank->lock);
  
@@@ -1,5 -1,5 +1,5 @@@
  /*
-  * tps65910-gpio.c  --  TI TPS6591x
+  * TI TPS6591x GPIO driver
   *
   * Copyright 2010 Texas Instruments Inc.
   *
@@@ -81,10 -81,8 +81,10 @@@ void tps65910_gpio_init(struct tps6591
        switch(tps65910_chip_id(tps65910)) {
        case TPS65910:
                tps65910->gpio.ngpio    = 6;
 +              break;
        case TPS65911:
                tps65910->gpio.ngpio    = 9;
 +              break;
        default:
                return;
        }
@@@ -1,5 -1,5 +1,5 @@@
  /*
-  * wm831x-gpio.c  --  gpiolib support for Wolfson WM831x PMICs
+  * gpiolib support for Wolfson WM831x PMICs
   *
   * Copyright 2009 Wolfson Microelectronics PLC.
   *
@@@ -180,7 -180,6 +180,7 @@@ static void wm831x_gpio_dbg_show(struc
                        break;
                case WM831X_GPIO_PULL_UP:
                        pull = "pullup";
 +                      break;
                default:
                        pull = "INVALID PULL";
                        break;
diff --combined drivers/leds/Kconfig
@@@ -1,10 -1,3 +1,10 @@@
 +config LEDS_GPIO_REGISTER
 +      bool
 +      help
 +        This option provides the function gpio_led_register_device.
 +        As this function is used by arch code it must not be compiled as a
 +        module.
 +
  menuconfig NEW_LEDS
        bool "LED Support"
        help
          This is not related to standard keyboard LEDs which are controlled
          via the input system.
  
 +if NEW_LEDS
 +
  config LEDS_CLASS
        bool "LED Class Support"
 -      depends on NEW_LEDS
        help
          This option enables the led sysfs class in /sys/class/leds.  You'll
          need this to do anything useful with LEDs.  If unsure, say N.
  
 -config LEDS_GPIO_REGISTER
 -      bool
 -      help
 -        This option provides the function gpio_led_register_device.
 -        As this function is used by arch code it must not be compiled as a
 -        module.
 -
 -if NEW_LEDS
 -
  comment "LED drivers"
  
  config LEDS_88PM860X
@@@ -92,7 -93,7 +92,7 @@@ config LEDS_NET48X
  config LEDS_NET5501
        tristate "LED Support for Soekris net5501 series Error LED"
        depends on LEDS_TRIGGERS
-       depends on X86 && LEDS_GPIO_PLATFORM && GPIO_CS5535
+       depends on X86 && GPIO_CS5535
        select LEDS_TRIGGER_DEFAULT_ON
        default n
        help
@@@ -182,23 -183,6 +182,6 @@@ config LEDS_GPI
          defined as platform devices and/or OpenFirmware platform devices.
          The code to use these bindings can be selected below.
  
- config LEDS_GPIO_PLATFORM
-       bool "Platform device bindings for GPIO LEDs"
-       depends on LEDS_GPIO
-       default y
-       help
-         Let the leds-gpio driver drive LEDs which have been defined as
-         platform devices.  If you don't know what this means, say yes.
- config LEDS_GPIO_OF
-       bool "OpenFirmware platform device bindings for GPIO LEDs"
-       depends on LEDS_GPIO && OF_DEVICE
-       default y
-       help
-         Let the leds-gpio driver drive LEDs which have been defined as
-         of_platform devices.  For instance, LEDs which are listed in a "dts"
-         file.
  config LEDS_LP3944
        tristate "LED Support for N.S. LP3944 (Fun Light) I2C chip"
        depends on LEDS_CLASS
@@@ -390,7 -374,6 +373,7 @@@ config LEDS_NETXBI
  
  config LEDS_ASIC3
        bool "LED support for the HTC ASIC3"
 +      depends on LEDS_CLASS
        depends on MFD_ASIC3
        default y
        help