diff options
author | Jorgyano Vieira <jorgyano@gmail.com> | 2012-02-14 21:20:06 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-02-15 20:21:42 -0500 |
commit | 413db8c1c75a276c046ea476fee9364ca8df4d12 (patch) | |
tree | 646c16600460dee851edf4b8fadd8670d59c63e0 /drivers | |
parent | 2140dc9ce752d9b32c84c63c42d157ef863fe84a (diff) |
Staging: crystalhd: crystalhd_misc: improved debug macros
Improvement of debug macros to ensure safe use on if/else statements.
Signed-off-by: Jorgyano Vieira <jorgyano@gmail.com>
Acked-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/staging/crystalhd/crystalhd_misc.h | 46 |
1 files changed, 25 insertions, 21 deletions
diff --git a/drivers/staging/crystalhd/crystalhd_misc.h b/drivers/staging/crystalhd/crystalhd_misc.h index 4d617235742..e0aa361c393 100644 --- a/drivers/staging/crystalhd/crystalhd_misc.h +++ b/drivers/staging/crystalhd/crystalhd_misc.h | |||
@@ -203,26 +203,30 @@ enum _chd_log_levels { | |||
203 | BCMLOG_ENTER_LEAVE = 0x00000008, /* stack tracking */ | 203 | BCMLOG_ENTER_LEAVE = 0x00000008, /* stack tracking */ |
204 | }; | 204 | }; |
205 | 205 | ||
206 | #define BCMLOG_ENTER \ | 206 | #define BCMLOG_ENTER \ |
207 | if (g_linklog_level & BCMLOG_ENTER_LEAVE) { \ | 207 | do { \ |
208 | printk(KERN_DEBUG "Entered %s\n", __func__); \ | 208 | if (g_linklog_level & BCMLOG_ENTER_LEAVE) \ |
209 | } | 209 | printk(KERN_DEBUG "Entered %s\n", __func__); \ |
210 | 210 | } while (0) | |
211 | #define BCMLOG_LEAVE \ | 211 | |
212 | if (g_linklog_level & BCMLOG_ENTER_LEAVE) { \ | 212 | #define BCMLOG_LEAVE \ |
213 | printk(KERN_DEBUG "Leaving %s\n", __func__); \ | 213 | do { \ |
214 | } | 214 | if (g_linklog_level & BCMLOG_ENTER_LEAVE) \ |
215 | 215 | printk(KERN_DEBUG "Leaving %s\n", __func__); \ | |
216 | #define BCMLOG(trace, fmt, args...) \ | 216 | } while (0) \ |
217 | if (g_linklog_level & trace) { \ | 217 | |
218 | printk(fmt, ##args); \ | 218 | #define BCMLOG(trace, fmt, args...) \ |
219 | } | 219 | do { \ |
220 | 220 | if (g_linklog_level & trace) \ | |
221 | #define BCMLOG_ERR(fmt, args...) \ | 221 | printk(fmt, ##args); \ |
222 | do { \ | 222 | } while (0) |
223 | if (g_linklog_level & BCMLOG_ERROR) { \ | 223 | |
224 | printk(KERN_ERR "*ERR*:%s:%d: "fmt, __FILE__, __LINE__, ##args); \ | 224 | |
225 | } \ | 225 | #define BCMLOG_ERR(fmt, args...) \ |
226 | } while (0); | 226 | do { \ |
227 | if (g_linklog_level & BCMLOG_ERROR) \ | ||
228 | printk(KERN_ERR "*ERR*:%s:%d: "fmt, \ | ||
229 | __FILE__, __LINE__, ##args); \ | ||
230 | } while (0) | ||
227 | 231 | ||
228 | #endif | 232 | #endif |