ARC: [SMP] IPI ACK interface doesn't need "self" cpu-id
authorVineet Gupta <vgupta@synopsys.com>
Mon, 25 Nov 2013 09:07:14 +0000 (14:37 +0530)
committerVineet Gupta <vgupta@synopsys.com>
Mon, 23 Dec 2013 06:35:04 +0000 (12:05 +0530)
The interface is confusing, it feels like we are getting "sender" info,
whereas it is the "receiver", which can very well be retrived by
smp_processor_id(), if need be.

Signed-off-by: Vineet Gupta <vgupta@synopsys.com>
arch/arc/include/asm/smp.h
arch/arc/kernel/smp.c
arch/arc/plat-arcfpga/smp.c

index 9b40e3b..5d06eee 100644 (file)
@@ -47,13 +47,13 @@ extern int smp_ipi_irq_setup(int cpu, int irq);
  * @info:              SoC SMP specific info for /proc/cpuinfo etc
  * @cpu_kick:          For Master to kickstart a cpu (optionally at a PC)
  * @ipi_send:          To send IPI to a @cpu
- * @ips_clear:         To clear IPI received by @cpu at @irq
+ * @ips_clear:         To clear IPI received at @irq
  */
 struct plat_smp_ops {
        const char      *info;
        void            (*cpu_kick)(int cpu, unsigned long pc);
        void            (*ipi_send)(int cpu);
-       void            (*ipi_clear)(int cpu, int irq);
+       void            (*ipi_clear)(int irq);
 };
 
 /* TBD: stop exporting it for direct population by platform */
index f2fdef9..9f771b5 100644 (file)
@@ -310,7 +310,7 @@ irqreturn_t do_IPI(int irq, void *dev_id)
        unsigned long ops;
 
        if (plat_smp_ops.ipi_clear)
-               plat_smp_ops.ipi_clear(cpu, irq);
+               plat_smp_ops.ipi_clear(irq);
 
        /*
         * XXX: is this loop really needed
index 5e4fe95..8a12741 100644 (file)
@@ -93,9 +93,9 @@ static void iss_model_ipi_send(int cpu)
        idu_irq_assert(cpu);
 }
 
-static void iss_model_ipi_clear(int cpu, int irq)
+static void iss_model_ipi_clear(int irq)
 {
-       idu_irq_clear(IDU_INTERRUPT_0 + cpu);
+       idu_irq_clear(IDU_INTERRUPT_0 + smp_processor_id());
 }
 
 void iss_model_init_early_smp(void)