castet.matthieu@xxxxxxx wrote: >Hi, > >Selon Michael Krufky <mkrufky@xxxxxxx>: > >>Johannes Stezenbach wrote: >> >>>Michael Krufky wrote: >>> >>> >>>>+#define dprintk(args...) \ >>>>+do { \ >>>>+if (debug) printk(KERN_DEBUG "lgdt3302: " args); \ >>>>+} while (0) >>>> >>>Interesting formatting... >>> >>> >Yes but why do you need do {} while(0) stuff ? >You don't need to declare a new block as you add only a if statement, so > > >>>+#define dprintk(args...) if (debug) printk(KERN_DEBUG "lgdt3302: " args) >>> >>> >will be a lot's of cleanner... > > >Matthieu > > Agreed... I will add make that change. -- Michael Krufky