[PATCH v3 0/4] gpio: sim, virtuser: minor usability enhancements

Koichiro Den posted 4 patches 1 year, 1 month ago
drivers/gpio/gpio-sim.c      | 48 ++++++++++++++++---
drivers/gpio/gpio-virtuser.c | 91 +++++++++++++++++++++++++++---------
2 files changed, 110 insertions(+), 29 deletions(-)
[PATCH v3 0/4] gpio: sim, virtuser: minor usability enhancements
Posted by Koichiro Den 1 year, 1 month ago
This is just a minor usability enhancement proposal and some bug fixes for
gpio-virtuser and gpio-sim.

Koichiro Den (4):
  gpio: virtuser: fix missing lookup table cleanups
  gpio: virtuser: fix handling of multiple conn_ids in lookup table
  gpio: virtuser: lock up configfs that an instantiated device depends on
  gpio: sim: lock up configfs that an instantiated device depends on

 drivers/gpio/gpio-sim.c      | 48 ++++++++++++++++---
 drivers/gpio/gpio-virtuser.c | 91 +++++++++++++++++++++++++++---------
 2 files changed, 110 insertions(+), 29 deletions(-)

---
v2->v3:
- Incorporated feedback from Bartosz:
  * introduced gpio_virtuser_remove_lookup_table() as the counterpart to
    the existing gpio_virtuser_make_lookup_table()
- Fixed a memory leak that was spotted during the revision.

v1->v2:
- Incorporated feedback from Bartosz:
  * added more gpiod_remove_lookup_table() calls where appropriate.
  * assigned subsys at declaration.
- Added missing kfree(dev->lookup_table).
- Fixed typos in commit messages.

v2: https://lore.kernel.org/all/20250103030402.81954-1-koichiro.den@canonical.com/
v1: https://lore.kernel.org/all/20241224060819.1492472-1-koichiro.den@canonical.com/

-- 
2.43.0
Re: [PATCH v3 0/4] gpio: sim, virtuser: minor usability enhancements
Posted by Bartosz Golaszewski 1 year, 1 month ago
From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>


On Fri, 03 Jan 2025 23:18:25 +0900, Koichiro Den wrote:
> This is just a minor usability enhancement proposal and some bug fixes for
> gpio-virtuser and gpio-sim.
> 
> Koichiro Den (4):
>   gpio: virtuser: fix missing lookup table cleanups
>   gpio: virtuser: fix handling of multiple conn_ids in lookup table
>   gpio: virtuser: lock up configfs that an instantiated device depends on
>   gpio: sim: lock up configfs that an instantiated device depends on
> 
> [...]

Applied, thanks!

[1/4] gpio: virtuser: fix missing lookup table cleanups
      commit: a619cba8c69c434258ff4101d463322cd63e1bdc
[2/4] gpio: virtuser: fix handling of multiple conn_ids in lookup table
      commit: 656cc2e892f128b03ea9ef19bd11d70f71d5472b
[3/4] gpio: virtuser: lock up configfs that an instantiated device depends on
      commit: c7c434c1dba955005f5161dae73f09c0a922cfa7
[4/4] gpio: sim: lock up configfs that an instantiated device depends on
      commit: 8bd76b3d3f3af7ac2898b6a27ad90c444fec418f

Best regards,
-- 
Bartosz Golaszewski <bartosz.golaszewski@linaro.org>