diff options
author | Cornelia Huck <cornelia.huck@de.ibm.com> | 2008-12-04 10:55:47 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2009-01-06 13:44:33 -0500 |
commit | d0d85ff989222f08dd1fa66321fef5567bbc4a7b (patch) | |
tree | 77af68b3dcd51d95c231025dcb3e4dd3bd21b285 | |
parent | b9daa99ee533578e3f88231e7a16784dcb44ec42 (diff) |
Make DEBUG take precedence over DYNAMIC_PRINTK_DEBUG
Statically defined DEBUG should take precedence over
dynamically enabled debugging; otherwise adding DEBUG
(like, for example, via CONFIG_DEBUG_KOBJECT) does not
have the expected result of printing pr_debug() and dev_dbg()
messages unconditionally.
Signed-off-by: Cornelia Huck <cornelia.huck@de.ibm.com>
Acked-by: Jason Baron <jbaron@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r-- | include/linux/device.h | 8 | ||||
-rw-r--r-- | include/linux/kernel.h | 8 |
2 files changed, 8 insertions, 8 deletions
diff --git a/include/linux/device.h b/include/linux/device.h index e21b5d69d67c..b97a0cf1eb05 100644 --- a/include/linux/device.h +++ b/include/linux/device.h | |||
@@ -553,13 +553,13 @@ extern const char *dev_driver_string(const struct device *dev); | |||
553 | #define dev_info(dev, format, arg...) \ | 553 | #define dev_info(dev, format, arg...) \ |
554 | dev_printk(KERN_INFO , dev , format , ## arg) | 554 | dev_printk(KERN_INFO , dev , format , ## arg) |
555 | 555 | ||
556 | #if defined(CONFIG_DYNAMIC_PRINTK_DEBUG) | 556 | #if defined(DEBUG) |
557 | #define dev_dbg(dev, format, arg...) \ | ||
558 | dev_printk(KERN_DEBUG , dev , format , ## arg) | ||
559 | #elif defined(CONFIG_DYNAMIC_PRINTK_DEBUG) | ||
557 | #define dev_dbg(dev, format, ...) do { \ | 560 | #define dev_dbg(dev, format, ...) do { \ |
558 | dynamic_dev_dbg(dev, format, ##__VA_ARGS__); \ | 561 | dynamic_dev_dbg(dev, format, ##__VA_ARGS__); \ |
559 | } while (0) | 562 | } while (0) |
560 | #elif defined(DEBUG) | ||
561 | #define dev_dbg(dev, format, arg...) \ | ||
562 | dev_printk(KERN_DEBUG , dev , format , ## arg) | ||
563 | #else | 563 | #else |
564 | #define dev_dbg(dev, format, arg...) \ | 564 | #define dev_dbg(dev, format, arg...) \ |
565 | ({ if (0) dev_printk(KERN_DEBUG, dev, format, ##arg); 0; }) | 565 | ({ if (0) dev_printk(KERN_DEBUG, dev, format, ##arg); 0; }) |
diff --git a/include/linux/kernel.h b/include/linux/kernel.h index ca9ff6411dfa..d242fe1381fd 100644 --- a/include/linux/kernel.h +++ b/include/linux/kernel.h | |||
@@ -349,13 +349,13 @@ static inline char *pack_hex_byte(char *buf, u8 byte) | |||
349 | printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__) | 349 | printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__) |
350 | 350 | ||
351 | /* If you are writing a driver, please use dev_dbg instead */ | 351 | /* If you are writing a driver, please use dev_dbg instead */ |
352 | #if defined(CONFIG_DYNAMIC_PRINTK_DEBUG) | 352 | #if defined(DEBUG) |
353 | #define pr_debug(fmt, ...) \ | ||
354 | printk(KERN_DEBUG pr_fmt(fmt), ##__VA_ARGS__) | ||
355 | #elif defined(CONFIG_DYNAMIC_PRINTK_DEBUG) | ||
353 | #define pr_debug(fmt, ...) do { \ | 356 | #define pr_debug(fmt, ...) do { \ |
354 | dynamic_pr_debug(pr_fmt(fmt), ##__VA_ARGS__); \ | 357 | dynamic_pr_debug(pr_fmt(fmt), ##__VA_ARGS__); \ |
355 | } while (0) | 358 | } while (0) |
356 | #elif defined(DEBUG) | ||
357 | #define pr_debug(fmt, ...) \ | ||
358 | printk(KERN_DEBUG pr_fmt(fmt), ##__VA_ARGS__) | ||
359 | #else | 359 | #else |
360 | #define pr_debug(fmt, ...) \ | 360 | #define pr_debug(fmt, ...) \ |
361 | ({ if (0) printk(KERN_DEBUG pr_fmt(fmt), ##__VA_ARGS__); 0; }) | 361 | ({ if (0) printk(KERN_DEBUG pr_fmt(fmt), ##__VA_ARGS__); 0; }) |