iio: accel: bma220: Use BIT() and GENMASK() macros

Code is better to read when numbers of bit are explicitly used.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20200831090813.78841-7-andriy.shevchenko@linux.intel.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

diff --git a/drivers/iio/accel/bma220_spi.c b/drivers/iio/accel/bma220_spi.c
index 73927f9c9b015026b205e293b9823d6e7f764595..efdf22e0a23ae1d4089d383bd4dbf7d0c686800d 100644 (file)
--- a/drivers/iio/accel/bma220_spi.c
+++ b/drivers/iio/accel/bma220_spi.c
@@ -5,6 +5,7 @@
  * Copyright (c) 2016, Intel Corporation.
  */
 
+#include <linux/bits.h>
 #include <linux/kernel.h>
 #include <linux/mod_devicetable.h>
 #include <linux/module.h>
@@ -24,8 +25,8 @@
 #define BMA220_REG_SUSPEND                     0x18
 
 #define BMA220_CHIP_ID                         0xDD
-#define BMA220_READ_MASK                       0x80
-#define BMA220_RANGE_MASK                      0x03
+#define BMA220_READ_MASK                       BIT(7)
+#define BMA220_RANGE_MASK                      GENMASK(1, 0)
 #define BMA220_DATA_SHIFT                      2
 #define BMA220_SUSPEND_SLEEP                   0xFF
 #define BMA220_SUSPEND_WAKE                    0x00