powerpc/irq: Move get_irq() comment into header
authorMichael Ellerman <michael@ellerman.id.au>
Wed, 22 Apr 2009 15:31:38 +0000 (15:31 +0000)
committerBenjamin Herrenschmidt <benh@kernel.crashing.org>
Thu, 21 May 2009 05:43:59 +0000 (15:43 +1000)
The guts of do_IRQ() isn't really the right place to be documenting
the ppc_md.get_irq() interface. So move the comment into machdep.h

Signed-off-by: Michael Ellerman <michael@ellerman.id.au>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
arch/powerpc/include/asm/machdep.h
arch/powerpc/kernel/irq.c

index 0efdb1dfdc5f577ea336d4a96fd8dd246d703514..11d1fc3a89629ae39d685e5d450a2f29160d3c56 100644 (file)
@@ -110,6 +110,10 @@ struct machdep_calls {
        void            (*show_percpuinfo)(struct seq_file *m, int i);
 
        void            (*init_IRQ)(void);
+
+       /* Return an irq, or NO_IRQ to indicate there are none pending.
+        * If for some reason there is no irq, but the interrupt
+        * shouldn't be counted as spurious, return NO_IRQ_IGNORE. */
        unsigned int    (*get_irq)(void);
 #ifdef CONFIG_KEXEC
        void            (*kexec_cpu_down)(int crash_shutdown, int secondary);
index 463b6ab9d6b9f06eab3f591d26a8c3f15cddca8c..a5fe12aa937dc142acac695f10c68774006ef838 100644 (file)
@@ -326,13 +326,6 @@ void do_IRQ(struct pt_regs *regs)
 
        check_stack_overflow();
 
-       /*
-        * Every platform is required to implement ppc_md.get_irq.
-        * This function will either return an irq number or NO_IRQ to
-        * indicate there are no more pending.
-        * The value NO_IRQ_IGNORE is for buggy hardware and means that this
-        * IRQ has already been handled. -- Tom
-        */
        irq = ppc_md.get_irq();
 
        if (irq != NO_IRQ && irq != NO_IRQ_IGNORE)