intc/xilinx_intc: Dont lower IRQ when HIE cleared
authorPeter Crosthwaite <peter.crosthwaite@xilinx.com>
Tue, 11 Jun 2013 01:00:40 +0000 (11:00 +1000)
committerEdgar E. Iglesias <edgar.iglesias@gmail.com>
Tue, 18 Jun 2013 07:45:00 +0000 (09:45 +0200)
This is a little strange. It is lowering the parent IRQ pin on input
when HIE is cleared. There is no such behaviour in the real hardware.

ISR changes based on interrupt pin state are already guarded on HIE
being set. So we can just delete this if in its entirety.

Signed-off-by: Peter Crosthwaite <peter.crosthwaite@xilinx.com>
Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
hw/intc/xilinx_intc.c

index ddedfa3..297f537 100644 (file)
@@ -143,11 +143,6 @@ static void irq_handler(void *opaque, int irq, int level)
 {
     struct xlx_pic *p = opaque;
 
-    if (!(p->regs[R_MER] & 2)) {
-        qemu_irq_lower(p->parent_irq);
-        return;
-    }
-
     /* edge triggered interrupt */
     if (p->c_kind_of_intr & (1 << irq) && p->regs[R_MER] & 2) {
         p->regs[R_ISR] |= (level << irq);