[PATCH v4 0/6] gpio: acpi: modernize resource management using cleanup.h

Marco Scardovi posted 6 patches 3 weeks, 1 day ago
drivers/gpio/gpiolib-acpi-core.c | 57 ++++++++++++++------------------
include/linux/acpi.h             |  2 ++
include/linux/gpio/driver.h      |  2 ++
3 files changed, 28 insertions(+), 33 deletions(-)
[PATCH v4 0/6] gpio: acpi: modernize resource management using cleanup.h
Posted by Marco Scardovi 3 weeks, 1 day ago
Hi Andy,
as requested I've added the missing Signed-off-by and related commits' messages.
Let me know if it's everything alright now.

Marco Scardovi (6):
  ACPI: Move DEFINE_FREE(acpi_free) to global header
  gpiolib: Move DEFINE_FREE(free_gpio_desc) to driver header
  gpio: acpi: ignore out-of-range pins in acpi_gpiochip_alloc_event()
  gpio: acpi: use guard(mutex) for conn_lock
  gpio: acpi: use cleanup.h for automated resource deallocation
  gpio: acpi: sort header inclusion alphabetically

 drivers/gpio/gpiolib-acpi-core.c | 57 ++++++++++++++------------------
 include/linux/acpi.h             |  2 ++
 include/linux/gpio/driver.h      |  2 ++
 3 files changed, 28 insertions(+), 33 deletions(-)

-- 
2.54.0
Re: [PATCH v4 0/6] gpio: acpi: modernize resource management using cleanup.h
Posted by Andy Shevchenko 3 weeks, 1 day ago
On Fri, May 08, 2026 at 08:17:23AM +0200, Marco Scardovi wrote:
> Hi Andy,
> as requested I've added the missing Signed-off-by and related commits' messages.
> Let me know if it's everything alright now.

Note, cover letter is not just for a simple comment like the above, it's mainly
for the overview of the series as a whole, what it doing and why. No need to
resend a series, just try to write down one in a reply to this message.

-- 
With Best Regards,
Andy Shevchenko
Re: [PATCH v4 0/6] gpio: acpi: modernize resource management using cleanup.h
Posted by Andy Shevchenko 3 weeks, 1 day ago
On Fri, May 08, 2026 at 10:21:11AM +0300, Andy Shevchenko wrote:
> On Fri, May 08, 2026 at 08:17:23AM +0200, Marco Scardovi wrote:
> > Hi Andy,
> > as requested I've added the missing Signed-off-by and related commits' messages.
> > Let me know if it's everything alright now.
> 
> Note, cover letter is not just for a simple comment like the above, it's mainly
> for the overview of the series as a whole, what it doing and why. No need to
> resend a series, just try to write down one in a reply to this message.

Also this is v4 of the series, you need to provide a changelog for each delta
between two sequential versions (here we expect to see a v1->v2, v2->v3, and
v3->v4).

-- 
With Best Regards,
Andy Shevchenko