On Sun, Oct 21, 2007 at 07:54:53PM +0200, Roel Kluin wrote: > +config GSC_DEBUG > + bool "VSC_GSC_HSC bus debugging" What an ugly prompt string ... > + depends on GSC > + default n default is 'n', by default. No need to specify it. > +#ifdef GSC_DEBUG > + #define DEBUG 1 > +#endif Why the indent? > - DEBPRINTK("%s intr, mask=0x%x\n", gsc_asic->name, irr); > + printk("%s intr, mask=0x%x\n", gsc_asic->name, irr); Presumably you meant pr_debug? -- Intel are signing my paycheques ... these opinions are still mine "Bill, look, we understand that you're interested in selling us this operating system, but compare it to ours. We can't possibly take such a retrograde step." - To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html