pinctrl: intel: Decrease indentation in intel_gpio_set()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Thu, 24 Aug 2017 08:19:34 +0000 (11:19 +0300)
committerLinus Walleij <linus.walleij@linaro.org>
Thu, 31 Aug 2017 12:46:36 +0000 (14:46 +0200)
Decrease indentation in intel_gpio_set() to make it looking slightly better
and be in align with intel_gpio_get().

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/intel/pinctrl-intel.c

index 8f87215386161575f6667c6b4993ff996cf196f9..ac806891ff817d6e25f0753335cc9ea9e2c6891b 100644 (file)
@@ -762,22 +762,22 @@ static int intel_gpio_get(struct gpio_chip *chip, unsigned offset)
 static void intel_gpio_set(struct gpio_chip *chip, unsigned offset, int value)
 {
        struct intel_pinctrl *pctrl = gpiochip_get_data(chip);
+       unsigned long flags;
        void __iomem *reg;
+       u32 padcfg0;
 
        reg = intel_get_padcfg(pctrl, offset, PADCFG0);
-       if (reg) {
-               unsigned long flags;
-               u32 padcfg0;
+       if (!reg)
+               return;
 
-               raw_spin_lock_irqsave(&pctrl->lock, flags);
-               padcfg0 = readl(reg);
-               if (value)
-                       padcfg0 |= PADCFG0_GPIOTXSTATE;
-               else
-                       padcfg0 &= ~PADCFG0_GPIOTXSTATE;
-               writel(padcfg0, reg);
-               raw_spin_unlock_irqrestore(&pctrl->lock, flags);
-       }
+       raw_spin_lock_irqsave(&pctrl->lock, flags);
+       padcfg0 = readl(reg);
+       if (value)
+               padcfg0 |= PADCFG0_GPIOTXSTATE;
+       else
+               padcfg0 &= ~PADCFG0_GPIOTXSTATE;
+       writel(padcfg0, reg);
+       raw_spin_unlock_irqrestore(&pctrl->lock, flags);
 }
 
 static int intel_gpio_direction_input(struct gpio_chip *chip, unsigned offset)