diff options
| author | Masahiro Yamada <[email protected]> | 2017-09-18 00:01:44 +0900 |
|---|---|---|
| committer | Petr Mladek <[email protected]> | 2017-10-19 15:29:24 +0200 |
| commit | 93b138dd9d6e01cd360f347512b6c5bf6f0c4ce2 (patch) | |
| tree | e801803461e9e88959b83efdb9cbdaf6175f9e1c /include/linux | |
| parent | cef5d0f952a03d42051141742632078d488b0c6b (diff) | |
printk: simplify no_printk()
Commit 069f0cd00df0 ("printk: Make the printk*once() variants return
a value") surrounded the macro implementation with ({ ... }).
Now, the inner do { ... } while (0); is redundant.
Link: http://lkml.kernel.org/r/[email protected]
Cc: Andrew Morton <[email protected]>
Cc: Ingo Molnar <[email protected]>
Cc: Masahiro Yamada <[email protected]>
Cc: Steven Rostedt <[email protected]>
Cc: [email protected]
Signed-off-by: Masahiro Yamada <[email protected]>
Reviewed-by: Sergey Senozhatsky <[email protected]>
Signed-off-by: Petr Mladek <[email protected]>
Diffstat (limited to 'include/linux')
| -rw-r--r-- | include/linux/printk.h | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/include/linux/printk.h b/include/linux/printk.h index e10f27468322..7911f7364346 100644 --- a/include/linux/printk.h +++ b/include/linux/printk.h @@ -131,10 +131,8 @@ struct va_format { */ #define no_printk(fmt, ...) \ ({ \ - do { \ - if (0) \ - printk(fmt, ##__VA_ARGS__); \ - } while (0); \ + if (0) \ + printk(fmt, ##__VA_ARGS__); \ 0; \ }) |