ASoC: codecs: tlv320adc3xxx: use new GPIO line value setter callbacks
authorBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Tue, 8 Apr 2025 07:38:25 +0000 (09:38 +0200)
committerMark Brown <broonie@kernel.org>
Sun, 13 Apr 2025 19:23:20 +0000 (20:23 +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.

Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
Link: https://patch.msgid.link/20250408-gpiochip-set-rv-sound-v1-7-dd54b6ca1ef9@linaro.org
Reviewed-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/tlv320adc3xxx.c

index 191e067ed1c9bc20b51dda0c9ec4a2f59e9673cc..1035ba17dc5d41a77a402a401a1afdb3ddd5723a 100644 (file)
@@ -1015,10 +1015,10 @@ static int adc3xxx_gpio_direction_out(struct gpio_chip *chip,
  * so we set the output mode and output value in the same call. Hence
  * .set in practice does the same thing as .direction_out .
  */
-static void adc3xxx_gpio_set(struct gpio_chip *chip, unsigned int offset,
-                            int value)
+static int adc3xxx_gpio_set(struct gpio_chip *chip, unsigned int offset,
+                           int value)
 {
-       (void) adc3xxx_gpio_direction_out(chip, offset, value);
+       return adc3xxx_gpio_direction_out(chip, offset, value);
 }
 
 /* Even though we only support GPIO output for now, some GPIO clients
@@ -1052,7 +1052,7 @@ static const struct gpio_chip adc3xxx_gpio_chip = {
        .owner                  = THIS_MODULE,
        .request                = adc3xxx_gpio_request,
        .direction_output       = adc3xxx_gpio_direction_out,
-       .set                    = adc3xxx_gpio_set,
+       .set_rv                 = adc3xxx_gpio_set,
        .get                    = adc3xxx_gpio_get,
        .can_sleep              = 1,
 };