mbox series

[v2,0/5] iio: adc: use devm_regulator_get_enable_read_voltage round 1

Message ID 20240612-iio-adc-ref-supply-refactor-v2-0-fa622e7354e9@baylibre.com (mailing list archive)
Headers show
Series iio: adc: use devm_regulator_get_enable_read_voltage round 1 | expand

Message

David Lechner June 12, 2024, 9:03 p.m. UTC 
Following up from [1], this is the first round of patches to convert the
ADC drivers to use devm_regulator_get_enable_read_voltage().

Some of these are trivial but some aren't so I'm breaking them up into
smaller series to spread out the review load (and my work load).

[1]: https://lore.kernel.org/linux-iio/20240429-regulator-get-enable-get-votlage-v2-0-b1f11ab766c1@baylibre.com

---
Changes in v2:
- avoid else in return value checks
- fix unintentional changes in behavior
- use macros instead of comments for internal reference voltage
- Link to v1: https://lore.kernel.org/r/20240531-iio-adc-ref-supply-refactor-v1-0-4b313c0615ad@baylibre.com

---
David Lechner (5):
      iio: adc: ad7192: use devm_regulator_get_enable_read_voltage
      iio: adc: ad7266: use devm_regulator_get_enable_read_voltage
      iio: adc: ad7292: use devm_regulator_get_enable_read_voltage
      iio: adc: ad7793: use devm_regulator_get_enable_read_voltage
      iio: adc: ad7944: use devm_regulator_get_enable_read_voltage

 drivers/iio/adc/ad7192.c | 100 +++++++++++++++++------------------------------
 drivers/iio/adc/ad7266.c |  33 +++-------------
 drivers/iio/adc/ad7292.c |  36 +++--------------
 drivers/iio/adc/ad7793.c |  24 ++----------
 drivers/iio/adc/ad7944.c |  54 ++++++-------------------
 5 files changed, 62 insertions(+), 185 deletions(-)
---
base-commit: cc1ce839526a65620778617da0b022bd88e8a139
change-id: 20240531-iio-adc-ref-supply-refactor-93f962d40c23

Comments

Nuno Sá June 14, 2024, 3:18 p.m. UTC | #1 
On Wed, 2024-06-12 at 16:03 -0500, David Lechner wrote:
> Following up from [1], this is the first round of patches to convert the
> ADC drivers to use devm_regulator_get_enable_read_voltage().
> 
> Some of these are trivial but some aren't so I'm breaking them up into
> smaller series to spread out the review load (and my work load).
> 
> [1]:
> https://lore.kernel.org/linux-iio/20240429-regulator-get-enable-get-votlage-v2-0-b1f11ab766c1@baylibre.com
> 
> ---

Just two things that look like typos... With that:

Reviewed-by: Nuno Sa <nuno.sa@analog.com>