Commit db1baf69 authored by Dan Carpenter's avatar Dan Carpenter Committed by Bartosz Golaszewski
Browse files

gpio: aggregator: Return an error if there are no GPIOs in gpio_aggregator_parse() 



The error handling in gpio_aggregator_parse() was re-written.  It now
returns success if there are no GPIOs.  Restore the previous behavior
and return -EINVAL instead.

Fixes: 83c8e3df ("gpio: aggregator: expose aggregator created via legacy sysfs to configfs")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@linaro.org>
Acked-by: default avatarKoichiro Den <koichiro.den@canonical.com>
Link: https://lore.kernel.org/r/9dcd5fda7a3819e896d9eee4156e7c46c9a64595.1744452787.git.dan.carpenter@linaro.org


Signed-off-by: default avatarBartosz Golaszewski <brgl@bgdev.pl>
parent 2e8636ca

drivers/gpio/gpio-aggregator.c

+1 −0
Original line number Diff line number Diff line
@@ -1128,6 +1128,7 @@ static int gpio_aggregator_parse(struct gpio_aggregator *aggr) 


	if (!n) {

		pr_err("No GPIOs specified\n");

		error = -EINVAL;

		goto err;

	}