From: Jonathan Cameron <jic23@kernel.org>
To: linux-iio@vger.kernel.org
Cc: Marius Cristea <marius.cristea@microchip.com>,
Mihail Chindris <mihail.chindris@analog.com>,
Marcelo Schmitt <marcelo.schmitt1@gmail.com>,
Kim Seer Paller <kimseer.paller@analog.com>,
Dumitru Ceclan <mitrutzceclan@gmail.com>,
Cosmin Tanislav <demonsingur@gmail.com>,
Nuno Sa <nuno.sa@analog.com>,
Linus Walleij <linus.walleij@linaro.org>,
Jonathan Cameron <Jonathan.Cameron@huawei.com>
Subject: [PATCH 0/8] IIO: More use of device_for_each_child_node_scoped() and __free()
Date: Sat, 30 Mar 2024 19:08:41 +0000 [thread overview]
Message-ID: <20240330190849.1321065-1-jic23@kernel.org> (raw)
From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Mixture of new code that entered in parallel with the original conversion
set to use this new scoped cleanup and cases I missed. Note the
relevant code didn't make the 6.9 merge window but is queued up for
6.10 in the togreg branch of iio.git and linux-next.
Includes allowing easier building of the ab8500-gpadc driver to enable
testing the changes. Seems a sensible change to make in general.
This series is making us of the automated cleanup introduced in
linux/cleanup.h and new device_for_each_child_node_scoped() to avoid
the need to remember to call fwnode_handle_put() in early exits from
loops over the child nodes. This can also be used for simple scope
based cleanup as seen in the ad3552 patch. In general this makes it
easier to avoid a common class of bug.
Jonathan Cameron (8):
iio: adc: ab8500-gpadc: Allow COMPILE_TEST builds
iio: adc: ab8500-gpadc: Fix kernel-doc parameter names.
iio: adc: ab8500-gpadc: Use device_for_each_child_node_scoped() to
simplify erorr paths.
iio: adc: ad4130: Use device_for_each_child_node_scoped() to simplify
error paths.
iio: adc: ad7173: Use device_for_each_child_node_scoped() to simplify
error paths.
iio: frequency: admfm2000: Use device_for_each_child_node_scoped() to
simplify error paths.
iio: dac: ad3552: Use __free(fwnode_handle) to simplify error
handling.
iio: adc: pac1934: Use device_for_each_available_child_node_scoped()
to simplify error handling.
drivers/iio/adc/Kconfig | 2 +-
drivers/iio/adc/ab8500-gpadc.c | 8 ++--
drivers/iio/adc/ad4130.c | 7 +--
drivers/iio/adc/ad7173.c | 24 +++-------
drivers/iio/adc/pac1934.c | 77 +++++++++++++------------------
drivers/iio/dac/ad3552r.c | 59 +++++++++--------------
drivers/iio/frequency/admfm2000.c | 24 +++-------
7 files changed, 73 insertions(+), 128 deletions(-)
--
2.44.0
next reply other threads:[~2024-03-30 19:09 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-30 19:08 Jonathan Cameron [this message]
2024-03-30 19:08 ` [PATCH 1/8] iio: adc: ab8500-gpadc: Allow COMPILE_TEST builds Jonathan Cameron
2024-04-04 11:36 ` Linus Walleij
2024-04-05 10:36 ` Jonathan Cameron
2024-04-06 10:27 ` Jonathan Cameron
2024-04-08 9:16 ` Linus Walleij
2024-04-13 9:57 ` Jonathan Cameron
2024-04-15 7:06 ` Linus Walleij
2024-03-30 19:08 ` [PATCH 2/8] iio: adc: ab8500-gpadc: Fix kernel-doc parameter names Jonathan Cameron
2024-04-04 11:36 ` Linus Walleij
2024-03-30 19:08 ` [PATCH 3/8] iio: adc: ab8500-gpadc: Use device_for_each_child_node_scoped() to simplify erorr paths Jonathan Cameron
2024-04-04 11:37 ` Linus Walleij
2024-03-30 19:08 ` [PATCH 4/8] iio: adc: ad4130: Use device_for_each_child_node_scoped() to simplify error paths Jonathan Cameron
2024-03-30 19:08 ` [PATCH 5/8] iio: adc: ad7173: " Jonathan Cameron
2024-03-30 19:08 ` [PATCH 6/8] iio: frequency: admfm2000: " Jonathan Cameron
2024-03-30 19:08 ` [PATCH 7/8] iio: dac: ad3552: Use __free(fwnode_handle) to simplify error handling Jonathan Cameron
2024-03-30 19:08 ` [PATCH 8/8] iio: adc: pac1934: Use device_for_each_available_child_node_scoped() " Jonathan Cameron
2024-04-04 9:09 ` [PATCH 0/8] IIO: More use of device_for_each_child_node_scoped() and __free() Nuno Sá
2024-04-06 10:29 ` Jonathan Cameron
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240330190849.1321065-1-jic23@kernel.org \
--to=jic23@kernel.org \
--cc=Jonathan.Cameron@huawei.com \
--cc=demonsingur@gmail.com \
--cc=kimseer.paller@analog.com \
--cc=linus.walleij@linaro.org \
--cc=linux-iio@vger.kernel.org \
--cc=marcelo.schmitt1@gmail.com \
--cc=marius.cristea@microchip.com \
--cc=mihail.chindris@analog.com \
--cc=mitrutzceclan@gmail.com \
--cc=nuno.sa@analog.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).