Re: [PATCH v10 3/3] iio: dac: Add AD5529R DAC driver support
Andy Shevchenko <[email protected]>
| Newsgroups | org.infradead.lists.linux-riscv,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-doc,org.kernel.vger.linux-iio,org.kernel.vger.linux-kernel,org.kernel.vger.linux-spi |
|---|---|
| Organization | Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs, Bertel Jungin Aukio 5, 02600 Espoo |
| Message-ID | <[email protected]> |
On Thu, Aug 27, 2026 at 01:17:13PM +0200, Janani Sunil wrote:
> On 8/27/26 10:23, Andy Shevchenko wrote:
> > On Thu, Aug 27, 2026 at 09:34:48AM +0200, Janani Sunil wrote:
...
> > > + for (unsigned int i = 0; i < ARRAY_SIZE(ad5529r_vss_supply_names); i++) {
> > > + ret = devm_regulator_get_enable_optional(dev,
> > > + ad5529r_vss_supply_names[i]);
> > > + if (ret && ret != -ENODEV)
> > > + return dev_err_probe(dev, ret,
> > > + "Failed to get and enable %s regulator\n",
> > > + ad5529r_vss_supply_names[i]);
> > > + }
> > Hmm... Can we use bulk regulator approach here?
>
> I don´t think the bulk helper fits here. Each VSS supply is independently
> optional, and there is currently no optional bulk get-enable regulator
> helper.
Ah, I see. Yeah, then we have to live with a for-loop...
--
With Best Regards,
Andy Shevchenko
_______________________________________________
linux-riscv mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/linux-riscv