It's now clean. Well, from a coding style guide, not from a logic standpoint, the whole file needs to be tossed overboard and cheered on as the sharks tear it to individual bits. Cc: Hank Janssen <hjanssen@xxxxxxxxxxxxx> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxx> --- drivers/staging/hv/include/logging.h | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/hv/include/logging.h b/drivers/staging/hv/include/logging.h index 74c18f1..9e55617 100644 --- a/drivers/staging/hv/include/logging.h +++ b/drivers/staging/hv/include/logging.h @@ -62,11 +62,11 @@ extern unsigned int vmbus_loglevel; #define ASSERT(expr) \ - if (!(expr)) { \ + if (!(expr)) { \ printk(KERN_CRIT "Assertion failed! %s,%s,%s,line=%d\n", \ #expr, __FILE__, __func__, __LINE__); \ __asm__ __volatile__("int3"); \ - } + } #define DPRINT(mod, lvl, fmt, args...) do {\ if ((mod & (HIWORD(vmbus_loglevel))) && \ @@ -89,7 +89,7 @@ extern unsigned int vmbus_loglevel; #define DPRINT_WARN(mod, fmt, args...) do {\ if ((mod & (HIWORD(vmbus_loglevel))) && \ (WARNING_LVL <= LOWORD(vmbus_loglevel))) \ - printk(KERN_WARNING #mod": WARNING! " fmt "\n", ## args);\ + printk(KERN_WARNING #mod": WARNING! " fmt "\n", ## args);\ } while (0) #define DPRINT_ERR(mod, fmt, args...) do {\ @@ -103,7 +103,7 @@ extern unsigned int vmbus_loglevel; #define DPRINT_ENTER(mod) do {\ if ((mod & (HIWORD(vmbus_loglevel))) && \ (DEBUG_LVL_ENTEREXIT <= LOWORD(vmbus_loglevel))) \ - printk(KERN_DEBUG "["#mod"]: %s() enter\n", __func__);\ + printk(KERN_DEBUG "["#mod"]: %s() enter\n", __func__);\ } while (0) #define DPRINT_EXIT(mod) do {\ -- 1.6.4.2 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel