pinctrl/amd: white space cleanups in amd_gpio_dbg_show()
authorDan Carpenter <dan.carpenter@oracle.com>
Sat, 7 Jan 2017 06:32:15 +0000 (09:32 +0300)
committerLinus Walleij <linus.walleij@linaro.org>
Wed, 11 Jan 2017 13:40:37 +0000 (14:40 +0100)
We accidentally deleted two tabs from the first line, but even with that
fixed the conditions were not really kernel style.  Put the && at the
end of the line so we can align the condition clauses.  Also add spaces
around the "+" operator.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinctrl-amd.c

index c2203699f1ab32e51fb7058b04dcafb177cb9cb6..1ee107f147d04f891c96495103cbb5f268dc4fe6 100644 (file)
@@ -219,14 +219,14 @@ static void amd_gpio_dbg_show(struct seq_file *s, struct gpio_chip *gc)
                        if (pin_reg & BIT(INTERRUPT_ENABLE_OFF)) {
                                interrupt_enable = "interrupt is enabled|";
 
-               if (!(pin_reg & BIT(ACTIVE_LEVEL_OFF))
-                               && !(pin_reg & BIT(ACTIVE_LEVEL_OFF+1)))
+                               if (!(pin_reg & BIT(ACTIVE_LEVEL_OFF)) &&
+                                   !(pin_reg & BIT(ACTIVE_LEVEL_OFF + 1)))
                                        active_level = "Active low|";
-                               else if (pin_reg & BIT(ACTIVE_LEVEL_OFF)
-                               && !(pin_reg & BIT(ACTIVE_LEVEL_OFF+1)))
+                               else if (pin_reg & BIT(ACTIVE_LEVEL_OFF) &&
+                                        !(pin_reg & BIT(ACTIVE_LEVEL_OFF + 1)))
                                        active_level = "Active high|";
-                               else if (!(pin_reg & BIT(ACTIVE_LEVEL_OFF))
-                                       && pin_reg & BIT(ACTIVE_LEVEL_OFF+1))
+                               else if (!(pin_reg & BIT(ACTIVE_LEVEL_OFF)) &&
+                                        pin_reg & BIT(ACTIVE_LEVEL_OFF + 1))
                                        active_level = "Active on both|";
                                else
                                        active_level = "Unknow Active level|";