Series comparison

-[PATCH v1 0/5] gpiolib: acpi: Refactor to shrink the code by ~8%
+[PATCH v2 0/6] gpiolib: acpi: Refactor to shrink the code by ~8%
 A simple refactoring of the GPIO ACPI library parts to get an impressive
 ~8% code shrink on x86_64 and ~2% on x86_32. Also reduces a C code a bit.
-Andy Shevchenko (5):
+add/remove: 0/2 grow/shrink: 0/5 up/down: 0/-1221 (-1221)
 Function                                     old     new   delta
 acpi_gpio_property_lookup                    425     414     -11
 acpi_find_gpio.__UNIQUE_ID_ddebug478          56       -     -56
 acpi_dev_gpio_irq_wake_get_by.__UNIQUE_ID_ddebug480      56       -     -56
 acpi_find_gpio                               354     216    -138
 acpi_get_gpiod_by_index                      462     307    -155
 __acpi_find_gpio                             877     638    -239
 acpi_dev_gpio_irq_wake_get_by                695     129    -566
 Total: Before=15375, After=14154, chg -7.94%
 In v2:
 - renamed par to params (Mika, Bart)
 Andy Shevchenko (6):
   gpiolib: acpi: Improve struct acpi_gpio_info memory footprint
   gpiolib: acpi: Remove index parameter from acpi_gpio_property_lookup()
   gpiolib: acpi: Reduce memory footprint for struct acpi_gpio_params
+  gpiolib: acpi: Rename par to params for better readability
   gpiolib: acpi: Reuse struct acpi_gpio_params in struct
     acpi_gpio_lookup
   gpiolib: acpi: Deduplicate some code in __acpi_find_gpio()
- drivers/gpio/gpiolib-acpi.c   | 138 +++++++++++++++++-----------------
+ drivers/gpio/gpiolib-acpi.c   | 146 +++++++++++++++++-----------------
  include/linux/gpio/consumer.h |   2 +-
-files changed, 68 insertions(+), 72 deletions(-)
+files changed, 72 insertions(+), 76 deletions(-)
 --
 .47.2

