Tarun Kanti DebBarma <tarun.ka...@ti.com> writes:

> This local variable is just assigned zero and then OR'ed
> with isr. It does not appear to serve any purpose and so
> removing it.

Still missing the changelog changes I requested in v1.

Kevin

> Signed-off-by: Tarun Kanti DebBarma <tarun.ka...@ti.com>
> Reviewed-by: Santosh Shilimkar <santosh.shilim...@ti.com>
> Acked-by: Felipe Balbi <ba...@ti.com>
> ---
>  drivers/gpio/gpio-omap.c |    3 ---
>  1 files changed, 0 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpio/gpio-omap.c b/drivers/gpio/gpio-omap.c
> index 3765654..de5fe8f 100644
> --- a/drivers/gpio/gpio-omap.c
> +++ b/drivers/gpio/gpio-omap.c
> @@ -626,7 +626,6 @@ static void gpio_irq_handler(unsigned int irq, struct 
> irq_desc *desc)
>       u32 isr;
>       unsigned int gpio_irq, gpio_index;
>       struct gpio_bank *bank;
> -     u32 retrigger = 0;
>       int unmasked = 0;
>       struct irq_chip *chip = irq_desc_get_chip(desc);
>  
> @@ -663,8 +662,6 @@ static void gpio_irq_handler(unsigned int irq, struct 
> irq_desc *desc)
>                       chained_irq_exit(chip, desc);
>               }
>  
> -             isr |= retrigger;
> -             retrigger = 0;
>               if (!isr)
>                       break;
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to