Commit 3775dac1 authored by Dan Carpenter's avatar Dan Carpenter Committed by Linus Walleij

pinctrl/amd: white space cleanups in amd_gpio_dbg_show()

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: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
parent 6e8b66c1
...@@ -219,14 +219,14 @@ static void amd_gpio_dbg_show(struct seq_file *s, struct gpio_chip *gc) ...@@ -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)) { if (pin_reg & BIT(INTERRUPT_ENABLE_OFF)) {
interrupt_enable = "interrupt is enabled|"; interrupt_enable = "interrupt is enabled|";
if (!(pin_reg & BIT(ACTIVE_LEVEL_OFF)) if (!(pin_reg & BIT(ACTIVE_LEVEL_OFF)) &&
&& !(pin_reg & BIT(ACTIVE_LEVEL_OFF+1))) !(pin_reg & BIT(ACTIVE_LEVEL_OFF + 1)))
active_level = "Active low|"; active_level = "Active low|";
else if (pin_reg & BIT(ACTIVE_LEVEL_OFF) else if (pin_reg & BIT(ACTIVE_LEVEL_OFF) &&
&& !(pin_reg & BIT(ACTIVE_LEVEL_OFF+1))) !(pin_reg & BIT(ACTIVE_LEVEL_OFF + 1)))
active_level = "Active high|"; active_level = "Active high|";
else if (!(pin_reg & BIT(ACTIVE_LEVEL_OFF)) else if (!(pin_reg & BIT(ACTIVE_LEVEL_OFF)) &&
&& pin_reg & BIT(ACTIVE_LEVEL_OFF+1)) pin_reg & BIT(ACTIVE_LEVEL_OFF + 1))
active_level = "Active on both|"; active_level = "Active on both|";
else else
active_level = "Unknow Active level|"; active_level = "Unknow Active level|";
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment