* Sam Ravnborg <sam@xxxxxxxxxxxx> wrote: > I hit the following build error on sparc: > > drivers/gpio/gpiolib.c: In function `gpiolib_dbg_show': > drivers/gpio/gpiolib.c:1146: error: implicit declaration of function `irq_to_desc' > drivers/gpio/gpiolib.c:1146: warning: initialization makes pointer from integer without a cast > > > This is with an allmodconfig where I did: > > make ARCH=sparc vmlinux > > sparc do _not_ have GENERIC_HARDIRQS set to 'y'. > > I did a simple: > #include <linux/irqnr.h> but that did not help. yeah. Does the patch below do the trick? btw., i'm curious, does the unification of the sparc architectures mean that sparc32 will be using genirq too some time in the future? Ingo ----------------> >From 08522e214542b4afc6c2401bb02900cfca76f600 Mon Sep 17 00:00:00 2001 From: Ingo Molnar <mingo@xxxxxxx> Date: Mon, 5 Jan 2009 14:34:42 +0100 Subject: [PATCH] genirq: provide irq_to_desc() to non-genirq architectures too Impact: build fix on non-genirq architectures Sam Ravnborg reported this build failure on sparc32 allmodconfig, the GPIO drivers assume the presence of irq_to_desc(): drivers/gpio/gpiolib.c: In function `gpiolib_dbg_show': drivers/gpio/gpiolib.c:1146: error: implicit declaration of function 'irq_to_desc' Add it in the !genirq case too. Reported-by: Sam Ravnborg <sam@xxxxxxxxxxxx> Signed-off-by: Ingo Molnar <mingo@xxxxxxx> --- include/linux/irqnr.h | 7 ++++++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/include/linux/irqnr.h b/include/linux/irqnr.h index 5504a5c..6b719fa 100644 --- a/include/linux/irqnr.h +++ b/include/linux/irqnr.h @@ -8,7 +8,12 @@ #ifndef CONFIG_GENERIC_HARDIRQS #include <asm/irq.h> -# define nr_irqs NR_IRQS + +/* + * Wrappers for non-genirq architectures: + */ +#define nr_irqs NR_IRQS +#define irq_to_desc(irq) irq_desc[irq] # define for_each_irq_desc(irq, desc) \ for (irq = 0; irq < nr_irqs; irq++) -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
