On Fri, 1 Dec 2006 11:19:34 +0100, "Franck Bui-Huu" <vagabon.xyz@xxxxxxxxx> wrote: > ok bad example. Why not making the select thing part of the platform > config like this ? > > diff --git a/arch/mips/Kconfig b/arch/mips/Kconfig > index 5ff94e5..8565533 100644 > --- a/arch/mips/Kconfig > +++ b/arch/mips/Kconfig > @@ -233,6 +233,7 @@ config LASAT > select SYS_SUPPORTS_32BIT_KERNEL > select SYS_SUPPORTS_64BIT_KERNEL if EXPERIMENTAL > select SYS_SUPPORTS_LITTLE_ENDIAN > + select GENERIC_HARDIRQS_NO__DO_IRQ > > config MIPS_ATLAS > bool "MIPS Atlas board" > @@ -913,6 +914,10 @@ config SYS_SUPPORTS_BIG_ENDIAN > config SYS_SUPPORTS_LITTLE_ENDIAN > bool > > +config GENERIC_HARDIRQS_NO__DO_IRQ > + bool > + default n > + > config IRQ_CPU > bool This looks good for me. Also, if you selected GENERIC_HARDIRQS_NO__DO_IRQ, you can remove .end handler. But adding "#ifdef GENERIC_HARDIRQS_NO__DO_IRQ" for each .end might be slightly ugly... --- Atsushi Nemoto