diff options
| author | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2006-08-02 12:44:00 -0400 |
|---|---|---|
| committer | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2006-09-17 13:31:20 -0400 |
| commit | 611aa19fd60fe57059d9972fa6ae29f7472a13cf (patch) | |
| tree | a87f71ef393eec3e904291d1d5da1f2fdc5bb11b /drivers/ieee1394/dv1394.c | |
| parent | 4b60912e52bc6ccdf587f2b92f3435ee2678d730 (diff) | |
ieee1394: safer definition of empty macros
A deactivated macro, defined as "#define foo(bar)", will result in
silent corruption if somebody forgets a semicolon after a call to foo.
Replace it by "#define foo(bar) do {} while (0)" which will reveal any
respective syntax errors.
Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Diffstat (limited to 'drivers/ieee1394/dv1394.c')
| -rw-r--r-- | drivers/ieee1394/dv1394.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/ieee1394/dv1394.c b/drivers/ieee1394/dv1394.c index 6e71d68b1099..6c72f04b2b5d 100644 --- a/drivers/ieee1394/dv1394.c +++ b/drivers/ieee1394/dv1394.c | |||
| @@ -137,13 +137,13 @@ | |||
| 137 | #if DV1394_DEBUG_LEVEL >= 2 | 137 | #if DV1394_DEBUG_LEVEL >= 2 |
| 138 | #define irq_printk( args... ) printk( args ) | 138 | #define irq_printk( args... ) printk( args ) |
| 139 | #else | 139 | #else |
| 140 | #define irq_printk( args... ) | 140 | #define irq_printk( args... ) do {} while (0) |
| 141 | #endif | 141 | #endif |
| 142 | 142 | ||
| 143 | #if DV1394_DEBUG_LEVEL >= 1 | 143 | #if DV1394_DEBUG_LEVEL >= 1 |
| 144 | #define debug_printk( args... ) printk( args) | 144 | #define debug_printk( args... ) printk( args) |
| 145 | #else | 145 | #else |
| 146 | #define debug_printk( args... ) | 146 | #define debug_printk( args... ) do {} while (0) |
| 147 | #endif | 147 | #endif |
| 148 | 148 | ||
| 149 | /* issue a dummy PCI read to force the preceding write | 149 | /* issue a dummy PCI read to force the preceding write |