Skip to content

gpiolib: Avoid modifying GPIO chip fwnode

Andy Shevchenko linux-acpi linux-gpio lkml 2 versions Applied

Status from this message: Applied, thanks!

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest3 patches 4 repliesApplied
  2. v13 patches 3 repliesSuperseded

Changes in v2 from the cover letter

- replaced open coded device_match_acpi_handle()
- corrected commit message due to above change
- added tag (Benjamin)

Patches in v2 Applied

#Subject and filesSizeRepliesApplied
1/3gpiolib: of: Don't use GPIO chip fwnode in of_gpiochip_*()drivers/gpio/gpiolib-of.c+3 -30 replies
2/3gpiolib: acpi: Don't use GPIO chip fwnode in acpi_gpiochip_find()drivers/gpio/gpiolib-acpi.c+1 -12 replies
3/3gpiolib: Do not alter GPIO chip fwnode memberdrivers/gpio/gpiolib.c+8 -80 replies

Reviews trailers per patch

None of the 3 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchBenjamin TissoiresLinus Walleij
SeriesR-b
1/3T-b
2/3T-b
3/3T-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [PATCH v2 0/3] gpiolib: Avoid modifying GPIO chip fwnode

Ideally the GPIO chip data structure has to be constant. In real life it's not true, but we can make it closer to that. Hence the series.

Bart, the idea is that this series has to land immediately after v6.5-rc1 is out so we will have longer time to fix downsides and regressions found, if any.

Benjamin, thank you for testing!

Changelog v2:
- replaced open coded device_match_acpi_handle()
- corrected commit message due to above change
- added tag (Benjamin)

Andy Shevchenko (3):
gpiolib: of: Don't use GPIO chip fwnode in of_gpiochip_*()
gpiolib: acpi: Don't use GPIO chip fwnode in acpi_gpiochip_find()
gpiolib: Do not alter GPIO chip fwnode member

 drivers/gpio/gpiolib-acpi.c |  2 +-
 drivers/gpio/gpiolib-of.c   |  6 +++---
 drivers/gpio/gpiolib.c      | 16 ++++++++--------
 3 files changed, 12 insertions(+), 12 deletions(-)
Signature · 1 line
2.40.0.1.gaa8946217a0b