Message ID | 20250220-gpio-set-retval-v2-1-bc4cfd38dae3@linaro.org |
---|---|
State | New |
Headers | show |
Series | gpiolib: indicate errors in value setters | expand |
On Thu, 20 Feb 2025 10:56:58 +0100, Bartosz Golaszewski wrote: > While gpiod_set_value() currently returns void, it will soon be converted > to return an integer instead. Don't do `return gpiod_set...`. > > Applied, thanks! [01/15] leds: aw200xx: don't use return with gpiod_set_value() variants commit: 5d5e2a6f15a6c5e0c431c1388fd90e14b448da1e -- Lee Jones [李琼斯]
On Thu, Mar 6, 2025 at 11:51 PM Lee Jones <lee@kernel.org> wrote: > > On Thu, 20 Feb 2025 10:56:58 +0100, Bartosz Golaszewski wrote: > > While gpiod_set_value() currently returns void, it will soon be converted > > to return an integer instead. Don't do `return gpiod_set...`. > > > > > > Applied, thanks! > > [01/15] leds: aw200xx: don't use return with gpiod_set_value() variants > commit: 5d5e2a6f15a6c5e0c431c1388fd90e14b448da1e > Hi Lee! Can you please drop it from your tree? You acked v1 of this patch after I had already sent v2 (this patch remained unchanged) folded into a respin of the bigger GPIO series that had triggered build bots to point this issue out in the first place. I picked the entire series up and this commit is already in next as 129fdfe25ac513018d5fe85b0c493025193ef19f. Thanks, Bartosz
diff --git a/drivers/leds/leds-aw200xx.c b/drivers/leds/leds-aw200xx.c index 08cca128458c..fe223d363a5d 100644 --- a/drivers/leds/leds-aw200xx.c +++ b/drivers/leds/leds-aw200xx.c @@ -379,7 +379,7 @@ static void aw200xx_enable(const struct aw200xx *const chip) static void aw200xx_disable(const struct aw200xx *const chip) { - return gpiod_set_value_cansleep(chip->hwen, 0); + gpiod_set_value_cansleep(chip->hwen, 0); } static int aw200xx_probe_get_display_rows(struct device *dev,