]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
gpio: creg-snps: use new line value setter callbacks
authorBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Mon, 10 Mar 2025 12:40:24 +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-10-03798bb833eb@linaro.org
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
drivers/gpio/gpio-creg-snps.c

index 4968232f70f2a1710de1096f4837a6dee26af1ef..8b49f02c78967a9c9eec71935c1e5ccfce9efefa 100644 (file)
@@ -27,7 +27,7 @@ struct creg_gpio {
        const struct creg_layout *layout;
 };
 
-static void creg_gpio_set(struct gpio_chip *gc, unsigned int offset, int val)
+static int creg_gpio_set(struct gpio_chip *gc, unsigned int offset, int val)
 {
        struct creg_gpio *hcg = gpiochip_get_data(gc);
        const struct creg_layout *layout = hcg->layout;
@@ -47,13 +47,13 @@ static void creg_gpio_set(struct gpio_chip *gc, unsigned int offset, int val)
        reg |=  (value << reg_shift);
        writel(reg, hcg->regs);
        spin_unlock_irqrestore(&hcg->lock, flags);
+
+       return 0;
 }
 
 static int creg_gpio_dir_out(struct gpio_chip *gc, unsigned int offset, int val)
 {
-       creg_gpio_set(gc, offset, val);
-
-       return 0;
+       return creg_gpio_set(gc, offset, val);
 }
 
 static int creg_gpio_validate_pg(struct device *dev, struct creg_gpio *hcg,
@@ -167,7 +167,7 @@ static int creg_gpio_probe(struct platform_device *pdev)
        hcg->gc.label = dev_name(dev);
        hcg->gc.base = -1;
        hcg->gc.ngpio = ngpios;
-       hcg->gc.set = creg_gpio_set;
+       hcg->gc.set_rv = creg_gpio_set;
        hcg->gc.direction_output = creg_gpio_dir_out;
 
        ret = devm_gpiochip_add_data(dev, &hcg->gc, hcg);