All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 00/20] pinctrl: Use scope based of_node_put() cleanups
@ 2024-05-04 13:19 ` Peng Fan (OSS)
  0 siblings, 0 replies; 203+ messages in thread
From: Peng Fan @ 2024-05-04 13:12 UTC (permalink / raw)
  To: linux-aspeed

Use scope based of_node_put() to simplify code. It reduces the chance
of forgetting of_node_put(), and also simplifies error handling path.
I not able to test the changes on all the hardwares, so driver owners,
please help review when you have time.

This patchset was inspired from Dan's comments on pinctrl-scmi-imx.c,
thanks.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
Changes in v2:
- Drop aspeed changes per Andrew Jeffery
- Drop changes to code pattern that of_node_get(or other refcount
increasing) followed by of_node_put. That said, but I still have a
change for samsung pinctrl that drops several of_node_put places. If
this is not welcomed, patch 20/20 could be dropped.
- Add Fix tag for patch 1
- Add A-b for patch 4
- Drop unneeded {} in patch 8 Per Dan Carpenter
- Add a new patch 18.
- Moved patch [19,20]/20, in case people are not happy with the changes,
the two patch could be dropped when apply if no v3 patchset.
- Link to v1: https://lore.kernel.org/r/20240501-pinctrl-cleanup-v1-0-797ceca46e5c at nxp.com

---
Peng Fan (20):
      pinctrl: ti: iodelay: Use scope based of_node_put() cleanups
      pinctrl: tegra: Use scope based of_node_put() cleanups
      pinctrl: stm32: Use scope based of_node_put() cleanups
      pinctrl: starfive: Use scope based of_node_put() cleanups
      pinctrl: sprd: Use scope based of_node_put() cleanups
      pinctrl: spear: Use scope based of_node_put() cleanups
      pinctrl: renesas: Use scope based of_node_put() cleanups
      pinctrl: st: Use scope based of_node_put() cleanups
      pinctrl: rockchip: Use scope based of_node_put() cleanups
      pinctrl: equilibrium: Use scope based of_node_put() cleanups
      pinctrl: at91: Use scope based of_node_put() cleanups
      pinctrl: s32cc: Use scope based of_node_put() cleanups
      pinctrl: nomadik: Use scope based of_node_put() cleanups
      pinctrl: mediatek: Use scope based of_node_put() cleanups
      pinctrl: freescale: Use scope based of_node_put() cleanups
      pinctrl: bcm: bcm63xx: Use scope based of_node_put() cleanups
      pinctrl: pinconf-generic: Use scope based of_node_put() cleanups
      pinctrl: freescale: mxs: Fix refcount of child
      pinctrl: k210: Use scope based of_node_put() cleanups
      pinctrl: samsung: Use scope based of_node_put() cleanups

 drivers/pinctrl/bcm/pinctrl-bcm63xx.c              |  4 +--
 drivers/pinctrl/freescale/pinctrl-imx.c            | 25 ++++-----------
 drivers/pinctrl/freescale/pinctrl-imx1-core.c      | 16 +++-------
 drivers/pinctrl/freescale/pinctrl-mxs.c            | 18 ++++-------
 drivers/pinctrl/mediatek/pinctrl-mtk-common.c      |  4 +--
 drivers/pinctrl/mediatek/pinctrl-paris.c           |  4 +--
 drivers/pinctrl/nomadik/pinctrl-abx500.c           |  4 +--
 drivers/pinctrl/nomadik/pinctrl-nomadik.c          |  4 +--
 drivers/pinctrl/nxp/pinctrl-s32cc.c                | 31 ++++++------------
 drivers/pinctrl/pinconf-generic.c                  |  7 ++--
 drivers/pinctrl/pinctrl-at91-pio4.c                |  7 ++--
 drivers/pinctrl/pinctrl-at91.c                     | 14 +++-----
 drivers/pinctrl/pinctrl-equilibrium.c              | 21 +++---------
 drivers/pinctrl/pinctrl-k210.c                     |  7 ++--
 drivers/pinctrl/pinctrl-rockchip.c                 | 11 ++-----
 drivers/pinctrl/pinctrl-st.c                       | 37 +++++++---------------
 drivers/pinctrl/renesas/pinctrl-rza1.c             | 14 +++-----
 drivers/pinctrl/renesas/pinctrl-rzg2l.c            |  7 ++--
 drivers/pinctrl/renesas/pinctrl-rzn1.c             | 23 ++++----------
 drivers/pinctrl/renesas/pinctrl-rzv2m.c            |  7 ++--
 drivers/pinctrl/renesas/pinctrl.c                  |  7 ++--
 drivers/pinctrl/samsung/pinctrl-exynos.c           | 16 +++-------
 drivers/pinctrl/samsung/pinctrl-samsung.c          | 19 +++--------
 drivers/pinctrl/spear/pinctrl-spear.c              | 13 +++-----
 drivers/pinctrl/sprd/pinctrl-sprd.c                | 14 +++-----
 drivers/pinctrl/starfive/pinctrl-starfive-jh7100.c | 27 +++++++---------
 drivers/pinctrl/starfive/pinctrl-starfive-jh7110.c | 18 +++++------
 drivers/pinctrl/stm32/pinctrl-stm32.c              |  4 +--
 drivers/pinctrl/tegra/pinctrl-tegra-xusb.c         |  7 ++--
 drivers/pinctrl/tegra/pinctrl-tegra.c              |  4 +--
 drivers/pinctrl/ti/pinctrl-ti-iodelay.c            | 37 ++++++++--------------
 31 files changed, 133 insertions(+), 298 deletions(-)
---
base-commit: bb7a2467e6beef44a80a17d45ebf2931e7631083
change-id: 20240429-pinctrl-cleanup-e4d461c32648

Best regards,
-- 
Peng Fan <peng.fan@nxp.com>


^ permalink raw reply	[flat|nested] 203+ messages in thread

end of thread, other threads:[~2024-06-26 10:41 UTC | newest]

Thread overview: 203+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-05-04 13:12 [PATCH v2 00/20] pinctrl: Use scope based of_node_put() cleanups Peng Fan
2024-05-04 13:19 ` Peng Fan (OSS)
2024-05-04 13:19 ` Peng Fan (OSS)
2024-05-04 13:19 ` Peng Fan (OSS)
2024-05-04 13:12 ` [PATCH v2 01/20] pinctrl: ti: iodelay: " Peng Fan
2024-05-04 13:19   ` Peng Fan (OSS)
2024-05-04 13:19   ` Peng Fan (OSS)
2024-05-04 13:19   ` Peng Fan (OSS)
2024-05-04 13:13 ` [PATCH v2 02/20] pinctrl: tegra: " Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-30 10:46   ` Thierry Reding
2024-05-30 10:46     ` Thierry Reding
2024-05-30 10:46     ` Thierry Reding
2024-05-30 10:46     ` Thierry Reding
2024-05-04 13:13 ` [PATCH v2 03/20] pinctrl: stm32: " Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-17  6:11   ` Patrice CHOTARD
2024-05-17  6:53     ` Patrice CHOTARD
2024-05-17  6:11     ` Patrice CHOTARD
2024-05-17  6:11     ` Patrice CHOTARD
2024-05-27 12:56   ` Linus Walleij
2024-05-27 12:57     ` Linus Walleij
2024-05-27 12:56     ` Linus Walleij
2024-05-27 12:56     ` Linus Walleij
2024-05-04 13:13 ` [PATCH v2 04/20] pinctrl: starfive: " Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-27 14:25   ` Markus Elfring
2024-05-27 14:27     ` Markus Elfring
2024-05-27 14:25     ` Markus Elfring
2024-05-27 14:25     ` Markus Elfring
2024-05-30  8:41     ` Peng Fan
2024-05-30  8:42       ` Peng Fan
2024-05-30  8:41       ` Peng Fan
2024-05-30  8:41       ` Peng Fan
2024-05-30  9:40       ` [v2 " Markus Elfring
2024-05-30  9:42         ` Markus Elfring
2024-05-30  9:40         ` Markus Elfring
2024-05-30  9:40         ` Markus Elfring
2024-05-04 13:13 ` [PATCH v2 05/20] pinctrl: sprd: " Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:14 ` [PATCH v2 06/20] pinctrl: spear: " Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:14 ` [PATCH v2 07/20] pinctrl: renesas: " Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-13 11:59   ` Geert Uytterhoeven
2024-05-13 11:59     ` Geert Uytterhoeven
2024-05-13 11:59     ` Geert Uytterhoeven
2024-05-13 11:59     ` Geert Uytterhoeven
2024-05-13 11:59     ` Geert Uytterhoeven
2024-05-13 12:04     ` Peng Fan
2024-05-13 12:04       ` Peng Fan
2024-05-13 12:04       ` Peng Fan
2024-05-13 12:04       ` Peng Fan
2024-05-13 12:04       ` Peng Fan
2024-05-13 20:51     ` Linus Walleij
2024-05-13 20:51       ` Linus Walleij
2024-05-13 20:51       ` Linus Walleij
2024-05-13 20:51       ` Linus Walleij
2024-05-13 20:51       ` Linus Walleij
2024-05-14  6:35       ` Geert Uytterhoeven
2024-05-14  6:36         ` Geert Uytterhoeven
2024-05-14  6:35         ` Geert Uytterhoeven
2024-05-14  6:35         ` Geert Uytterhoeven
2024-05-14  7:33         ` Linus Walleij
2024-05-14  7:33           ` Linus Walleij
2024-05-14  7:33           ` Linus Walleij
2024-05-14  7:33           ` Linus Walleij
2024-05-14  8:06           ` Geert Uytterhoeven
2024-05-14  8:07             ` Geert Uytterhoeven
2024-05-14  8:06             ` Geert Uytterhoeven
2024-05-14  8:06             ` Geert Uytterhoeven
2024-05-30  9:02     ` Andy Shevchenko
2024-05-30  9:49       ` Andy Shevchenko
2024-05-30  9:02       ` Andy Shevchenko
2024-05-30  9:02       ` Andy Shevchenko
2024-05-30  9:21       ` Geert Uytterhoeven
2024-05-30  9:22         ` Geert Uytterhoeven
2024-05-30  9:21         ` Geert Uytterhoeven
2024-05-30  9:21         ` Geert Uytterhoeven
2024-05-04 13:14 ` [PATCH v2 08/20] pinctrl: st: " Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-17  6:44   ` Patrice CHOTARD
2024-05-17  6:46     ` Patrice CHOTARD
2024-05-17  6:44     ` Patrice CHOTARD
2024-05-17  6:44     ` Patrice CHOTARD
2024-05-27 12:57   ` Linus Walleij
2024-05-27 12:57     ` Linus Walleij
2024-05-27 12:57     ` Linus Walleij
2024-05-27 12:57     ` Linus Walleij
2024-05-30  9:04   ` Andy Shevchenko
2024-05-30  9:05     ` Andy Shevchenko
2024-05-30  9:04     ` Andy Shevchenko
2024-05-30  9:04     ` Andy Shevchenko
2024-05-04 13:15 ` [PATCH v2 09/20] pinctrl: rockchip: " Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:15 ` [PATCH v2 10/20] pinctrl: equilibrium: " Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:15 ` [PATCH v2 11/20] pinctrl: at91: " Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:15 ` [PATCH v2 12/20] pinctrl: s32cc: " Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:16 ` [PATCH v2 13/20] pinctrl: nomadik: " Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:16 ` [PATCH v2 14/20] pinctrl: mediatek: " Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:16 ` [PATCH v2 15/20] pinctrl: freescale: " Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:16 ` [PATCH v2 16/20] pinctrl: bcm: bcm63xx: " Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:17 ` [PATCH v2 17/20] pinctrl: pinconf-generic: " Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:17 ` [PATCH v2 18/20] pinctrl: freescale: mxs: Fix refcount of child Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-27 15:56   ` Markus Elfring
2024-05-27 15:58     ` Markus Elfring
2024-05-27 15:56     ` Markus Elfring
2024-05-27 15:56     ` Markus Elfring
2024-05-30  8:44     ` Peng Fan
2024-05-30  8:44       ` Peng Fan
2024-05-30  8:44       ` Peng Fan
2024-05-30  8:44       ` Peng Fan
2024-05-30  9:50       ` [v2 " Markus Elfring
2024-05-30  9:51         ` Markus Elfring
2024-05-30  9:50         ` Markus Elfring
2024-05-30  9:50         ` Markus Elfring
2024-05-30  9:54         ` Peng Fan
2024-05-30  9:54           ` Peng Fan
2024-05-30  9:54           ` Peng Fan
2024-05-30  9:54           ` Peng Fan
2024-05-04 13:17 ` [PATCH v2 19/20] pinctrl: k210: Use scope based of_node_put() cleanups Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:18 ` [PATCH v2 20/20] pinctrl: samsung: " Peng Fan
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-05-04 13:20   ` Peng Fan (OSS)
2024-06-26 10:22   ` (subset) " Krzysztof Kozlowski
2024-06-26 10:22     ` Krzysztof Kozlowski
2024-06-26 10:22     ` Krzysztof Kozlowski
2024-06-26 10:22     ` Krzysztof Kozlowski
2024-06-26 10:40     ` Linus Walleij
2024-06-26 10:40       ` Linus Walleij
2024-06-26 10:40       ` Linus Walleij
2024-06-26 10:40       ` Linus Walleij
2024-05-30 11:56 ` [PATCH v2 00/20] pinctrl: " Geert Uytterhoeven
2024-05-30 11:57   ` Geert Uytterhoeven
2024-05-30 11:56   ` Geert Uytterhoeven
2024-05-30 11:56   ` Geert Uytterhoeven
2024-05-31  3:07   ` Peng Fan
2024-05-31  3:07     ` Peng Fan
2024-05-31  3:07     ` Peng Fan
2024-05-31  3:07     ` Peng Fan
2024-05-31  7:06     ` Geert Uytterhoeven
2024-05-31  7:07       ` Geert Uytterhoeven
2024-05-31  7:06       ` Geert Uytterhoeven
2024-05-31  7:06       ` Geert Uytterhoeven
2024-06-18 11:52 ` Peng Fan
2024-06-18 11:52   ` Peng Fan
2024-06-18 11:52   ` Peng Fan
2024-06-18 11:52   ` Peng Fan
2024-06-26 10:02   ` Linus Walleij
2024-06-26 10:02     ` Linus Walleij
2024-06-26 10:02     ` Linus Walleij
2024-06-26 10:02     ` Linus Walleij
2024-06-26 10:20     ` Krzysztof Kozlowski
2024-06-26 10:21       ` Krzysztof Kozlowski
2024-06-26 10:20       ` Krzysztof Kozlowski
2024-06-26 10:20       ` Krzysztof Kozlowski

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.