]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
gpio: crystalcove: use new line value setter callbacks
authorBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Mon, 10 Mar 2025 12:40:26 +0000 (13:40 +0100)
committerBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Mon, 17 Mar 2025 07:27:41 +0000 (08:27 +0100)
struct gpio_chip now has callbacks for setting line values that return
an integer, allowing to indicate failures. Convert the driver to using
them.

Link: https://lore.kernel.org/r/20250310-gpiochip-set-conversion-v1-12-03798bb833eb@linaro.org
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
drivers/gpio/gpio-crystalcove.c

index 56effd0f50c763e38ed0a07799f34c8cf4176d4f..8db7cca3a060b65c80095cd14369cfb7380d3d04 100644 (file)
@@ -168,18 +168,18 @@ static int crystalcove_gpio_get(struct gpio_chip *chip, unsigned int gpio)
        return val & 0x1;
 }
 
-static void crystalcove_gpio_set(struct gpio_chip *chip, unsigned int gpio, int value)
+static int crystalcove_gpio_set(struct gpio_chip *chip, unsigned int gpio, int value)
 {
        struct crystalcove_gpio *cg = gpiochip_get_data(chip);
        int reg = to_reg(gpio, CTRL_OUT);
 
        if (reg < 0)
-               return;
+               return 0;
 
        if (value)
-               regmap_update_bits(cg->regmap, reg, 1, 1);
-       else
-               regmap_update_bits(cg->regmap, reg, 1, 0);
+               return regmap_update_bits(cg->regmap, reg, 1, 1);
+
+       return regmap_update_bits(cg->regmap, reg, 1, 0);
 }
 
 static int crystalcove_irq_type(struct irq_data *data, unsigned int type)
@@ -349,7 +349,7 @@ static int crystalcove_gpio_probe(struct platform_device *pdev)
        cg->chip.direction_input = crystalcove_gpio_dir_in;
        cg->chip.direction_output = crystalcove_gpio_dir_out;
        cg->chip.get = crystalcove_gpio_get;
-       cg->chip.set = crystalcove_gpio_set;
+       cg->chip.set_rv = crystalcove_gpio_set;
        cg->chip.base = -1;
        cg->chip.ngpio = CRYSTALCOVE_VGPIO_NUM;
        cg->chip.can_sleep = true;