diff options
author | Lennert Buytenhek <buytenh@wantstofly.org> | 2008-05-18 13:46:59 -0400 |
---|---|---|
committer | Lennert Buytenhek <buytenh@marvell.com> | 2008-06-22 16:44:43 -0400 |
commit | 000e99c3334cdddaec020d7324fe4ca2a683d3dd (patch) | |
tree | 9cbb07726ed3a2a0ae940b2881e938b6f07c528b /arch/arm/plat-orion/irq.c | |
parent | ee0dd840052e303a3eb171363be5e6364d200a51 (diff) |
[ARM] Orion: top-level IRQs are level-triggered
Make it clear that Orion top-level IRQs are level-triggered. This
means that we don't need an ->ack() handler, or at least, we don't
need the ->ack() handler (or the acking part of the ->mask_ack()
handler) to actually do anything.
Given that, we might as well point our ->mask_ack() handler at the
->mask() handler instead of providing a dummy ->ack() handler, since
providing a ->mask_ack() handler on level IRQ sources will prevent
->ack() from ever being called.
Signed-off-by: Lennert Buytenhek <buytenh@marvell.com>
Acked-by: Russell King <linux@arm.linux.org.uk>
Diffstat (limited to 'arch/arm/plat-orion/irq.c')
-rw-r--r-- | arch/arm/plat-orion/irq.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/arch/arm/plat-orion/irq.c b/arch/arm/plat-orion/irq.c index c5b669d234bc..fe66a1835169 100644 --- a/arch/arm/plat-orion/irq.c +++ b/arch/arm/plat-orion/irq.c | |||
@@ -36,8 +36,8 @@ static void orion_irq_unmask(u32 irq) | |||
36 | 36 | ||
37 | static struct irq_chip orion_irq_chip = { | 37 | static struct irq_chip orion_irq_chip = { |
38 | .name = "orion_irq", | 38 | .name = "orion_irq", |
39 | .ack = orion_irq_mask, | ||
40 | .mask = orion_irq_mask, | 39 | .mask = orion_irq_mask, |
40 | .mask_ack = orion_irq_mask, | ||
41 | .unmask = orion_irq_unmask, | 41 | .unmask = orion_irq_unmask, |
42 | }; | 42 | }; |
43 | 43 | ||
@@ -59,6 +59,7 @@ void __init orion_irq_init(unsigned int irq_start, void __iomem *maskaddr) | |||
59 | set_irq_chip(irq, &orion_irq_chip); | 59 | set_irq_chip(irq, &orion_irq_chip); |
60 | set_irq_chip_data(irq, maskaddr); | 60 | set_irq_chip_data(irq, maskaddr); |
61 | set_irq_handler(irq, handle_level_irq); | 61 | set_irq_handler(irq, handle_level_irq); |
62 | irq_desc[irq].status |= IRQ_LEVEL; | ||
62 | set_irq_flags(irq, IRQF_VALID); | 63 | set_irq_flags(irq, IRQF_VALID); |
63 | } | 64 | } |
64 | } | 65 | } |