diff options
author | David Brownell <dbrownell@users.sourceforge.net> | 2008-07-29 22:28:12 -0700 |
---|---|---|
committer | David Woodhouse <David.Woodhouse@intel.com> | 2008-07-30 14:21:05 +0100 |
commit | 95b1bc20532c18e3f19cd460c8350350c84ffbb2 (patch) | |
tree | 0ec3aef3787c979f1425184e51a157e5e351cdf3 | |
parent | 771999b65f79264acde4b855e5d35696eca5e80c (diff) | |
download | op-kernel-dev-95b1bc20532c18e3f19cd460c8350350c84ffbb2.zip op-kernel-dev-95b1bc20532c18e3f19cd460c8350350c84ffbb2.tar.gz |
[MTD] MTD_DEBUG always does compile-time typechecks
The current style for debug messages is to ensure they're always
parsed by the compiler and then subjected to dead code removal.
That way builds won't break only when debug options get enabled,
which is common when they are stripped out early by CPP.
This patch makes CONFIG_MTD_DEBUG adopt that convention.
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
-rw-r--r-- | include/linux/mtd/mtd.h | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/include/linux/mtd/mtd.h b/include/linux/mtd/mtd.h index 4ed40ca..9226365 100644 --- a/include/linux/mtd/mtd.h +++ b/include/linux/mtd/mtd.h @@ -272,7 +272,11 @@ static inline void mtd_erase_callback(struct erase_info *instr) printk(KERN_INFO args); \ } while(0) #else /* CONFIG_MTD_DEBUG */ -#define DEBUG(n, args...) do { } while(0) +#define DEBUG(n, args...) \ + do { \ + if (0) \ + printk(KERN_INFO args); \ + } while(0) #endif /* CONFIG_MTD_DEBUG */ |