1 files changed, 11 insertions, 28 deletions

diff --git a/arch/sh/boards/superh/microdev/irq.c b/arch/sh/boards/superh/microdev/irq.c
index 236398fbc08..8c64baa3036 100644
--- a/arch/sh/boards/superh/microdev/irq.c
+++ b/arch/sh/boards/superh/microdev/irq.c
@@ -11,14 +11,12 @@
 
 #include <linux/init.h>
 #include <linux/irq.h>
-
 #include <asm/system.h>
 #include <asm/io.h>
 #include <asm/microdev.h>
 
 #define NUM_EXTERNAL_IRQS 16    /* IRL0 .. IRL15 */
 
-
 static const struct {
         unsigned char fpgaIrq;
         unsigned char mapped;
@@ -93,53 +91,42 @@ static struct hw_interrupt_type microdev_irq_type = {
 
 static void disable_microdev_irq(unsigned int irq)
 {
-        unsigned int flags;
         unsigned int fpgaIrq;
 
-        if (irq >= NUM_EXTERNAL_IRQS) return;
-        if (!fpgaIrqTable[irq].mapped) return;
+        if (irq >= NUM_EXTERNAL_IRQS)
+                return;
+        if (!fpgaIrqTable[irq].mapped)
+                return;
 
         fpgaIrq = fpgaIrqTable[irq].fpgaIrq;
 
-                /* disable interrupts */
-        local_irq_save(flags);
-
-                /* disable interupts on the FPGA INTC register */
+        /* disable interupts on the FPGA INTC register */
         ctrl_outl(MICRODEV_FPGA_INTC_MASK(fpgaIrq), MICRODEV_FPGA_INTDSB_REG);
-
-                /* restore interrupts */
-        local_irq_restore(flags);
 }
 
 static void enable_microdev_irq(unsigned int irq)
 {
         unsigned long priorityReg, priorities, pri;
-        unsigned int flags;
         unsigned int fpgaIrq;
 
-
-        if (irq >= NUM_EXTERNAL_IRQS) return;
-        if (!fpgaIrqTable[irq].mapped) return;
+        if (unlikely(irq >= NUM_EXTERNAL_IRQS))
+                return;
+        if (unlikely(!fpgaIrqTable[irq].mapped))
+                return;
 
         pri = 15 - irq;
 
         fpgaIrq = fpgaIrqTable[irq].fpgaIrq;
         priorityReg = MICRODEV_FPGA_INTPRI_REG(fpgaIrq);
 
-                /* disable interrupts */
-        local_irq_save(flags);
-
-                /* set priority for the interrupt */
+        /* set priority for the interrupt */
         priorities = ctrl_inl(priorityReg);
         priorities &= ~MICRODEV_FPGA_INTPRI_MASK(fpgaIrq);
         priorities |= MICRODEV_FPGA_INTPRI_LEVEL(fpgaIrq, pri);
         ctrl_outl(priorities, priorityReg);
 
-                /* enable interupts on the FPGA INTC register */
+        /* enable interupts on the FPGA INTC register */
         ctrl_outl(MICRODEV_FPGA_INTC_MASK(fpgaIrq), MICRODEV_FPGA_INTENB_REG);
-
-                /* restore interrupts */
-        local_irq_restore(flags);
 }
 
         /* This functions sets the desired irq handler to be a MicroDev type */
@@ -158,9 +145,7 @@ static void mask_and_ack_microdev(unsigned int irq)
 static void end_microdev_irq(unsigned int irq)
 {
         if (!(irq_desc[irq].status & (IRQ_DISABLED|IRQ_INPROGRESS)))
-        {
                 enable_microdev_irq(irq);
-        }
 }
 
 extern void __init init_microdev_irq(void)
@@ -171,9 +156,7 @@ extern void __init init_microdev_irq(void)
         ctrl_outl(~0ul, MICRODEV_FPGA_INTDSB_REG);
 
         for (i = 0; i < NUM_EXTERNAL_IRQS; i++)
-        {
                 make_microdev_irq(i);
-        }
 }
 
 extern void microdev_print_fpga_intc_status(void)