MIPS: Simplify and correct interrupt handling for MSP4200

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Linux Kernel Mailing List
Date: Tuesday, August 4, 2009 - 5:02 pm

Gitweb:     http://git.kernel.org/linus/0ca71737fee65521ede964afbd2d5484976ed0ed
Commit:     0ca71737fee65521ede964afbd2d5484976ed0ed
Parent:     6577890fd68c2671850214663dd9ae97feacbc47
Author:     Shane McDonald <mcdonald.shane@gmail.com>
AuthorDate: Sun Jul 12 10:42:06 2009 -0600
Committer:  Ralf Baechle <ralf@linux-mips.org>
CommitDate: Mon Aug 3 17:52:43 2009 +0100

    MIPS: Simplify and correct interrupt handling for MSP4200
    
    The current interrupt handling code for the MSP4200 always masks an
    interrupt before acknowledging it.  This is not required, as that will be
    handled by the level interrupt handler.  This change simplifies the MSP4200
    code to remove the masking in the ack routine, and makes sure that the
    minimum required operation is performed for masking and acking, rather
    than always both masking and acking the interrupt.
    
    Signed-off-by: Shane McDonald <mcdonald.shane@gmail.com>
    Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
---
 arch/mips/pmc-sierra/msp71xx/msp_irq_slp.c |   10 +---------
 1 files changed, 1 insertions(+), 9 deletions(-)

diff --git a/arch/mips/pmc-sierra/msp71xx/msp_irq_slp.c b/arch/mips/pmc-sierra/msp71xx/msp_irq_slp.c
index 66f6f85..61f3902 100644
--- a/arch/mips/pmc-sierra/msp71xx/msp_irq_slp.c
+++ b/arch/mips/pmc-sierra/msp71xx/msp_irq_slp.c
@@ -45,13 +45,6 @@ static inline void mask_msp_slp_irq(unsigned int irq)
  */
 static inline void ack_msp_slp_irq(unsigned int irq)
 {
-	mask_msp_slp_irq(irq);
-
-	/*
-	 * only really necessary for 18, 16-14 and sometimes 3:0 (since
-	 * these can be edge sensitive) but it doesn't hurt  for the others.
-	 */
-
 	/* check for PER interrupt range */
 	if (irq < MSP_PER_INTBASE)
 		*SLP_INT_STS_REG = (1 << (irq - MSP_SLP_INTBASE));
@@ -62,8 +55,7 @@ static inline void ack_msp_slp_irq(unsigned int irq)
 static struct irq_chip msp_slp_irq_controller = {
 	.name = "MSP_SLP",
 	.ack = ack_msp_slp_irq,
-	.mask = ack_msp_slp_irq,
-	.mask_ack = ack_msp_slp_irq,
+	.mask = mask_msp_slp_irq,
 	.unmask = unmask_msp_slp_irq,
 };
 
--
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
MIPS: Simplify and correct interrupt handling for MSP4200, Linux Kernel Mailing ..., (Tue Aug 4, 5:02 pm)