diff options
| author | Wolfgang Wallner <[email protected]> | 2025-10-24 14:52:45 +0200 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2025-11-06 08:20:22 -0600 |
| commit | 854901ccd43a2f547d2bf090456214c6033144c5 (patch) | |
| tree | c543e941c323214d4e33197ecb44b6c610b5fbef | |
| parent | c9b43591ea5c63d09310713068d9aa7a924ab983 (diff) | |
gpio: Return -ENODEV if gpio_hog_lookup_name() is empty
If CONFIG_GPIO_HOG is not set, then gpio_hog_lookup_name() is empty,
and thus does not initialize any of its parameters. It does still
return 0 though, and so any calling function might assume that the
parameters have been initialized successfully.
Change the return value to -ENODEV in this case, as the function
would in the case when CONFIG_GPIO_HOG is set but the gpio hog
could not be found.
Signed-off-by: Wolfgang Wallner <[email protected]>
Reviewed-by: Heiko Schocher <[email protected]>
| -rw-r--r-- | drivers/gpio/gpio-uclass.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpio/gpio-uclass.c b/drivers/gpio/gpio-uclass.c index 3d9f8b32b8d..7559b8dc7e2 100644 --- a/drivers/gpio/gpio-uclass.c +++ b/drivers/gpio/gpio-uclass.c @@ -378,7 +378,7 @@ U_BOOT_DRIVER(gpio_hog) = { #else int gpio_hog_lookup_name(const char *name, struct gpio_desc **desc) { - return 0; + return -ENODEV; } #endif |