Re: [PATCH v10 3/3] iio: dac: Add AD5529R DAC driver support

On Thu, Aug 27, 2026 at 01:17:13PM +0200, Janani Sunil wrote:

On 8/27/26 10:23, Andy Shevchenko wrote:

quoted

On Thu, Aug 27, 2026 at 09:34:48AM +0200, Janani Sunil wrote:

...

quoted

quoted

+	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