Kenji Kaneshige wrote: > How about below. I think we can write printk in one line. > > #define dbg(format, arg...) \ > do { \ > if (pciehp_debug) \ > printk(KERN_DEBUG "%s: " format, MY_NAME, ## arg); \ > } while (0) I wanted to keep the change as simple as possible and didn't wanna play with surrounding indentation. But yes, if thats preferred... i gonna change the patch to this. Shall i? Thanks, Frank -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html