Re: [PATCH v1 18/20] ARC: [plat-eznps] define IPI_IRQ

From: Vineet Gupta
Date: Mon Nov 02 2015 - 02:52:45 EST


On Saturday 31 October 2015 06:53 PM, Noam Camus wrote:
> From: Noam Camus <noamc@xxxxxxxxxx>
>
> We add IPI irq definition to be used later by any
> irqchip such NPS400 IC.
>
> Signed-off-by: Noam Camus <noamc@xxxxxxxxxx>

Again this break bisectability - it needs to be placed before you use this #define
in patch 3/20.
Anyhow since you are doing stuff via DT, why not get this value from DT itself
instead of hard coding.

> ---
> arch/arc/include/asm/irq.h | 4 ++++
> 1 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arc/include/asm/irq.h b/arch/arc/include/asm/irq.h
> index 4fd7d62..c5f8f0f 100644
> --- a/arch/arc/include/asm/irq.h
> +++ b/arch/arc/include/asm/irq.h
> @@ -16,7 +16,11 @@
> #ifdef CONFIG_ISA_ARCOMPACT
> #define TIMER0_IRQ 3
> #define TIMER1_IRQ 4
> +#if defined(CONFIG_SMP) && defined(CONFIG_ARC_PLAT_EZNPS)
> +#define IPI_IRQ 5
> +#else
> #define IPI_IRQ (NR_CPU_IRQS-1) /* dummy to enable SMP build for up hardware */
> +#endif
> #else
> #define TIMER0_IRQ 16
> #define TIMER1_IRQ 17

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/