s3c24xx: Fix level irqs on external interrupts.
authorLars-Peter Clausen <lars@metafoo.de>
Sat, 8 May 2010 12:32:09 +0000 (14:32 +0200)
committerMartin Jansa <Martin.Jansa@gmail.com>
Thu, 1 Sep 2011 09:49:33 +0000 (11:49 +0200)
commit56ee57980e903f6fbcac9fd35d0410b7bb363ae5
tree0fbac29ba2b48d15d0cc9e21e0d33ebedaf51433
parentf58597e68f36da7d4f4dec08520d3e12258addf2
s3c24xx: Fix level irqs on external interrupts.

Although the external interrupts support level and edge triggered irqs their
handler is currently always set to handle_edge_irq().
While being technically wrong for a level triggered irq to be handled by
handle_edge_irq() it will cause serious problems in combination with a oneshot
irq. handle_edge_irq() will unmask the irq immediately and as a result the irq
will be triggered again before the threaded irq handler had a chance to run and
clear the irq source.

Thus level triggered irqs should be handled by handle_level_irq.
According to the specs the irq controller will remember if an irq has been
triggered while it had been masked and will issue it when the irq gets
unmasked. Thus it is sufficient to use handle_level_irq() for edge triggered
irqs as well. Hence handle_level_irq() can always be used for external
interrupts.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
arch/arm/plat-s3c24xx/irq.c