-[PATCH v1 1/5] gpiolib: acpi: Improve struct acpi_gpio_info memory footprint
+[PATCH v2 1/6] gpiolib: acpi: Improve struct acpi_gpio_info memory footprint
 The struct acpi_gpio_info has two boolean members that are located
 not close to each other making two gaps due to alignment requirements.
 Group them to improve memory footprint.
 `pahole` difference before and after (on 32-bit):
 -       /* size: 36, cachelines: 1, members: 9 */
 -       /* sum members: 30, holes: 2, sum holes: 6 */
 +       /* size: 32, cachelines: 1, members: 9 */
 +       /* sum members: 30, holes: 1, sum holes: 2 */
 Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
 ---
  drivers/gpio/gpiolib-acpi.c | 4 ++--
 file changed, 2 insertions(+), 2 deletions(-)
 diff --git a/drivers/gpio/gpiolib-acpi.c b/drivers/gpio/gpiolib-acpi.c
 index XXXXXXX..XXXXXXX 100644
 --- a/drivers/gpio/gpiolib-acpi.c
 +++ b/drivers/gpio/gpiolib-acpi.c
 @@ -XXX,XX +XXX,XX @@ struct acpi_gpio_chip {
   * @adev: reference to ACPI device which consumes GPIO resource
   * @flags: GPIO initialization flags
   * @gpioint: if %true this GPIO is of type GpioInt otherwise type is GpioIo
 + * @wake_capable: wake capability as provided by ACPI
   * @pin_config: pin bias as provided by ACPI
   * @polarity: interrupt polarity as provided by ACPI
   * @triggering: triggering type as provided by ACPI
 - * @wake_capable: wake capability as provided by ACPI
   * @debounce: debounce timeout as provided by ACPI
   * @quirks: Linux specific quirks as provided by struct acpi_gpio_mapping
   */
 @@ -XXX,XX +XXX,XX @@ struct acpi_gpio_info {
      struct acpi_device *adev;
      enum gpiod_flags flags;
      bool gpioint;
 +    bool wake_capable;
      int pin_config;
      int polarity;
      int triggering;
 -    bool wake_capable;
      unsigned int debounce;
      unsigned int quirks;
  };
 --
 .47.2

-[PATCH v1 2/5] gpiolib: acpi: Remove index parameter from acpi_gpio_property_lookup()
+[PATCH v2 2/6] gpiolib: acpi: Remove index parameter from acpi_gpio_property_lookup()
 In all cases the supplied index is the same as the passed one in
 the struct acpi_gpio_lookup. Remove index parameter and reuse one
 from the struct acpi_gpio_lookup instead.
+add/remove: 0/0 grow/shrink: 0/2 up/down: 0/-14 (-14)
+Function                                     old     new   delta
+acpi_get_gpiod_by_index                      462     456      -6
+acpi_gpio_property_lookup                    425     417      -8
+Total: Before=15375, After=15361, chg -0.09%
 Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
 ---
  drivers/gpio/gpiolib-acpi.c | 13 ++++++-------
 file changed, 6 insertions(+), 7 deletions(-)
 ...

-[PATCH v1 3/5] gpiolib: acpi: Reduce memory footprint for struct acpi_gpio_params
+[PATCH v2 3/6] gpiolib: acpi: Reduce memory footprint for struct acpi_gpio_params
 The line_index member in the struct acpi_gpio_params replicates
 what is covered in the ACPI GpioIo() or GpioInt() resource.
 The value there is limited to 16-bit one, so we don't really need
 to have a full 32-bit storage for it. Together with followed
 boolean the structure will be smaller.
+add/remove: 0/0 grow/shrink: 0/1 up/down: 0/-3 (-3)
+Function                                     old     new   delta
+acpi_gpio_property_lookup                    417     414      -3
+Total: Before=15361, After=15358, chg -0.02%
 `pahole` difference before and after:
 -    /* size: 12, cachelines: 1, members: 3 */
 -    /* padding: 3 */
 ...

-New patch
+[PATCH v2 4/6] gpiolib: acpi: Rename par to params for better readability
+Rename par to params for better readability in acpi_get_driver_gpio_data().
+No functional changes intended.
+Requested-by: Bartosz Golaszewski <brgl@bgdev.pl>
+Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
+---
+ drivers/gpio/gpiolib-acpi.c | 8 ++++----
+file changed, 4 insertions(+), 4 deletions(-)
+diff --git a/drivers/gpio/gpiolib-acpi.c b/drivers/gpio/gpiolib-acpi.c
+index XXXXXXX..XXXXXXX 100644
+--- a/drivers/gpio/gpiolib-acpi.c
++++ b/drivers/gpio/gpiolib-acpi.c
+@@ -XXX,XX +XXX,XX @@ static bool acpi_get_driver_gpio_data(struct acpi_device *adev,
+     for (gm = adev->driver_gpios; gm->name; gm++)
+         if (!strcmp(name, gm->name) && gm->data && index < gm->size) {
+-            const struct acpi_gpio_params *par = gm->data + index;
++            const struct acpi_gpio_params *params = gm->data + index;
+             args->fwnode = acpi_fwnode_handle(adev);
+-            args->args[0] = par->crs_entry_index;
+-            args->args[1] = par->line_index;
+-            args->args[2] = par->active_low;
++            args->args[0] = params->crs_entry_index;
++            args->args[1] = params->line_index;
++            args->args[2] = params->active_low;
+             args->nargs = 3;
+             *quirks = gm->quirks;
+--
+.47.2

-[PATCH v1 4/5] gpiolib: acpi: Reuse struct acpi_gpio_params in struct acpi_gpio_lookup
+[PATCH v2 5/6] gpiolib: acpi: Reuse struct acpi_gpio_params in struct acpi_gpio_lookup
 Some of the contents of struct acpi_gpio_lookup repeats what we have
 in the struct acpi_gpio_params. Reuse the latter in the former.
 Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
 ---
- drivers/gpio/gpiolib-acpi.c | 34 ++++++++++++++++++----------------
+ drivers/gpio/gpiolib-acpi.c | 36 +++++++++++++++++++-----------------
-file changed, 18 insertions(+), 16 deletions(-)
+file changed, 19 insertions(+), 17 deletions(-)
 diff --git a/drivers/gpio/gpiolib-acpi.c b/drivers/gpio/gpiolib-acpi.c
 index XXXXXXX..XXXXXXX 100644
 --- a/drivers/gpio/gpiolib-acpi.c
 +++ b/drivers/gpio/gpiolib-acpi.c
 ...
  struct acpi_gpio_lookup {
      struct acpi_gpio_info info;
 -    int index;
 -    u16 pin_index;
 -    bool active_low;
-+    struct acpi_gpio_params par;
++    struct acpi_gpio_params params;
      struct gpio_desc *desc;
      int n;
  };
 @@ -XXX,XX +XXX,XX @@ struct acpi_gpio_lookup {
  static int acpi_populate_gpio_lookup(struct acpi_resource *ares, void *data)
  {
      struct acpi_gpio_lookup *lookup = data;
-+    struct acpi_gpio_params *par = &lookup->par;
++    struct acpi_gpio_params *params = &lookup->params;
      if (ares->type != ACPI_RESOURCE_TYPE_GPIO)
          return 1;
 @@ -XXX,XX +XXX,XX @@ static int acpi_populate_gpio_lookup(struct acpi_resource *ares, void *data)
          u16 pin_index;
          if (lookup->info.quirks & ACPI_GPIO_QUIRK_ONLY_GPIOIO && gpioint)
 -            lookup->index++;
-+            par->crs_entry_index++;
++            params->crs_entry_index++;
 -        if (lookup->n++ != lookup->index)
-+        if (lookup->n++ != par->crs_entry_index)
++        if (lookup->n++ != params->crs_entry_index)
              return 1;
 -        pin_index = lookup->pin_index;
-+        pin_index = par->line_index;
++        pin_index = params->line_index;
          if (pin_index >= agpio->pin_table_length)
              return 1;
 @@ -XXX,XX +XXX,XX @@ static int acpi_populate_gpio_lookup(struct acpi_resource *ares, void *data)
              lookup->info.polarity = agpio->polarity;
              lookup->info.triggering = agpio->triggering;
          } else {
 -            lookup->info.polarity = lookup->active_low;
-+            lookup->info.polarity = par->active_low;
++            lookup->info.polarity = params->active_low;
          }
          lookup->info.flags = acpi_gpio_to_gpiod_flags(agpio, lookup->info.polarity);
 @@ -XXX,XX +XXX,XX @@ static int acpi_gpio_property_lookup(struct fwnode_handle *fwnode, const char *p
                       struct acpi_gpio_lookup *lookup)
  {
      struct fwnode_reference_args args;
 -    unsigned int index = lookup->index;
-+    struct acpi_gpio_params *par = &lookup->par;
++    struct acpi_gpio_params *params = &lookup->params;
-+    unsigned int index = par->crs_entry_index;
++    unsigned int index = params->crs_entry_index;
      unsigned int quirks = 0;
      int ret;
 @@ -XXX,XX +XXX,XX @@ static int acpi_gpio_property_lookup(struct fwnode_handle *fwnode, const char *p
      if (args.nargs != 3)
          return -EPROTO;
 -    lookup->index = args.args[0];
 -    lookup->pin_index = args.args[1];
 -    lookup->active_low = !!args.args[2];
-+    par->crs_entry_index = args.args[0];
++    params->crs_entry_index = args.args[0];
-+    par->line_index = args.args[1];
++    params->line_index = args.args[1];
-+    par->active_low = !!args.args[2];
++    params->active_low = !!args.args[2];
      lookup->info.adev = to_acpi_device_node(args.fwnode);
      lookup->info.quirks = quirks;
 @@ -XXX,XX +XXX,XX @@ static struct gpio_desc *acpi_get_gpiod_by_index(struct acpi_device *adev,
                           struct acpi_gpio_info *info)
  {
      struct acpi_gpio_lookup lookup;
-+    struct acpi_gpio_params *par = &lookup.par;
++    struct acpi_gpio_params *params = &lookup.params;
      int ret;
      memset(&lookup, 0, sizeof(lookup));
 -    lookup.index = index;
-+    par->crs_entry_index = index;
++    params->crs_entry_index = index;
      if (propname) {
          dev_dbg(&adev->dev, "GPIO: looking up %s\n", propname);
 @@ -XXX,XX +XXX,XX @@ static struct gpio_desc *acpi_get_gpiod_by_index(struct acpi_device *adev,
          if (ret)
 ...
 -        dev_dbg(&adev->dev, "GPIO: _DSD returned %s %d %u %u\n",
 -            dev_name(&lookup.info.adev->dev), lookup.index,
 -            lookup.pin_index, lookup.active_low);
 +        dev_dbg(&adev->dev, "GPIO: _DSD returned %s %u %u %u\n",
 +            dev_name(&lookup.info.adev->dev),
-+            par->crs_entry_index, par->line_index, par->active_low);
++            params->crs_entry_index, params->line_index, params->active_low);
      } else {
-         dev_dbg(&adev->dev, "GPIO: looking up %d in _CRS\n", index);
+-        dev_dbg(&adev->dev, "GPIO: looking up %d in _CRS\n", index);
 +        dev_dbg(&adev->dev, "GPIO: looking up %u in _CRS\n", params->crs_entry_index);
          lookup.info.adev = adev;
+     }
 @@ -XXX,XX +XXX,XX @@ static struct gpio_desc *acpi_get_gpiod_from_data(struct fwnode_handle *fwnode,
                            struct acpi_gpio_info *info)
  {
      struct acpi_gpio_lookup lookup;
-+    struct acpi_gpio_params *par = &lookup.par;
++    struct acpi_gpio_params *params = &lookup.params;
      int ret;
      if (!is_acpi_data_node(fwnode))
 @@ -XXX,XX +XXX,XX @@ static struct gpio_desc *acpi_get_gpiod_from_data(struct fwnode_handle *fwnode,
          return ERR_PTR(-EINVAL);
      memset(&lookup, 0, sizeof(lookup));
 -    lookup.index = index;
-+    par->crs_entry_index = index;
++    params->crs_entry_index = index;
      ret = acpi_gpio_property_lookup(fwnode, propname, &lookup);
      if (ret)
 --
 .47.2

-[PATCH v1 5/5] gpiolib: acpi: Deduplicate some code in __acpi_find_gpio()
+[PATCH v2 6/6] gpiolib: acpi: Deduplicate some code in __acpi_find_gpio()
 ...
 acpi_dev_gpio_irq_wake_get_by                695     129    -566
 Total: Before=15358, After=14154, chg -7.84%
 Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
 ---
- drivers/gpio/gpiolib-acpi.c | 103 +++++++++++++++++-------------------
+ drivers/gpio/gpiolib-acpi.c | 101 +++++++++++++++++-------------------
-file changed, 49 insertions(+), 54 deletions(-)
+file changed, 48 insertions(+), 53 deletions(-)
 diff --git a/drivers/gpio/gpiolib-acpi.c b/drivers/gpio/gpiolib-acpi.c
 index XXXXXXX..XXXXXXX 100644
 --- a/drivers/gpio/gpiolib-acpi.c
 +++ b/drivers/gpio/gpiolib-acpi.c
 ...
 -                         struct acpi_gpio_info *info)
 +static int acpi_get_gpiod_by_index(struct acpi_device *adev, const char *propname,
 +                   struct acpi_gpio_lookup *lookup)
  {
 -    struct acpi_gpio_lookup lookup;
--    struct acpi_gpio_params *par = &lookup.par;
+-    struct acpi_gpio_params *params = &lookup.params;
 +    struct acpi_gpio_info *info = &lookup->info;
-+    struct acpi_gpio_params *par = &lookup->par;
++    struct acpi_gpio_params *params = &lookup->params;
      int ret;
 -    memset(&lookup, 0, sizeof(lookup));
--    par->crs_entry_index = index;
+-    params->crs_entry_index = index;
 -
      if (propname) {
          dev_dbg(&adev->dev, "GPIO: looking up %s\n", propname);
 -        ret = acpi_gpio_property_lookup(acpi_fwnode_handle(adev), propname, &lookup);
 ...
 +            return ret;
          dev_dbg(&adev->dev, "GPIO: _DSD returned %s %u %u %u\n",
 -            dev_name(&lookup.info.adev->dev),
 +            dev_name(&info->adev->dev),
-             par->crs_entry_index, par->line_index, par->active_low);
+             params->crs_entry_index, params->line_index, params->active_low);
      } else {
--        dev_dbg(&adev->dev, "GPIO: looking up %d in _CRS\n", index);
+         dev_dbg(&adev->dev, "GPIO: looking up %u in _CRS\n", params->crs_entry_index);
 -        lookup.info.adev = adev;
-+        dev_dbg(&adev->dev, "GPIO: looking up %u in _CRS\n", par->crs_entry_index);
 +        info->adev = adev;
      }
 -    ret = acpi_gpio_resource_lookup(&lookup, info);
 -    return ret ? ERR_PTR(ret) : lookup.desc;
 ...
 -                          struct acpi_gpio_info *info)
 +static int acpi_get_gpiod_from_data(struct fwnode_handle *fwnode, const char *propname,
 +                    struct acpi_gpio_lookup *lookup)
  {
 -    struct acpi_gpio_lookup lookup;
--    struct acpi_gpio_params *par = &lookup.par;
+-    struct acpi_gpio_params *params = &lookup.params;
      int ret;
      if (!is_acpi_data_node(fwnode))
 -        return ERR_PTR(-ENODEV);
 +        return -ENODEV;
      if (!propname)
 -        return ERR_PTR(-EINVAL);
 +        return -EINVAL;
 -    memset(&lookup, 0, sizeof(lookup));
--    par->crs_entry_index = index;
+-    params->crs_entry_index = index;
 -
 -    ret = acpi_gpio_property_lookup(fwnode, propname, &lookup);
 +    ret = acpi_gpio_property_lookup(fwnode, propname, lookup);
      if (ret)
 -        return ERR_PTR(ret);
 ...
      struct gpio_desc *desc;
      char propname[32];
 +    int ret;
 +
 +    memset(&lookup, 0, sizeof(lookup));
-+    lookup.par.crs_entry_index = idx;
++    lookup.params.crs_entry_index = idx;
      /* Try first from _DSD */
      for_each_gpio_property_name(propname, con_id) {
          if (adev)
 -            desc = acpi_get_gpiod_by_index(adev,
 ...

The struct acpi_gpio_info has two boolean members that are located
not close to each other making two gaps due to alignment requirements.
Group them to improve memory footprint.

`pahole` difference before and after (on 32-bit):

-       /* size: 36, cachelines: 1, members: 9 */
-       /* sum members: 30, holes: 2, sum holes: 6 */

+       /* size: 32, cachelines: 1, members: 9 */
+       /* sum members: 30, holes: 1, sum holes: 2 */

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/gpio/gpiolib-acpi.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpio/gpiolib-acpi.c b/drivers/gpio/gpiolib-acpi.c
index XXXXXXX..XXXXXXX 100644
--- a/drivers/gpio/gpiolib-acpi.c
+++ b/drivers/gpio/gpiolib-acpi.c
@@ -XXX,XX +XXX,XX @@ struct acpi_gpio_chip {
  * @adev: reference to ACPI device which consumes GPIO resource
  * @flags: GPIO initialization flags
  * @gpioint: if %true this GPIO is of type GpioInt otherwise type is GpioIo
+ * @wake_capable: wake capability as provided by ACPI
  * @pin_config: pin bias as provided by ACPI
  * @polarity: interrupt polarity as provided by ACPI
  * @triggering: triggering type as provided by ACPI
- * @wake_capable: wake capability as provided by ACPI
  * @debounce: debounce timeout as provided by ACPI
  * @quirks: Linux specific quirks as provided by struct acpi_gpio_mapping
  */
@@ -XXX,XX +XXX,XX @@ struct acpi_gpio_info {
 	struct acpi_device *adev;
 	enum gpiod_flags flags;
 	bool gpioint;
+	bool wake_capable;
 	int pin_config;
 	int polarity;
 	int triggering;
-	bool wake_capable;
 	unsigned int debounce;
 	unsigned int quirks;
 };
-- 
2.47.2

In all cases the supplied index is the same as the passed one in
the struct acpi_gpio_lookup. Remove index parameter and reuse one
from the struct acpi_gpio_lookup instead.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/gpio/gpiolib-acpi.c | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

The line_index member in the struct acpi_gpio_params replicates
what is covered in the ACPI GpioIo() or GpioInt() resource.
The value there is limited to 16-bit one, so we don't really need
to have a full 32-bit storage for it. Together with followed
boolean the structure will be smaller.

`pahole` difference before and after:

-	/* size: 12, cachelines: 1, members: 3 */
-	/* padding: 3 */

+	/* size: 8, cachelines: 1, members: 3 */
+	/* padding: 1 */

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 include/linux/gpio/consumer.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/linux/gpio/consumer.h b/include/linux/gpio/consumer.h
index XXXXXXX..XXXXXXX 100644
--- a/include/linux/gpio/consumer.h
+++ b/include/linux/gpio/consumer.h
@@ -XXX,XX +XXX,XX @@ struct gpio_desc *devm_fwnode_gpiod_get(struct device *dev,
 
 struct acpi_gpio_params {
 	unsigned int crs_entry_index;
-	unsigned int line_index;
+	unsigned short line_index;
 	bool active_low;
 };
 
-- 
2.47.2

Some of the contents of struct acpi_gpio_lookup repeats what we have
in the struct acpi_gpio_params. Reuse the latter in the former.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/gpio/gpiolib-acpi.c | 34 ++++++++++++++++++----------------
 1 file changed, 18 insertions(+), 16 deletions(-)

diff --git a/drivers/gpio/gpiolib-acpi.c b/drivers/gpio/gpiolib-acpi.c
index XXXXXXX..XXXXXXX 100644
--- a/drivers/gpio/gpiolib-acpi.c
+++ b/drivers/gpio/gpiolib-acpi.c
@@ -XXX,XX +XXX,XX @@ static int acpi_gpio_update_gpiod_lookup_flags(unsigned long *lookupflags,
 
 struct acpi_gpio_lookup {
 	struct acpi_gpio_info info;
-	int index;
-	u16 pin_index;
-	bool active_low;
+	struct acpi_gpio_params par;
 	struct gpio_desc *desc;
 	int n;
 };
@@ -XXX,XX +XXX,XX @@ struct acpi_gpio_lookup {
 static int acpi_populate_gpio_lookup(struct acpi_resource *ares, void *data)
 {
 	struct acpi_gpio_lookup *lookup = data;
+	struct acpi_gpio_params *par = &lookup->par;
 
 	if (ares->type != ACPI_RESOURCE_TYPE_GPIO)
 		return 1;
@@ -XXX,XX +XXX,XX @@ static int acpi_populate_gpio_lookup(struct acpi_resource *ares, void *data)
 		u16 pin_index;
 
 		if (lookup->info.quirks & ACPI_GPIO_QUIRK_ONLY_GPIOIO && gpioint)
-			lookup->index++;
+			par->crs_entry_index++;
 
-		if (lookup->n++ != lookup->index)
+		if (lookup->n++ != par->crs_entry_index)
 			return 1;
 
-		pin_index = lookup->pin_index;
+		pin_index = par->line_index;
 		if (pin_index >= agpio->pin_table_length)
 			return 1;
 
@@ -XXX,XX +XXX,XX @@ static int acpi_populate_gpio_lookup(struct acpi_resource *ares, void *data)
 			lookup->info.polarity = agpio->polarity;
 			lookup->info.triggering = agpio->triggering;
 		} else {
-			lookup->info.polarity = lookup->active_low;
+			lookup->info.polarity = par->active_low;
 		}
 
 		lookup->info.flags = acpi_gpio_to_gpiod_flags(agpio, lookup->info.polarity);
@@ -XXX,XX +XXX,XX @@ static int acpi_gpio_property_lookup(struct fwnode_handle *fwnode, const char *p
 				     struct acpi_gpio_lookup *lookup)
 {
 	struct fwnode_reference_args args;
-	unsigned int index = lookup->index;
+	struct acpi_gpio_params *par = &lookup->par;
+	unsigned int index = par->crs_entry_index;
 	unsigned int quirks = 0;
 	int ret;
 
@@ -XXX,XX +XXX,XX @@ static int acpi_gpio_property_lookup(struct fwnode_handle *fwnode, const char *p
 	if (args.nargs != 3)
 		return -EPROTO;
 
-	lookup->index = args.args[0];
-	lookup->pin_index = args.args[1];
-	lookup->active_low = !!args.args[2];
+	par->crs_entry_index = args.args[0];
+	par->line_index = args.args[1];
+	par->active_low = !!args.args[2];
 
 	lookup->info.adev = to_acpi_device_node(args.fwnode);
 	lookup->info.quirks = quirks;
@@ -XXX,XX +XXX,XX @@ static struct gpio_desc *acpi_get_gpiod_by_index(struct acpi_device *adev,
 						 struct acpi_gpio_info *info)
 {
 	struct acpi_gpio_lookup lookup;
+	struct acpi_gpio_params *par = &lookup.par;
 	int ret;
 
 	memset(&lookup, 0, sizeof(lookup));
-	lookup.index = index;
+	par->crs_entry_index = index;
 
 	if (propname) {
 		dev_dbg(&adev->dev, "GPIO: looking up %s\n", propname);
@@ -XXX,XX +XXX,XX @@ static struct gpio_desc *acpi_get_gpiod_by_index(struct acpi_device *adev,
 		if (ret)
 			return ERR_PTR(ret);
 
-		dev_dbg(&adev->dev, "GPIO: _DSD returned %s %d %u %u\n",
-			dev_name(&lookup.info.adev->dev), lookup.index,
-			lookup.pin_index, lookup.active_low);
+		dev_dbg(&adev->dev, "GPIO: _DSD returned %s %u %u %u\n",
+			dev_name(&lookup.info.adev->dev),
+			par->crs_entry_index, par->line_index, par->active_low);
 	} else {
 		dev_dbg(&adev->dev, "GPIO: looking up %d in _CRS\n", index);
 		lookup.info.adev = adev;
@@ -XXX,XX +XXX,XX @@ static struct gpio_desc *acpi_get_gpiod_from_data(struct fwnode_handle *fwnode,
 						  struct acpi_gpio_info *info)
 {
 	struct acpi_gpio_lookup lookup;
+	struct acpi_gpio_params *par = &lookup.par;
 	int ret;
 
 	if (!is_acpi_data_node(fwnode))
@@ -XXX,XX +XXX,XX @@ static struct gpio_desc *acpi_get_gpiod_from_data(struct fwnode_handle *fwnode,
 		return ERR_PTR(-EINVAL);
 
 	memset(&lookup, 0, sizeof(lookup));
-	lookup.index = index;
+	par->crs_entry_index = index;
 
 	ret = acpi_gpio_property_lookup(fwnode, propname, &lookup);
 	if (ret)
-- 
2.47.2

__acpi_find_gpio() calls two functions depending on the supplied con_id
and possibility to fallback to _CRS lookup. Those functions have the same
pieces of code that can be done only in one place. Do it so.

This gives an impressive shrink of the generated code for x86_64:

add/remove: 0/2 grow/shrink: 0/4 up/down: 0/-1204 (-1204)
Function                                     old     new   delta
acpi_find_gpio.__UNIQUE_ID_ddebug478          56       -     -56
acpi_dev_gpio_irq_wake_get_by.__UNIQUE_ID_ddebug480      56       -     -56
acpi_find_gpio                               354     216    -138
acpi_get_gpiod_by_index                      456     307    -149
__acpi_find_gpio                             877     638    -239
acpi_dev_gpio_irq_wake_get_by                695     129    -566
Total: Before=15358, After=14154, chg -7.84%

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/gpio/gpiolib-acpi.c | 103 +++++++++++++++++-------------------
 1 file changed, 49 insertions(+), 54 deletions(-)

diff --git a/drivers/gpio/gpiolib-acpi.c b/drivers/gpio/gpiolib-acpi.c
index XXXXXXX..XXXXXXX 100644
--- a/drivers/gpio/gpiolib-acpi.c
+++ b/drivers/gpio/gpiolib-acpi.c
@@ -XXX,XX +XXX,XX @@ static int acpi_populate_gpio_lookup(struct acpi_resource *ares, void *data)
 	return 1;
 }
 
-static int acpi_gpio_resource_lookup(struct acpi_gpio_lookup *lookup,
-				     struct acpi_gpio_info *info)
+static int acpi_gpio_resource_lookup(struct acpi_gpio_lookup *lookup)
 {
 	struct acpi_device *adev = lookup->info.adev;
 	struct list_head res_list;
@@ -XXX,XX +XXX,XX @@ static int acpi_gpio_resource_lookup(struct acpi_gpio_lookup *lookup,
 	if (!lookup->desc)
 		return -ENOENT;
 
-	if (info)
-		*info = lookup->info;
 	return 0;
 }
 
@@ -XXX,XX +XXX,XX @@ static int acpi_gpio_property_lookup(struct fwnode_handle *fwnode, const char *p
  * acpi_get_gpiod_by_index() - get a GPIO descriptor from device resources
  * @adev: pointer to a ACPI device to get GPIO from
  * @propname: Property name of the GPIO (optional)
- * @index: index of GpioIo/GpioInt resource (starting from %0)
- * @info: info pointer to fill in (optional)
+ * @lookup: pointer to struct acpi_gpio_lookup to fill in
  *
- * Function goes through ACPI resources for @adev and based on @index looks
+ * Function goes through ACPI resources for @adev and based on @lookup.index looks
  * up a GpioIo/GpioInt resource, translates it to the Linux GPIO descriptor,
- * and returns it. @index matches GpioIo/GpioInt resources only so if there
- * are total %3 GPIO resources, the index goes from %0 to %2.
+ * and returns it. @lookup.index matches GpioIo/GpioInt resources only so if there
+ * are total 3 GPIO resources, the index goes from 0 to 2.
  *
  * If @propname is specified the GPIO is looked using device property. In
  * that case @index is used to select the GPIO entry in the property value
  * (in case of multiple).
  *
  * Returns:
- * GPIO descriptor to use with Linux generic GPIO API.
- * If the GPIO cannot be translated or there is an error an ERR_PTR is
- * returned.
+ * 0 on success, negative errno on failure.
+ *
+ * The @lookup is filled with GPIO descriptor to use with Linux generic GPIO API.
+ * If the GPIO cannot be translated an error will be returned.
  *
  * Note: if the GPIO resource has multiple entries in the pin list, this
  * function only returns the first.
  */
-static struct gpio_desc *acpi_get_gpiod_by_index(struct acpi_device *adev,
-						 const char *propname,
-						 int index,
-						 struct acpi_gpio_info *info)
+static int acpi_get_gpiod_by_index(struct acpi_device *adev, const char *propname,
+				   struct acpi_gpio_lookup *lookup)
 {
-	struct acpi_gpio_lookup lookup;
-	struct acpi_gpio_params *par = &lookup.par;
+	struct acpi_gpio_info *info = &lookup->info;
+	struct acpi_gpio_params *par = &lookup->par;
 	int ret;
 
-	memset(&lookup, 0, sizeof(lookup));
-	par->crs_entry_index = index;
-
 	if (propname) {
 		dev_dbg(&adev->dev, "GPIO: looking up %s\n", propname);
 
-		ret = acpi_gpio_property_lookup(acpi_fwnode_handle(adev), propname, &lookup);
+		ret = acpi_gpio_property_lookup(acpi_fwnode_handle(adev), propname, lookup);
 		if (ret)
-			return ERR_PTR(ret);
+			return ret;
 
 		dev_dbg(&adev->dev, "GPIO: _DSD returned %s %u %u %u\n",
-			dev_name(&lookup.info.adev->dev),
+			dev_name(&info->adev->dev),
 			par->crs_entry_index, par->line_index, par->active_low);
 	} else {
-		dev_dbg(&adev->dev, "GPIO: looking up %d in _CRS\n", index);
-		lookup.info.adev = adev;
+		dev_dbg(&adev->dev, "GPIO: looking up %u in _CRS\n", par->crs_entry_index);
+		info->adev = adev;
 	}
 
-	ret = acpi_gpio_resource_lookup(&lookup, info);
-	return ret ? ERR_PTR(ret) : lookup.desc;
+	return acpi_gpio_resource_lookup(lookup);
 }
 
 /**
  * acpi_get_gpiod_from_data() - get a GPIO descriptor from ACPI data node
  * @fwnode: pointer to an ACPI firmware node to get the GPIO information from
  * @propname: Property name of the GPIO
- * @index: index of GpioIo/GpioInt resource (starting from %0)
- * @info: info pointer to fill in (optional)
+ * @lookup: pointer to struct acpi_gpio_lookup to fill in
  *
  * This function uses the property-based GPIO lookup to get to the GPIO
  * resource with the relevant information from a data-only ACPI firmware node
  * and uses that to obtain the GPIO descriptor to return.
  *
  * Returns:
- * GPIO descriptor to use with Linux generic GPIO API.
- * If the GPIO cannot be translated or there is an error an ERR_PTR is
- * returned.
+ * 0 on success, negative errno on failure.
+ *
+ * The @lookup is filled with GPIO descriptor to use with Linux generic GPIO API.
+ * If the GPIO cannot be translated an error will be returned.
  */
-static struct gpio_desc *acpi_get_gpiod_from_data(struct fwnode_handle *fwnode,
-						  const char *propname,
-						  int index,
-						  struct acpi_gpio_info *info)
+static int acpi_get_gpiod_from_data(struct fwnode_handle *fwnode, const char *propname,
+				    struct acpi_gpio_lookup *lookup)
 {
-	struct acpi_gpio_lookup lookup;
-	struct acpi_gpio_params *par = &lookup.par;
 	int ret;
 
 	if (!is_acpi_data_node(fwnode))
-		return ERR_PTR(-ENODEV);
+		return -ENODEV;
 
 	if (!propname)
-		return ERR_PTR(-EINVAL);
+		return -EINVAL;
 
-	memset(&lookup, 0, sizeof(lookup));
-	par->crs_entry_index = index;
-
-	ret = acpi_gpio_property_lookup(fwnode, propname, &lookup);
+	ret = acpi_gpio_property_lookup(fwnode, propname, lookup);
 	if (ret)
-		return ERR_PTR(ret);
+		return ret;
 
-	ret = acpi_gpio_resource_lookup(&lookup, info);
-	return ret ? ERR_PTR(ret) : lookup.desc;
+	return acpi_gpio_resource_lookup(lookup);
 }
 
 static bool acpi_can_fallback_to_crs(struct acpi_device *adev,
@@ -XXX,XX +XXX,XX @@ __acpi_find_gpio(struct fwnode_handle *fwnode, const char *con_id, unsigned int
 		 bool can_fallback, struct acpi_gpio_info *info)
 {
 	struct acpi_device *adev = to_acpi_device_node(fwnode);
+	struct acpi_gpio_lookup lookup;
 	struct gpio_desc *desc;
 	char propname[32];
+	int ret;
+
+	memset(&lookup, 0, sizeof(lookup));
+	lookup.par.crs_entry_index = idx;
 
 	/* Try first from _DSD */
 	for_each_gpio_property_name(propname, con_id) {
 		if (adev)
-			desc = acpi_get_gpiod_by_index(adev,
-						       propname, idx, info);
+			ret = acpi_get_gpiod_by_index(adev, propname, &lookup);
 		else
-			desc = acpi_get_gpiod_from_data(fwnode,
-							propname, idx, info);
+			ret = acpi_get_gpiod_from_data(fwnode, propname, &lookup);
+		if (ret)
+			continue;
+
+		desc = lookup.desc;
 		if (PTR_ERR(desc) == -EPROBE_DEFER)
 			return desc;
 
@@ -XXX,XX +XXX,XX @@ __acpi_find_gpio(struct fwnode_handle *fwnode, const char *con_id, unsigned int
 	}
 
 	/* Then from plain _CRS GPIOs */
-	if (can_fallback)
-		return acpi_get_gpiod_by_index(adev, NULL, idx, info);
+	if (can_fallback) {
+		ret = acpi_get_gpiod_by_index(adev, NULL, &lookup);
+		if (ret)
+			return ERR_PTR(ret);
+
+		return lookup.desc;
+	}
 
 	return ERR_PTR(-ENOENT);
 }
-- 
2.47.2

A simple refactoring of the GPIO ACPI library parts to get an impressive
~8% code shrink on x86_64 and ~2% on x86_32. Also reduces a C code a bit.

add/remove: 0/2 grow/shrink: 0/5 up/down: 0/-1221 (-1221)
Function                                     old     new   delta
acpi_gpio_property_lookup                    425     414     -11
acpi_find_gpio.__UNIQUE_ID_ddebug478          56       -     -56
acpi_dev_gpio_irq_wake_get_by.__UNIQUE_ID_ddebug480      56       -     -56
acpi_find_gpio                               354     216    -138
acpi_get_gpiod_by_index                      462     307    -155
__acpi_find_gpio                             877     638    -239
acpi_dev_gpio_irq_wake_get_by                695     129    -566
Total: Before=15375, After=14154, chg -7.94%

In v2:
- renamed par to params (Mika, Bart)

Andy Shevchenko (6):
  gpiolib: acpi: Improve struct acpi_gpio_info memory footprint
  gpiolib: acpi: Remove index parameter from acpi_gpio_property_lookup()
  gpiolib: acpi: Reduce memory footprint for struct acpi_gpio_params
  gpiolib: acpi: Rename par to params for better readability
  gpiolib: acpi: Reuse struct acpi_gpio_params in struct
    acpi_gpio_lookup
  gpiolib: acpi: Deduplicate some code in __acpi_find_gpio()

drivers/gpio/gpiolib-acpi.c   | 146 +++++++++++++++++-----------------
 include/linux/gpio/consumer.h |   2 +-
 2 files changed, 72 insertions(+), 76 deletions(-)

-- 
2.47.2

The struct acpi_gpio_info has two boolean members that are located
not close to each other making two gaps due to alignment requirements.
Group them to improve memory footprint.

`pahole` difference before and after (on 32-bit):

-       /* size: 36, cachelines: 1, members: 9 */
-       /* sum members: 30, holes: 2, sum holes: 6 */

+       /* size: 32, cachelines: 1, members: 9 */
+       /* sum members: 30, holes: 1, sum holes: 2 */

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/gpio/gpiolib-acpi.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpio/gpiolib-acpi.c b/drivers/gpio/gpiolib-acpi.c
index XXXXXXX..XXXXXXX 100644
--- a/drivers/gpio/gpiolib-acpi.c
+++ b/drivers/gpio/gpiolib-acpi.c
@@ -XXX,XX +XXX,XX @@ struct acpi_gpio_chip {
  * @adev: reference to ACPI device which consumes GPIO resource
  * @flags: GPIO initialization flags
  * @gpioint: if %true this GPIO is of type GpioInt otherwise type is GpioIo
+ * @wake_capable: wake capability as provided by ACPI
  * @pin_config: pin bias as provided by ACPI
  * @polarity: interrupt polarity as provided by ACPI
  * @triggering: triggering type as provided by ACPI
- * @wake_capable: wake capability as provided by ACPI
  * @debounce: debounce timeout as provided by ACPI
  * @quirks: Linux specific quirks as provided by struct acpi_gpio_mapping
  */
@@ -XXX,XX +XXX,XX @@ struct acpi_gpio_info {
 	struct acpi_device *adev;
 	enum gpiod_flags flags;
 	bool gpioint;
+	bool wake_capable;
 	int pin_config;
 	int polarity;
 	int triggering;
-	bool wake_capable;
 	unsigned int debounce;
 	unsigned int quirks;
 };
-- 
2.47.2

In all cases the supplied index is the same as the passed one in
the struct acpi_gpio_lookup. Remove index parameter and reuse one
from the struct acpi_gpio_lookup instead.

add/remove: 0/0 grow/shrink: 0/2 up/down: 0/-14 (-14)
Function                                     old     new   delta
acpi_get_gpiod_by_index                      462     456      -6
acpi_gpio_property_lookup                    425     417      -8
Total: Before=15375, After=15361, chg -0.09%

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/gpio/gpiolib-acpi.c | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

add/remove: 0/0 grow/shrink: 0/1 up/down: 0/-3 (-3)
Function                                     old     new   delta
acpi_gpio_property_lookup                    417     414      -3
Total: Before=15361, After=15358, chg -0.02%

`pahole` difference before and after:

-	/* size: 12, cachelines: 1, members: 3 */
-	/* padding: 3 */

+	/* size: 8, cachelines: 1, members: 3 */
+	/* padding: 1 */

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 include/linux/gpio/consumer.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Rename par to params for better readability in acpi_get_driver_gpio_data().
No functional changes intended.

Requested-by: Bartosz Golaszewski <brgl@bgdev.pl>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/gpio/gpiolib-acpi.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/gpio/gpiolib-acpi.c b/drivers/gpio/gpiolib-acpi.c
index XXXXXXX..XXXXXXX 100644
--- a/drivers/gpio/gpiolib-acpi.c
+++ b/drivers/gpio/gpiolib-acpi.c
@@ -XXX,XX +XXX,XX @@ static bool acpi_get_driver_gpio_data(struct acpi_device *adev,
 
 	for (gm = adev->driver_gpios; gm->name; gm++)
 		if (!strcmp(name, gm->name) && gm->data && index < gm->size) {
-			const struct acpi_gpio_params *par = gm->data + index;
+			const struct acpi_gpio_params *params = gm->data + index;
 
 			args->fwnode = acpi_fwnode_handle(adev);
-			args->args[0] = par->crs_entry_index;
-			args->args[1] = par->line_index;
-			args->args[2] = par->active_low;
+			args->args[0] = params->crs_entry_index;
+			args->args[1] = params->line_index;
+			args->args[2] = params->active_low;
 			args->nargs = 3;
 
 			*quirks = gm->quirks;
-- 
2.47.2

Some of the contents of struct acpi_gpio_lookup repeats what we have
in the struct acpi_gpio_params. Reuse the latter in the former.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/gpio/gpiolib-acpi.c | 36 +++++++++++++++++++-----------------
 1 file changed, 19 insertions(+), 17 deletions(-)

diff --git a/drivers/gpio/gpiolib-acpi.c b/drivers/gpio/gpiolib-acpi.c
index XXXXXXX..XXXXXXX 100644
--- a/drivers/gpio/gpiolib-acpi.c
+++ b/drivers/gpio/gpiolib-acpi.c
@@ -XXX,XX +XXX,XX @@ static int acpi_gpio_update_gpiod_lookup_flags(unsigned long *lookupflags,
 
 struct acpi_gpio_lookup {
 	struct acpi_gpio_info info;
-	int index;
-	u16 pin_index;
-	bool active_low;
+	struct acpi_gpio_params params;
 	struct gpio_desc *desc;
 	int n;
 };
@@ -XXX,XX +XXX,XX @@ struct acpi_gpio_lookup {
 static int acpi_populate_gpio_lookup(struct acpi_resource *ares, void *data)
 {
 	struct acpi_gpio_lookup *lookup = data;
+	struct acpi_gpio_params *params = &lookup->params;
 
 	if (ares->type != ACPI_RESOURCE_TYPE_GPIO)
 		return 1;
@@ -XXX,XX +XXX,XX @@ static int acpi_populate_gpio_lookup(struct acpi_resource *ares, void *data)
 		u16 pin_index;
 
 		if (lookup->info.quirks & ACPI_GPIO_QUIRK_ONLY_GPIOIO && gpioint)
-			lookup->index++;
+			params->crs_entry_index++;
 
-		if (lookup->n++ != lookup->index)
+		if (lookup->n++ != params->crs_entry_index)
 			return 1;
 
-		pin_index = lookup->pin_index;
+		pin_index = params->line_index;
 		if (pin_index >= agpio->pin_table_length)
 			return 1;
 
@@ -XXX,XX +XXX,XX @@ static int acpi_populate_gpio_lookup(struct acpi_resource *ares, void *data)
 			lookup->info.polarity = agpio->polarity;
 			lookup->info.triggering = agpio->triggering;
 		} else {
-			lookup->info.polarity = lookup->active_low;
+			lookup->info.polarity = params->active_low;
 		}
 
 		lookup->info.flags = acpi_gpio_to_gpiod_flags(agpio, lookup->info.polarity);
@@ -XXX,XX +XXX,XX @@ static int acpi_gpio_property_lookup(struct fwnode_handle *fwnode, const char *p
 				     struct acpi_gpio_lookup *lookup)
 {
 	struct fwnode_reference_args args;
-	unsigned int index = lookup->index;
+	struct acpi_gpio_params *params = &lookup->params;
+	unsigned int index = params->crs_entry_index;
 	unsigned int quirks = 0;
 	int ret;
 
@@ -XXX,XX +XXX,XX @@ static int acpi_gpio_property_lookup(struct fwnode_handle *fwnode, const char *p
 	if (args.nargs != 3)
 		return -EPROTO;
 
-	lookup->index = args.args[0];
-	lookup->pin_index = args.args[1];
-	lookup->active_low = !!args.args[2];
+	params->crs_entry_index = args.args[0];
+	params->line_index = args.args[1];
+	params->active_low = !!args.args[2];
 
 	lookup->info.adev = to_acpi_device_node(args.fwnode);
 	lookup->info.quirks = quirks;
@@ -XXX,XX +XXX,XX @@ static struct gpio_desc *acpi_get_gpiod_by_index(struct acpi_device *adev,
 						 struct acpi_gpio_info *info)
 {
 	struct acpi_gpio_lookup lookup;
+	struct acpi_gpio_params *params = &lookup.params;
 	int ret;
 
 	memset(&lookup, 0, sizeof(lookup));
-	lookup.index = index;
+	params->crs_entry_index = index;
 
 	if (propname) {
 		dev_dbg(&adev->dev, "GPIO: looking up %s\n", propname);
@@ -XXX,XX +XXX,XX @@ static struct gpio_desc *acpi_get_gpiod_by_index(struct acpi_device *adev,
 		if (ret)
 			return ERR_PTR(ret);
 
-		dev_dbg(&adev->dev, "GPIO: _DSD returned %s %d %u %u\n",
-			dev_name(&lookup.info.adev->dev), lookup.index,
-			lookup.pin_index, lookup.active_low);
+		dev_dbg(&adev->dev, "GPIO: _DSD returned %s %u %u %u\n",
+			dev_name(&lookup.info.adev->dev),
+			params->crs_entry_index, params->line_index, params->active_low);
 	} else {
-		dev_dbg(&adev->dev, "GPIO: looking up %d in _CRS\n", index);
+		dev_dbg(&adev->dev, "GPIO: looking up %u in _CRS\n", params->crs_entry_index);
 		lookup.info.adev = adev;
 	}
 
@@ -XXX,XX +XXX,XX @@ static struct gpio_desc *acpi_get_gpiod_from_data(struct fwnode_handle *fwnode,
 						  struct acpi_gpio_info *info)
 {
 	struct acpi_gpio_lookup lookup;
+	struct acpi_gpio_params *params = &lookup.params;
 	int ret;
 
 	if (!is_acpi_data_node(fwnode))
@@ -XXX,XX +XXX,XX @@ static struct gpio_desc *acpi_get_gpiod_from_data(struct fwnode_handle *fwnode,
 		return ERR_PTR(-EINVAL);
 
 	memset(&lookup, 0, sizeof(lookup));
-	lookup.index = index;
+	params->crs_entry_index = index;
 
 	ret = acpi_gpio_property_lookup(fwnode, propname, &lookup);
 	if (ret)
-- 
2.47.2

This gives an impressive shrink of the generated code for x86_64:

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/gpio/gpiolib-acpi.c | 101 +++++++++++++++++-------------------
 1 file changed, 48 insertions(+), 53 deletions(-)

diff --git a/drivers/gpio/gpiolib-acpi.c b/drivers/gpio/gpiolib-acpi.c
index XXXXXXX..XXXXXXX 100644
--- a/drivers/gpio/gpiolib-acpi.c
+++ b/drivers/gpio/gpiolib-acpi.c
@@ -XXX,XX +XXX,XX @@ static int acpi_populate_gpio_lookup(struct acpi_resource *ares, void *data)
 	return 1;
 }
 
-static int acpi_gpio_resource_lookup(struct acpi_gpio_lookup *lookup,
-				     struct acpi_gpio_info *info)
+static int acpi_gpio_resource_lookup(struct acpi_gpio_lookup *lookup)
 {
 	struct acpi_device *adev = lookup->info.adev;
 	struct list_head res_list;
@@ -XXX,XX +XXX,XX @@ static int acpi_gpio_resource_lookup(struct acpi_gpio_lookup *lookup,
 	if (!lookup->desc)
 		return -ENOENT;
 
-	if (info)
-		*info = lookup->info;
 	return 0;
 }
 
@@ -XXX,XX +XXX,XX @@ static int acpi_gpio_property_lookup(struct fwnode_handle *fwnode, const char *p
  * acpi_get_gpiod_by_index() - get a GPIO descriptor from device resources
  * @adev: pointer to a ACPI device to get GPIO from
  * @propname: Property name of the GPIO (optional)
- * @index: index of GpioIo/GpioInt resource (starting from %0)
- * @info: info pointer to fill in (optional)
+ * @lookup: pointer to struct acpi_gpio_lookup to fill in
  *
- * Function goes through ACPI resources for @adev and based on @index looks
+ * Function goes through ACPI resources for @adev and based on @lookup.index looks
  * up a GpioIo/GpioInt resource, translates it to the Linux GPIO descriptor,
- * and returns it. @index matches GpioIo/GpioInt resources only so if there
- * are total %3 GPIO resources, the index goes from %0 to %2.
+ * and returns it. @lookup.index matches GpioIo/GpioInt resources only so if there
+ * are total 3 GPIO resources, the index goes from 0 to 2.
  *
  * If @propname is specified the GPIO is looked using device property. In
  * that case @index is used to select the GPIO entry in the property value
  * (in case of multiple).
  *
  * Returns:
- * GPIO descriptor to use with Linux generic GPIO API.
- * If the GPIO cannot be translated or there is an error an ERR_PTR is
- * returned.
+ * 0 on success, negative errno on failure.
+ *
+ * The @lookup is filled with GPIO descriptor to use with Linux generic GPIO API.
+ * If the GPIO cannot be translated an error will be returned.
  *
  * Note: if the GPIO resource has multiple entries in the pin list, this
  * function only returns the first.
  */
-static struct gpio_desc *acpi_get_gpiod_by_index(struct acpi_device *adev,
-						 const char *propname,
-						 int index,
-						 struct acpi_gpio_info *info)
+static int acpi_get_gpiod_by_index(struct acpi_device *adev, const char *propname,
+				   struct acpi_gpio_lookup *lookup)
 {
-	struct acpi_gpio_lookup lookup;
-	struct acpi_gpio_params *params = &lookup.params;
+	struct acpi_gpio_info *info = &lookup->info;
+	struct acpi_gpio_params *params = &lookup->params;
 	int ret;
 
-	memset(&lookup, 0, sizeof(lookup));
-	params->crs_entry_index = index;
-
 	if (propname) {
 		dev_dbg(&adev->dev, "GPIO: looking up %s\n", propname);
 
-		ret = acpi_gpio_property_lookup(acpi_fwnode_handle(adev), propname, &lookup);
+		ret = acpi_gpio_property_lookup(acpi_fwnode_handle(adev), propname, lookup);
 		if (ret)
-			return ERR_PTR(ret);
+			return ret;
 
 		dev_dbg(&adev->dev, "GPIO: _DSD returned %s %u %u %u\n",
-			dev_name(&lookup.info.adev->dev),
+			dev_name(&info->adev->dev),
 			params->crs_entry_index, params->line_index, params->active_low);
 	} else {
 		dev_dbg(&adev->dev, "GPIO: looking up %u in _CRS\n", params->crs_entry_index);
-		lookup.info.adev = adev;
+		info->adev = adev;
 	}
 
-	ret = acpi_gpio_resource_lookup(&lookup, info);
-	return ret ? ERR_PTR(ret) : lookup.desc;
+	return acpi_gpio_resource_lookup(lookup);
 }
 
 /**
  * acpi_get_gpiod_from_data() - get a GPIO descriptor from ACPI data node
  * @fwnode: pointer to an ACPI firmware node to get the GPIO information from
  * @propname: Property name of the GPIO
- * @index: index of GpioIo/GpioInt resource (starting from %0)
- * @info: info pointer to fill in (optional)
+ * @lookup: pointer to struct acpi_gpio_lookup to fill in
  *
  * This function uses the property-based GPIO lookup to get to the GPIO
  * resource with the relevant information from a data-only ACPI firmware node
  * and uses that to obtain the GPIO descriptor to return.
  *
  * Returns:
- * GPIO descriptor to use with Linux generic GPIO API.
- * If the GPIO cannot be translated or there is an error an ERR_PTR is
- * returned.
+ * 0 on success, negative errno on failure.
+ *
+ * The @lookup is filled with GPIO descriptor to use with Linux generic GPIO API.
+ * If the GPIO cannot be translated an error will be returned.
  */
-static struct gpio_desc *acpi_get_gpiod_from_data(struct fwnode_handle *fwnode,
-						  const char *propname,
-						  int index,
-						  struct acpi_gpio_info *info)
+static int acpi_get_gpiod_from_data(struct fwnode_handle *fwnode, const char *propname,
+				    struct acpi_gpio_lookup *lookup)
 {
-	struct acpi_gpio_lookup lookup;
-	struct acpi_gpio_params *params = &lookup.params;
 	int ret;
 
 	if (!is_acpi_data_node(fwnode))
-		return ERR_PTR(-ENODEV);
+		return -ENODEV;
 
 	if (!propname)
-		return ERR_PTR(-EINVAL);
+		return -EINVAL;
 
-	memset(&lookup, 0, sizeof(lookup));
-	params->crs_entry_index = index;
-
-	ret = acpi_gpio_property_lookup(fwnode, propname, &lookup);
+	ret = acpi_gpio_property_lookup(fwnode, propname, lookup);
 	if (ret)
-		return ERR_PTR(ret);
+		return ret;
 
-	ret = acpi_gpio_resource_lookup(&lookup, info);
-	return ret ? ERR_PTR(ret) : lookup.desc;
+	return acpi_gpio_resource_lookup(lookup);
 }
 
 static bool acpi_can_fallback_to_crs(struct acpi_device *adev,
@@ -XXX,XX +XXX,XX @@ __acpi_find_gpio(struct fwnode_handle *fwnode, const char *con_id, unsigned int
 		 bool can_fallback, struct acpi_gpio_info *info)
 {
 	struct acpi_device *adev = to_acpi_device_node(fwnode);
+	struct acpi_gpio_lookup lookup;
 	struct gpio_desc *desc;
 	char propname[32];
+	int ret;
+
+	memset(&lookup, 0, sizeof(lookup));
+	lookup.params.crs_entry_index = idx;
 
 	/* Try first from _DSD */
 	for_each_gpio_property_name(propname, con_id) {
 		if (adev)
-			desc = acpi_get_gpiod_by_index(adev,
-						       propname, idx, info);
+			ret = acpi_get_gpiod_by_index(adev, propname, &lookup);
 		else
-			desc = acpi_get_gpiod_from_data(fwnode,
-							propname, idx, info);
+			ret = acpi_get_gpiod_from_data(fwnode, propname, &lookup);
+		if (ret)
+			continue;
+
+		desc = lookup.desc;
 		if (PTR_ERR(desc) == -EPROBE_DEFER)
 			return desc;
 
@@ -XXX,XX +XXX,XX @@ __acpi_find_gpio(struct fwnode_handle *fwnode, const char *con_id, unsigned int
 	}
 
 	/* Then from plain _CRS GPIOs */
-	if (can_fallback)
-		return acpi_get_gpiod_by_index(adev, NULL, idx, info);
+	if (can_fallback) {
+		ret = acpi_get_gpiod_by_index(adev, NULL, &lookup);
+		if (ret)
+			return ERR_PTR(ret);
+
+		return lookup.desc;
+	}
 
 	return ERR_PTR(-ENOENT);
 }
-- 
2.47.2