[PATCH 5/6] ARM: nmk: update GPIO chained IRQ handler to entry/exit functions

Linus Walleij linus.walleij at linaro.org
Mon Apr 18 14:26:02 EDT 2011

2011/4/12 Will Deacon <will.deacon at arm.com>:

> This patch updates the Nomadik gpio chained IRQ handler to use the
> chained IRQ enter/exit functions in order to function correctly on
> primary controllers with different methods of flow control.
> Cc: Rabin Vincent <rabin at rab.in>
> Signed-off-by: Will Deacon <will.deacon at arm.com>

Acked-by: Linus Walleij <linus.walleij at linaro.org>

BUT I want it to go through Grants gpio tree. The reason is that
I plan to consolidate this driver into drivers/gpio/* and I don't want
collisions in merges.

And wait:

> +#include <asm/mach/irq.h>

> -       if (host_chip->irq_mask_ack)
> -               host_chip->irq_mask_ack(&desc->irq_data);
> -       else {
> -               host_chip->irq_mask(&desc->irq_data);
> -               if (host_chip->irq_ack)
> -                       host_chip->irq_ack(&desc->irq_data);
> -       }
> +       chained_irq_enter(host_chip, desc);


> -       host_chip->irq_unmask(&desc->irq_data);
> +       chained_irq_exit(host_chip, desc);

This looks an awful lot like this from drivers/gpio/pl061.c:


It gets me wondering what the proper way is to do this.
Isn't the proper place for chained_irq_[enter|exit]* in
include/linux/irq.h rather? It does not really look machine
or arch specific, is it, really?

I.e. this:

#ifndef __ASSEMBLY__
 * Entry/exit functions for chained handlers where the primary IRQ chip
 * may implement either fasteoi or level-trigger flow control.
static inline void chained_irq_enter(struct irq_chip *chip,
                                     struct irq_desc *desc)
        /* FastEOI controllers require no action on entry. */
        if (chip->irq_eoi)

        if (chip->irq_mask_ack) {
        } else {
                if (chip->irq_ack)

static inline void chained_irq_exit(struct irq_chip *chip,
                                    struct irq_desc *desc)
        if (chip->irq_eoi)

Excuse me if this was discussed before...

Linus Walleij

More information about the linux-arm-kernel mailing list