* Guennadi Liakhovetski ([EMAIL PROTECTED]) wrote:
> Provide {enable,disable}_irq_wakeup dummies for undefined
> CONFIG_GENERIC_HARDIRQS case. Completely untested, as I don't even have
> cross-compilers for platforms without CONFIG_GENERIC_HARDIRQS.
>
> Signed-off-by: Guennadi Liakhovetski <[EMAIL PROTECTED]>
>
It builds fine now.
Tested-by: Mathieu Desnoyers <[EMAIL PROTECTED]>
> ---
>
> On Tue, 18 Sep 2007, Andrew Morton wrote:
>
> > On Tue, 18 Sep 2007 16:54:03 -0400
> > Mathieu Desnoyers <[EMAIL PROTECTED]> wrote:
> >
> > > I got the following error when building 2.6.23-rc6-mm1 on sparc:
> > >
> > >
> > >
> > > /opt/crosstool/gcc-4.1.1-glibc-2.3.6/sparc-unknown-linux-gnu/bin/sparc-unknown-linux-gnu-gcc
> > > -Wp,-MD,drivers/serial/.serial_core.o.d -nostdinc -isystem
> > > /opt/crosstool/gcc-4.1.1-glibc-2.3.6/sparc-unknown-linux-gnu/lib/gcc/sparc-unknown-linux-gnu/4.1.1/include
> > > -D__KERNEL__ -Iinclude -Iinclude2
> > > -I/home/compudj/git/linux-2.6-lttng/include -include
> > > include/linux/autoconf.h
> > > -I/home/compudj/git/linux-2.6-lttng/drivers/serial -Idrivers/serial -Wall
> > > -Wundef -Wstrict-prototypes -Wno-trigraphs -fno-strict-aliasing
> > > -fno-common -Werror-implicit-function-declaration -Os -m32 -pipe -mno-fpu
> > > -fcall-used-g5 -fcall-used-g7 -fomit-frame-pointer -fno-stack-protector
> > > -Wdeclaration-after-statement -Wno-pointer-sign -D"KBUILD_STR(s)=#s"
> > > -D"KBUILD_BASENAME=KBUILD_STR(serial_core)"
> > > -D"KBUILD_MODNAME=KBUILD_STR(serial_core)" -c -o
> > > drivers/serial/.tmp_serial_core.o
> > > /home/compudj/git/linux-2.6-lttng/drivers/serial/serial_core.c
> > > /home/compudj/git/linux-2.6-lttng/drivers/serial/serial_core.c: In
> > > function 'uart_suspend_port':
> > > /home/compudj/git/linux-2.6-lttng/drivers/serial/serial_core.c:1980:
> > > error: implicit declaration of function 'enable_irq_wake'
> > > /home/compudj/git/linux-2.6-lttng/drivers/serial/serial_core.c: In
> > > function 'uart_resume_port':
> > > /home/compudj/git/linux-2.6-lttng/drivers/serial/serial_core.c:2035:
> > > error: implicit declaration of function 'disable_irq_wake'
> >
> > hm, I wonder why I didn't hit that.
> >
> > enable_irq_wake() was added by wake-up-from-a-serial-port.patch
> >
> > I note that git-input adds a call too, and might have a problem
> > with !CONFIG_GENERIC_HARDIRQS.
> >
> > Not sure what the best fix is here. We could sprinkle ifdefs all
> > over the code, or just add the suitable empty stubs for enable_irq_wake(),
> > etc.
>
> diff --git a/include/linux/interrupt.h b/include/linux/interrupt.h
> index 5323f62..ecade41 100644
> --- a/include/linux/interrupt.h
> +++ b/include/linux/interrupt.h
> @@ -205,6 +205,9 @@ static inline int disable_irq_wake(unsigned int irq)
> enable_irq(irq)
> # endif
>
> +#define enable_irq_wake(irq) ({ (void)(irq); 0; })
> +#define disable_irq_wake(irq) ({ (void)(irq); 0; })
> +
> #endif /* CONFIG_GENERIC_HARDIRQS */
>
> #ifndef __ARCH_SET_SOFTIRQ_PENDING
--
Mathieu Desnoyers
Computer Engineering Ph.D. Student, Ecole Polytechnique de Montreal
OpenPGP key fingerprint: 8CD5 52C3 8E3C 4140 715F BA06 3F25 A8FE 3BAE 9A68
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
