On Wednesday March 7, [EMAIL PROTECTED] wrote: > > Replace the apparently misspelled preprocessor variable "PRINTK" > with "CONFIG_PRINTK".
No, it is meant to be "PRINTK". It dates way way back before my time, but presumably the idea was you could -DPRINTK=something and if you didn't do that, it would figure out what it thought you wanted. Definitely not meant to be CONFIG_PRINTK. NeilBrown > > Signed-off-by: Robert P. J. Day <[EMAIL PROTECTED]> > > --- > > not sure who the official maintainer here is, sorry. > > diff --git a/drivers/md/bitmap.c b/drivers/md/bitmap.c > index 5554ada..0c09772 100644 > --- a/drivers/md/bitmap.c > +++ b/drivers/md/bitmap.c > @@ -53,7 +53,7 @@ > //#define DPRINTK PRINTK /* set this NULL to avoid verbose debug output */ > #define DPRINTK(x...) do { } while(0) > > -#ifndef PRINTK > +#ifndef CONFIG_PRINTK > # if DEBUG > 0 > # define PRINTK(x...) printk(KERN_DEBUG x) > # else > > -- > ======================================================================== > Robert P. J. Day > Linux Consulting, Training and Annoying Kernel Pedantry > Waterloo, Ontario, CANADA > > http://fsdev.net/wiki/index.php?title=Main_Page > ======================================================================== > - > 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/ - 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/