[PATCH] staging: iio: fix coding style

Jimmy P jimmyp11f155 at gmail.com
Thu Jun 12 09:42:43 UTC 2014


This patch fixes coding style errors reported by checkpatch.pl for
lines that was over 80 chars long. The macro value shoud be put in () as
well.

Signed-off-by: Jimmy P <jimmyp11f155 at gmail.com>
---
 drivers/staging/iio/frequency/ad5930.c | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/iio/frequency/ad5930.c b/drivers/staging/iio/frequency/ad5930.c
index a4aeee6..95a3550 100644
--- a/drivers/staging/iio/frequency/ad5930.c
+++ b/drivers/staging/iio/frequency/ad5930.c
@@ -21,7 +21,7 @@
 
 #define DRV_NAME "ad5930"
 
-#define value_mask (u16)0xf000
+#define value_mask ((u16)0xf000)
 #define addr_shift 12
 
 /* Register format: 4 bits addr + 12 bits value */
@@ -52,12 +52,15 @@ static ssize_t ad5930_set_parameter(struct device *dev,
 
 	config->control = (config->control & ~value_mask);
 	config->incnum = (config->control & ~value_mask) | (1 << addr_shift);
-	config->frqdelt[0] = (config->control & ~value_mask) | (2 << addr_shift);
+	config->frqdelt[0] = (config->control & ~value_mask) |
+				(2 << addr_shift);
 	config->frqdelt[1] = (config->control & ~value_mask) | 3 << addr_shift;
 	config->incitvl = (config->control & ~value_mask) | 4 << addr_shift;
 	config->buritvl = (config->control & ~value_mask) | 8 << addr_shift;
-	config->strtfrq[0] = (config->control & ~value_mask) | 0xc << addr_shift;
-	config->strtfrq[1] = (config->control & ~value_mask) | 0xd << addr_shift;
+	config->strtfrq[0] = (config->control & ~value_mask) |
+				0xc << addr_shift;
+	config->strtfrq[1] = (config->control & ~value_mask) |
+				0xd << addr_shift;
 
 	xfer.len = len;
 	xfer.tx_buf = config;
-- 
2.0.0



More information about the devel mailing list