Commit 4ebe6f46 authored Mar 25, 2015 by Helen Fornazier Committed by Greg Kroah-Hartman Mar 26, 2015

staging: fbtft: Remove do {} while(0) in single statement macro



This patch fixes the checkpatch.pl warning:

WARNING: Single statement macros should not use a do {} while (0) loop
+#define write_reg(par, ...)                                              \
+do {                                                                     \
+       par->fbtftops.write_register(par, NUMARGS(__VA_ARGS__), __VA_ARGS__); \
+} while (0)

Signed-off-by: Helen Fornazier <helen.fornazier@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent 1f5f636c

drivers/staging/fbtft/fbtft.h

+1 −3

Original line number	Diff line number	Diff line
		@@ -259,9 +259,7 @@ struct fbtft_par {
		#define NUMARGS(...) (sizeof((int[]){__VA_ARGS__})/sizeof(int))

		#define write_reg(par, ...) \
		do { \
		par->fbtftops.write_register(par, NUMARGS(__VA_ARGS__), __VA_ARGS__); \
		} while (0)
		par->fbtftops.write_register(par, NUMARGS(__VA_ARGS__), __VA_ARGS__)

		/* fbtft-core.c */
		extern void fbtft_dbg_hex(const struct device *dev,