From: Marcus Folkesson <marcus.folkesson@gmail.com>
To: David Lechner <dlechner@baylibre.com>
Cc: Jonathan Cameron <jic23@kernel.org>,
Marius Cristea <marius.cristea@microchip.com>,
Kent Gustavsson <kent@minoris.se>,
Liam Girdwood <lgirdwood@gmail.com>,
Mark Brown <broonie@kernel.org>,
linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 6/6] iio: adc: mcp3911: use devm_regulator_get_enable_read_voltage()
Date: Sat, 13 Jul 2024 07:48:08 +0200 [thread overview]
Message-ID: <ZpIVGIOkjiRSHq_z@gmail.com> (raw)
In-Reply-To: <20240712-iio-regulator-refactor-round-3-v1-6-835017bae43d@baylibre.com>
[-- Attachment #1: Type: text/plain, Size: 552 bytes --]
On Fri, Jul 12, 2024 at 11:03:57AM -0500, David Lechner wrote:
> This makes use of the new devm_regulator_get_enable_read_voltage()
> helper function to reduce boilerplate code in the MCP3911 ADC driver.
>
> The error message is slightly changed since there are fewer error
> return paths.
>
> An extra parameter is added to the config callback to avoid adding
> state that is not used outside of the probe() function.
>
> Signed-off-by: David Lechner <dlechner@baylibre.com>
Reviewed-by: Marcus Folkesson <marcus.folkesson@gmail.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2024-07-13 5:48 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-12 16:03 [PATCH 0/6] iio: use devm_regulator_get_enable_read_voltage round 3 David Lechner
2024-07-12 16:03 ` [PATCH 1/6] iio: dac: mcp4728: rename err to ret in probe function David Lechner
2024-07-12 16:03 ` [PATCH 2/6] iio: dac: mcp4728: use devm_regulator_get_enable_read_voltage() David Lechner
2024-07-12 16:03 ` [PATCH 3/6] iio: dac: mcp4922: " David Lechner
2024-07-12 16:03 ` [PATCH 4/6] iio: dac: mcp4922: drop remove() callback David Lechner
2024-07-12 16:03 ` [PATCH 5/6] iio: adc: mcp3564: use devm_regulator_get_enable_read_voltage() David Lechner
2024-07-17 13:02 ` Marius.Cristea
2024-07-12 16:03 ` [PATCH 6/6] iio: adc: mcp3911: " David Lechner
2024-07-13 5:48 ` Marcus Folkesson [this message]
2024-07-20 12:50 ` [PATCH 0/6] iio: use devm_regulator_get_enable_read_voltage round 3 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=ZpIVGIOkjiRSHq_z@gmail.com \
--to=marcus.folkesson@gmail.com \
--cc=broonie@kernel.org \
--cc=dlechner@baylibre.com \
--cc=jic23@kernel.org \
--cc=kent@minoris.se \
--cc=lgirdwood@gmail.com \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=marius.cristea@microchip.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 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.