gpio: cadence: use lock guards

Simplify the code by using lock guards for the bgpio_lock. While at it:
move the gpio/driver.h include into its correct place alphabetically.

Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Link: https://lore.kernel.org/r/20250702-gpio-mmio-rework-v2-5-6b77aab684d8@linaro.org
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

diff --git a/drivers/gpio/gpio-cadence.c b/drivers/gpio/gpio-cadence.c
index e9dd2564c54f879cf1d49442dfa3db0004473d35..e6ec341d55e9047b2fa8718799ba72d54624388e 100644 (file)
--- a/drivers/gpio/gpio-cadence.c
+++ b/drivers/gpio/gpio-cadence.c
@@ -8,8 +8,9 @@
  *  Boris Brezillon <boris.brezillon@free-electrons.com>
  */
 
-#include <linux/gpio/driver.h>
+#include <linux/cleanup.h>
 #include <linux/clk.h>
+#include <linux/gpio/driver.h>
 #include <linux/interrupt.h>
 #include <linux/kernel.h>
 #include <linux/module.h>
@@ -38,29 +39,24 @@ struct cdns_gpio_chip {
 static int cdns_gpio_request(struct gpio_chip *chip, unsigned int offset)
 {
        struct cdns_gpio_chip *cgpio = gpiochip_get_data(chip);
-       unsigned long flags;
 
-       raw_spin_lock_irqsave(&chip->bgpio_lock, flags);
+       guard(raw_spinlock)(&chip->bgpio_lock);
 
        iowrite32(ioread32(cgpio->regs + CDNS_GPIO_BYPASS_MODE) & ~BIT(offset),
                  cgpio->regs + CDNS_GPIO_BYPASS_MODE);
 
-       raw_spin_unlock_irqrestore(&chip->bgpio_lock, flags);
        return 0;
 }
 
 static void cdns_gpio_free(struct gpio_chip *chip, unsigned int offset)
 {
        struct cdns_gpio_chip *cgpio = gpiochip_get_data(chip);
-       unsigned long flags;
 
-       raw_spin_lock_irqsave(&chip->bgpio_lock, flags);
+       guard(raw_spinlock)(&chip->bgpio_lock);
 
        iowrite32(ioread32(cgpio->regs + CDNS_GPIO_BYPASS_MODE) |
                  (BIT(offset) & cgpio->bypass_orig),
                  cgpio->regs + CDNS_GPIO_BYPASS_MODE);
-
-       raw_spin_unlock_irqrestore(&chip->bgpio_lock, flags);
 }
 
 static void cdns_gpio_irq_mask(struct irq_data *d)
@@ -85,13 +81,12 @@ static int cdns_gpio_irq_set_type(struct irq_data *d, unsigned int type)
 {
        struct gpio_chip *chip = irq_data_get_irq_chip_data(d);
        struct cdns_gpio_chip *cgpio = gpiochip_get_data(chip);
-       unsigned long flags;
        u32 int_value;
        u32 int_type;
        u32 mask = BIT(d->hwirq);
        int ret = 0;
 
-       raw_spin_lock_irqsave(&chip->bgpio_lock, flags);
+       guard(raw_spinlock)(&chip->bgpio_lock);
 
        int_value = ioread32(cgpio->regs + CDNS_GPIO_IRQ_VALUE) & ~mask;
        int_type = ioread32(cgpio->regs + CDNS_GPIO_IRQ_TYPE) & ~mask;
@@ -108,15 +103,12 @@ static int cdns_gpio_irq_set_type(struct irq_data *d, unsigned int type)
        } else if (type == IRQ_TYPE_LEVEL_LOW) {
                int_type |= mask;
        } else {
-               ret = -EINVAL;
-               goto err_irq_type;
+               return -EINVAL;
        }
 
        iowrite32(int_value, cgpio->regs + CDNS_GPIO_IRQ_VALUE);
        iowrite32(int_type, cgpio->regs + CDNS_GPIO_IRQ_TYPE);
 
-err_irq_type:
-       raw_spin_unlock_irqrestore(&chip->bgpio_lock, flags);
        return ret;
 }