[PATCH 02/15] OMAP: GPIO: remove MPUIO handling from _clear_gpio_irqbank()

[charu@ti.com (Varadarajan, Charulatha)]

Kevin,

On Sat, Apr 23, 2011 at 04:31, Kevin Hilman <khilman@ti.com> wrote:
> Remove the OMAP1 #ifdef and MPUIO special case for _clear_gpio_irqbank()
>
> The MPUIOs do not need a register access to ack/clear the IRQ status,
> since reading the IRQ status clears it. ?In addition, the MPUIO
> irq_chip has an empty ack method, so _clear_gpio_irqbank() is never
> used for MPUIOs.
>
> Signed-off-by: Kevin Hilman <khilman@ti.com>
> ---
> ?arch/arm/plat-omap/gpio.c | ? ?6 ------
> ?1 files changed, 0 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm/plat-omap/gpio.c b/arch/arm/plat-omap/gpio.c
> index fe6971a..8b5ca6e 100644
> --- a/arch/arm/plat-omap/gpio.c
> +++ b/arch/arm/plat-omap/gpio.c
> @@ -770,12 +770,6 @@ static void _clear_gpio_irqbank(struct gpio_bank *bank, int gpio_mask)
> ? ? ? ?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

The default case has a "WARN_ON(1)". I guess WARN_ON()
is not required for METHOD_MPUIO.

-V Charulatha

> ?#ifdef CONFIG_ARCH_OMAP15XX
> ? ? ? ?case METHOD_GPIO_1510:
> ? ? ? ? ? ? ? ?reg += OMAP1510_GPIO_INT_STATUS;
> --
> 1.7.4
>
>