gpio: janz-ttl: Use BIT() macro
authorLinus Walleij <linus.walleij@linaro.org>
Mon, 12 Mar 2018 11:11:45 +0000 (12:11 +0100)
committerLinus Walleij <linus.walleij@linaro.org>
Mon, 19 Mar 2018 00:50:30 +0000 (01:50 +0100)
This makes the code more readable by using the BIT() macro.

Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-janz-ttl.c

index e9f4612..6b9bf8b 100644 (file)
@@ -18,6 +18,7 @@
 #include <linux/io.h>
 #include <linux/gpio/driver.h>
 #include <linux/slab.h>
+#include <linux/bitops.h>
 
 #include <linux/mfd/janz.h>
 
@@ -33,9 +34,9 @@
 #define MASTER_INT_CTL         0x00
 #define MASTER_CONF_CTL                0x01
 
-#define CONF_PAE               (1 << 2)
-#define CONF_PBE               (1 << 7)
-#define CONF_PCE               (1 << 4)
+#define CONF_PAE               BIT(2)
+#define CONF_PBE               BIT(7)
+#define CONF_PCE               BIT(4)
 
 struct ttl_control_regs {
        __be16 portc;
@@ -74,7 +75,7 @@ static int ttl_get_value(struct gpio_chip *gpio, unsigned offset)
        }
 
        spin_lock(&mod->lock);
-       ret = *shadow & (1 << offset);
+       ret = *shadow & BIT(offset);
        spin_unlock(&mod->lock);
        return !!ret;
 }
@@ -100,9 +101,9 @@ static void ttl_set_value(struct gpio_chip *gpio, unsigned offset, int value)
 
        spin_lock(&mod->lock);
        if (value)
-               *shadow |= (1 << offset);
+               *shadow |= BIT(offset);
        else
-               *shadow &= ~(1 << offset);
+               *shadow &= ~BIT(offset);
 
        iowrite16be(*shadow, port);
        spin_unlock(&mod->lock);