From: ChinmayVS <cvs...@gmail.com>

Macros with multiple statements should be enclosed in a do - while loop

Signed-off-by: ChinmayVS <cvs...@gmail.com>
---
 drivers/staging/greybus/loopback.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/greybus/loopback.c 
b/drivers/staging/greybus/loopback.c
index 7882306..39f0a25 100644
--- a/drivers/staging/greybus/loopback.c
+++ b/drivers/staging/greybus/loopback.c
@@ -177,9 +177,11 @@ static ssize_t name##_avg_show(struct device *dev,         
\
 static DEVICE_ATTR_RO(name##_avg)
 
 #define gb_loopback_stats_attrs(field)                         \
+do {                                                           \
        gb_loopback_ro_stats_attr(field, min, u);               \
        gb_loopback_ro_stats_attr(field, max, u);               \
-       gb_loopback_ro_avg_attr(field)
+       gb_loopback_ro_avg_attr(field)                          \
+} while (0)
 
 #define gb_loopback_attr(field, type)                                  \
 static ssize_t field##_show(struct device *dev,                                
\
-- 
2.7.4

Reply via email to