[PATCH] frv: Remove bogus NO_IRQ = -1 define

[PATCH] frv: Remove bogus NO_IRQ = -1 define

[Alan Cox]

The old NO_IRQ define some platforms had was long ago declared obsolete
and wrong. FRV should therefore not be re-introducing this, especially as
IRQs are usually unsigned in the kernel. The "no IRQ" case is defined to be
zero and Linus made this rather clear at the time.

arch/frv shows no dependancy on this but it might show up driver fixes
needing doing I guess

Signed-off-by: Alan Cox <alan@redhat.com>

--- include/asm-frv//irq.h~	2007-11-07 16:35:09.927759328 +0000
+++ include/asm-frv/irq.h	2007-11-07 16:35:09.944756744 +0000
@@ -12,9 +12,6 @@
 #ifndef _ASM_IRQ_H_
 #define _ASM_IRQ_H_
 
-/* this number is used when no interrupt has been assigned */
-#define NO_IRQ				(-1)
-
 #define NR_IRQS				48
 #define IRQ_BASE_CPU			(0 * 16)
 #define IRQ_BASE_FPGA			(1 * 16)

Re: [PATCH] frv: Remove bogus NO_IRQ = -1 define

[David Howells]

Alan Cox <alan@lxorguk.ukuu.org.uk> wrote:

> The old NO_IRQ define some platforms had was long ago declared obsolete
> and wrong. FRV should therefore not be re-introducing this, especially as
> IRQs are usually unsigned in the kernel. The "no IRQ" case is defined to be
> zero and Linus made this rather clear at the time.
> 
> arch/frv shows no dependancy on this but it might show up driver fixes
> needing doing I guess

FRV doesn't use IRQ 0, so using 0 instead is fine.

Acked-by: David Howells <dhowells@redhat.com>