]> Pileus Git - ~andy/linux/commit
irqchip: sirf: set IRQ_LEVEL status_flags
authorBarry Song <21cnbao@gmail.com>
Fri, 3 Jan 2014 03:34:46 +0000 (11:34 +0800)
committerOlof Johansson <olof@lixom.net>
Thu, 9 Jan 2014 06:02:14 +0000 (22:02 -0800)
commita87010ef3261912999302b3d82bd2e155898f816
tree9784c968e983c1ec3ed0b1e605d05ecfa5fe3f66
parentb93a35b170f9ad599e852855224b74982de2bde1
irqchip: sirf: set IRQ_LEVEL status_flags

SiRF internal interrupts are using level trigger. we need to tell the irq
core this information. otherwise, we might get some problems as below
1. disable_irq(n)
here irq core will mark the disabled flag but still keep the irq enabled
due to involved lazy-disable
2. doing someting after disable_irq(n)
in step 2, if one interrupt n comes, irq core will mark it as pending and
mask the HW interrupt really. we name the coming interrupt as "X".
3. enable_irq(n)
this will unmask the interrupt, so the level-trigger HW interrupt will come
again, irq_handler will enter as "E1". after that, irq core will also check
whether irq n is pending, if yes, and pending interrupt is not level-trigger,
irq core will execute the pending irq_handler.
so if we don't set the IRQ_LEVEL flag here, irq core will execute pending
X again as "E2", but actually the pending interrupt has been handled by "E1".
that makes a level-trigger HW interrupt is executed twice.

here we fix the issue to avoid redundant interrupt overload.

Signed-off-by: Barry Song <Baohua.Song@csr.com>
Signed-off-by: Huayi Li <Huayi.Li@csr.com>
Signed-off-by: Olof Johansson <olof@lixom.net>
drivers/irqchip/irq-sirfsoc.c