[PATCH v1 2/4] gpiolib: acpi: Simplify error handling in __acpi_find_gpio()

From: Andy Shevchenko
Date: Thu Apr 11 2024 - 14:17:34 EST


Now that we don't perform anything on the GPIO descriptor,
we may simplify the error path in newly introduced helper.
Do it so.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
---
drivers/gpio/gpiolib-acpi.c | 16 +++++-----------
1 file changed, 5 insertions(+), 11 deletions(-)

diff --git a/drivers/gpio/gpiolib-acpi.c b/drivers/gpio/gpiolib-acpi.c
index d47b22ac3ecb..fb2e14670b7a 100644
--- a/drivers/gpio/gpiolib-acpi.c
+++ b/drivers/gpio/gpiolib-acpi.c
@@ -974,20 +974,14 @@ __acpi_find_gpio(struct fwnode_handle *fwnode, const char *con_id, unsigned int
return ERR_CAST(desc);

if (!IS_ERR(desc))
- break;
- }
-
- /* Then from plain _CRS GPIOs */
- if (IS_ERR(desc)) {
- if (!adev || !acpi_can_fallback_to_crs(adev, con_id))
- return ERR_PTR(-ENOENT);
-
- desc = acpi_get_gpiod_by_index(adev, NULL, idx, info);
- if (IS_ERR(desc))
return desc;
}

- return desc;
+ /* Then from plain _CRS GPIOs */
+ if (!adev || !acpi_can_fallback_to_crs(adev, con_id))
+ return ERR_PTR(-ENOENT);
+
+ return acpi_get_gpiod_by_index(adev, NULL, idx, info);
}

struct gpio_desc *acpi_find_gpio(struct fwnode_handle *fwnode,
--
2.43.0.rc1.1336.g36b5255a03ac