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

2015-11-02 Thread Noam Camus
> From: Vineet Gupta [mailto:vineet.gup...@synopsys.com] 
> Sent: Monday, November 02, 2015 9:53 AM

> 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.

OK I will change order.
However last commit should protect us since without it nothing will be compiled.

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


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

2015-11-01 Thread Vineet Gupta
On Saturday 31 October 2015 06:53 PM, Noam Camus wrote:
> From: Noam Camus 
>
> We add IPI irq definition to be used later by any
> irqchip such NPS400 IC.
>
> Signed-off-by: Noam Camus 

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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


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

2015-10-31 Thread Noam Camus
From: Noam Camus 

We add IPI irq definition to be used later by any
irqchip such NPS400 IC.

Signed-off-by: Noam Camus 
---
 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
-- 
1.7.1

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