Re: [PATCH v2] staging: iio: ad9832: Use devm_regulator_get_enable()

Jonathan Cameron
2025-03-08T14:46:13Z

On Thu, 6 Mar 2025 05:30:40 +0530
Saalim Quadri <danascape@gmail.com> wrote:

> The regulators are only enabled at probe(),
> hence replace the boilerplate code by making use of
> devm_regulator_get_enable() helper.
I rewrapped the description to 75 chars and applied.

Thanks,

Jonathan

>
> Signed-off-by: Saalim Quadri <danascape@gmail.com>
> ---
> V1 -> V2: Fixed indentations and commit description
>
> drivers/staging/iio/frequency/ad9832.c | 37 +++-----------------------
> 1 file changed, 4 insertions(+), 33 deletions(-)
>
> diff --git a/drivers/staging/iio/frequency/ad9832.c b/drivers/staging/iio/frequency/ad9832.c
> index 140ee4f9c137..db42810c7664 100644
> --- a/drivers/staging/iio/frequency/ad9832.c
> +++ b/drivers/staging/iio/frequency/ad9832.c
> @@ -74,8 +74,6 @@
> /**
> * struct ad9832_state - driver instance specific data
> * @spi: spi_device
> - * @avdd: supply regulator for the analog section
> - * @dvdd: supply regulator for the digital section
> * @mclk: external master clock
> * @ctrl_fp: cached frequency/phase control word
> * @ctrl_ss: cached sync/selsrc control word
> @@ -94,8 +92,6 @@
>
> struct ad9832_state {
> struct spi_device *spi;
> - struct regulator *avdd;
> - struct regulator *dvdd;
> struct clk *mclk;
> unsigned short ctrl_fp;
> unsigned short ctrl_ss;
> @@ -297,11 +293,6 @@ static const struct iio_info ad9832_info = {
> .attrs = &ad9832_attribute_group,
> };
>
> -static void ad9832_reg_disable(void *reg)
> -{
> - regulator_disable(reg);
> -}
> -
> static int ad9832_probe(struct spi_device *spi)
> {
> struct ad9832_platform_data *pdata = dev_get_platdata(&spi->dev);
> @@ -320,33 +311,13 @@ static int ad9832_probe(struct spi_device *spi)
>
> st = iio_priv(indio_dev);
>
> - st->avdd = devm_regulator_get(&spi->dev, "avdd");
> - if (IS_ERR(st->avdd))
> - return PTR_ERR(st->avdd);
> -
> - ret = regulator_enable(st->avdd);
> - if (ret) {
> - dev_err(&spi->dev, "Failed to enable specified AVDD supply\n");
> - return ret;
> - }
> -
> - ret = devm_add_action_or_reset(&spi->dev, ad9832_reg_disable, st->avdd);
> + ret = devm_regulator_get_enable(&spi->dev, "avdd");
> if (ret)
> - return ret;
> -
> - st->dvdd = devm_regulator_get(&spi->dev, "dvdd");
> - if (IS_ERR(st->dvdd))
> - return PTR_ERR(st->dvdd);
> + return dev_err_probe(&spi->dev, ret, "failed to enable specified AVDD voltage\n");
>
> - ret = regulator_enable(st->dvdd);
> - if (ret) {
> - dev_err(&spi->dev, "Failed to enable specified DVDD supply\n");
> - return ret;
> - }
> -
> - ret = devm_add_action_or_reset(&spi->dev, ad9832_reg_disable, st->dvdd);
> + ret = devm_regulator_get_enable(&spi->dev, "dvdd");
> if (ret)
> - return ret;
> + return dev_err_probe(&spi->dev, ret, "Failed to enable specified DVDD supply\n");
>
> st->mclk = devm_clk_get_enabled(&spi->dev, "mclk");
> if (IS_ERR(st->mclk))