gpiolib: acpi: change acpi_find_gpio() to accept firmware node
authorDmitry Torokhov <dmitry.torokhov@gmail.com>
Fri, 11 Nov 2022 22:19:04 +0000 (14:19 -0800)
committerBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Tue, 15 Nov 2022 10:19:40 +0000 (11:19 +0100)
In preparation of switching all ACPI-based GPIO lookups to go through
acpi_find_gpio() let's change it to accept device node as its argument
as we do not always have access to device structure.

Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
drivers/gpio/gpiolib-acpi.c
drivers/gpio/gpiolib-acpi.h
drivers/gpio/gpiolib.c

index a7d2358736fe761bfdd08bc30f9c8b611f80fdb2..61b311e4560ffb695a368cc3a5c8b444a0851950 100644 (file)
@@ -906,18 +906,22 @@ static bool acpi_can_fallback_to_crs(struct acpi_device *adev,
        return con_id == NULL;
 }
 
-struct gpio_desc *acpi_find_gpio(struct device *dev,
+struct gpio_desc *acpi_find_gpio(struct fwnode_handle *fwnode,
                                 const char *con_id,
                                 unsigned int idx,
                                 enum gpiod_flags *dflags,
                                 unsigned long *lookupflags)
 {
-       struct acpi_device *adev = ACPI_COMPANION(dev);
+       struct acpi_device *adev;
        struct acpi_gpio_info info;
        struct gpio_desc *desc;
        char propname[32];
        int i;
 
+       adev = to_acpi_device_node(fwnode);
+       if (!adev)
+               return ERR_PTR(-ENODEV);
+
        /* Try first from _DSD */
        for (i = 0; i < ARRAY_SIZE(gpio_suffixes); i++) {
                if (con_id) {
index 1ac6816839dbce7544843dcaa2b47e5826f041c8..9fc34830639ceaf8ab82b6d71e4c4d3ea468fc75 100644 (file)
@@ -48,7 +48,7 @@ int acpi_gpio_update_gpiod_flags(enum gpiod_flags *flags,
 int acpi_gpio_update_gpiod_lookup_flags(unsigned long *lookupflags,
                                        struct acpi_gpio_info *info);
 
-struct gpio_desc *acpi_find_gpio(struct device *dev,
+struct gpio_desc *acpi_find_gpio(struct fwnode_handle *fwnode,
                                 const char *con_id,
                                 unsigned int idx,
                                 enum gpiod_flags *dflags,
@@ -83,7 +83,7 @@ acpi_gpio_update_gpiod_lookup_flags(unsigned long *lookupflags,
 }
 
 static inline struct gpio_desc *
-acpi_find_gpio(struct device *dev, const char *con_id,
+acpi_find_gpio(struct fwnode_handle *fwnode, const char *con_id,
               unsigned int idx, enum gpiod_flags *dflags,
               unsigned long *lookupflags)
 {
index a80fc8abb03f3109fb3b905be6a3a0357dd49365..e874bb0ef685e776c225d3659496770dcfd07769 100644 (file)
@@ -4133,7 +4133,8 @@ struct gpio_desc *__must_check gpiod_get_index(struct device *dev,
                                    con_id, idx, &lookupflags);
        } else if (is_acpi_node(fwnode)) {
                dev_dbg(dev, "using ACPI for GPIO lookup\n");
-               desc = acpi_find_gpio(dev, con_id, idx, &flags, &lookupflags);
+               desc = acpi_find_gpio(fwnode,
+                                     con_id, idx, &flags, &lookupflags);
        }
 
        /*