* [PATCH] ARC: [plat-eznps] remove IPI clear from SMP operations
@ 2016-11-07 7:34 ` Noam Camus
0 siblings, 0 replies; 4+ messages in thread
From: Noam Camus @ 2016-11-07 7:34 UTC (permalink / raw)
To: vgupta; +Cc: linux-snps-arc, linux-kernel, Noam Camus
From: Noam Camus <noamca@mellanox.com>
Generic IRQ mechanism is already acknowledge the IPI IRQ.
Doing this once more time in IPI handler is not needed.
Signed-off-by: Noam Camus <noamca@mellanox.com>
---
arch/arc/plat-eznps/smp.c | 6 ------
1 files changed, 0 insertions(+), 6 deletions(-)
diff --git a/arch/arc/plat-eznps/smp.c b/arch/arc/plat-eznps/smp.c
index 5e901f8..56a4c85 100644
--- a/arch/arc/plat-eznps/smp.c
+++ b/arch/arc/plat-eznps/smp.c
@@ -140,16 +140,10 @@ static void eznps_init_per_cpu(int cpu)
mtm_enable_core(cpu);
}
-static void eznps_ipi_clear(int irq)
-{
- write_aux_reg(CTOP_AUX_IACK, 1 << irq);
-}
-
struct plat_smp_ops plat_smp_ops = {
.info = smp_cpuinfo_buf,
.init_early_smp = eznps_init_cpumasks,
.cpu_kick = eznps_smp_wakeup_cpu,
.ipi_send = eznps_ipi_send,
.init_per_cpu = eznps_init_per_cpu,
- .ipi_clear = eznps_ipi_clear,
};
--
1.7.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH] ARC: [plat-eznps] remove IPI clear from SMP operations
@ 2016-11-07 7:34 ` Noam Camus
0 siblings, 0 replies; 4+ messages in thread
From: Noam Camus @ 2016-11-07 7:34 UTC (permalink / raw)
To: linux-snps-arc
From: Noam Camus <noamca@mellanox.com>
Generic IRQ mechanism is already acknowledge the IPI IRQ.
Doing this once more time in IPI handler is not needed.
Signed-off-by: Noam Camus <noamca at mellanox.com>
---
arch/arc/plat-eznps/smp.c | 6 ------
1 files changed, 0 insertions(+), 6 deletions(-)
diff --git a/arch/arc/plat-eznps/smp.c b/arch/arc/plat-eznps/smp.c
index 5e901f8..56a4c85 100644
--- a/arch/arc/plat-eznps/smp.c
+++ b/arch/arc/plat-eznps/smp.c
@@ -140,16 +140,10 @@ static void eznps_init_per_cpu(int cpu)
mtm_enable_core(cpu);
}
-static void eznps_ipi_clear(int irq)
-{
- write_aux_reg(CTOP_AUX_IACK, 1 << irq);
-}
-
struct plat_smp_ops plat_smp_ops = {
.info = smp_cpuinfo_buf,
.init_early_smp = eznps_init_cpumasks,
.cpu_kick = eznps_smp_wakeup_cpu,
.ipi_send = eznps_ipi_send,
.init_per_cpu = eznps_init_per_cpu,
- .ipi_clear = eznps_ipi_clear,
};
--
1.7.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] ARC: [plat-eznps] remove IPI clear from SMP operations
2016-11-07 7:34 ` Noam Camus
@ 2016-11-07 18:54 ` Vineet Gupta
-1 siblings, 0 replies; 4+ messages in thread
From: Vineet Gupta @ 2016-11-07 18:54 UTC (permalink / raw)
To: Noam Camus; +Cc: linux-snps-arc, linux-kernel
On 11/06/2016 11:34 PM, Noam Camus wrote:
> From: Noam Camus <noamca@mellanox.com>
>
> Generic IRQ mechanism is already acknowledge the IPI IRQ.
> Doing this once more time in IPI handler is not needed.
Changelog can be improved !
Generic IRQ mechanism can't ack the irq. The NPQ irqchip driver does this for all
irqs already - so no special handling is needed for IPI in NPS case.
-Vineet
>
> Signed-off-by: Noam Camus <noamca@mellanox.com>
> ---
> arch/arc/plat-eznps/smp.c | 6 ------
> 1 files changed, 0 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arc/plat-eznps/smp.c b/arch/arc/plat-eznps/smp.c
> index 5e901f8..56a4c85 100644
> --- a/arch/arc/plat-eznps/smp.c
> +++ b/arch/arc/plat-eznps/smp.c
> @@ -140,16 +140,10 @@ static void eznps_init_per_cpu(int cpu)
> mtm_enable_core(cpu);
> }
>
> -static void eznps_ipi_clear(int irq)
> -{
> - write_aux_reg(CTOP_AUX_IACK, 1 << irq);
> -}
> -
> struct plat_smp_ops plat_smp_ops = {
> .info = smp_cpuinfo_buf,
> .init_early_smp = eznps_init_cpumasks,
> .cpu_kick = eznps_smp_wakeup_cpu,
> .ipi_send = eznps_ipi_send,
> .init_per_cpu = eznps_init_per_cpu,
> - .ipi_clear = eznps_ipi_clear,
> };
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] ARC: [plat-eznps] remove IPI clear from SMP operations
@ 2016-11-07 18:54 ` Vineet Gupta
0 siblings, 0 replies; 4+ messages in thread
From: Vineet Gupta @ 2016-11-07 18:54 UTC (permalink / raw)
To: linux-snps-arc
On 11/06/2016 11:34 PM, Noam Camus wrote:
> From: Noam Camus <noamca at mellanox.com>
>
> Generic IRQ mechanism is already acknowledge the IPI IRQ.
> Doing this once more time in IPI handler is not needed.
Changelog can be improved !
Generic IRQ mechanism can't ack the irq. The NPQ irqchip driver does this for all
irqs already - so no special handling is needed for IPI in NPS case.
-Vineet
>
> Signed-off-by: Noam Camus <noamca at mellanox.com>
> ---
> arch/arc/plat-eznps/smp.c | 6 ------
> 1 files changed, 0 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arc/plat-eznps/smp.c b/arch/arc/plat-eznps/smp.c
> index 5e901f8..56a4c85 100644
> --- a/arch/arc/plat-eznps/smp.c
> +++ b/arch/arc/plat-eznps/smp.c
> @@ -140,16 +140,10 @@ static void eznps_init_per_cpu(int cpu)
> mtm_enable_core(cpu);
> }
>
> -static void eznps_ipi_clear(int irq)
> -{
> - write_aux_reg(CTOP_AUX_IACK, 1 << irq);
> -}
> -
> struct plat_smp_ops plat_smp_ops = {
> .info = smp_cpuinfo_buf,
> .init_early_smp = eznps_init_cpumasks,
> .cpu_kick = eznps_smp_wakeup_cpu,
> .ipi_send = eznps_ipi_send,
> .init_per_cpu = eznps_init_per_cpu,
> - .ipi_clear = eznps_ipi_clear,
> };
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-11-07 18:54 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-07 7:34 [PATCH] ARC: [plat-eznps] remove IPI clear from SMP operations Noam Camus
2016-11-07 7:34 ` Noam Camus
2016-11-07 18:54 ` Vineet Gupta
2016-11-07 18:54 ` Vineet Gupta
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.