Re: [PATCH] iio: dac: max5522: use devm_regulator_get_enable_read_voltage()

[David Lechner]

On 2/14/26 12:31 PM, Andy Shevchenko wrote:
> On Sat, Feb 14, 2026 at 06:20:52PM +0000, Jonathan Cameron wrote:
>> On Tue, 10 Feb 2026 21:35:26 +0200
>> Andy Shevchenko <andriy.shevchenko@xxxxxxxxx> wrote:
>>> On Tue, Feb 10, 2026 at 07:51:27PM +0200, Antoniu Miclaus wrote:
> 
> ...
> 
>>>> +	state->vref_mv = ret / 1000;  
>>>
>>> "1000" --> "(MICRO / MILLI)" ?
>>> (yes, this way with the parentheses).
>>
>> I know I've argued against proliferation of the X_PER_Y macros
>> but this one does seem to be very common. Maybe it's time to bring in
>>
>> MICROVOLTS_PER_MILLIVOLT even if it's longer than the open coded version?
>>
>> Andy, what do you think?  I do worry that people will see it as fine
>> to add many many others if we start with this one though. So maybe not.
> 
> IIRC (sorry, if I am wrong) it was you who objected in the first place
> and I proposed that from day 1 discussing the "(MICRO / MILLI)" approach
> a year (?) or so ago. Answering to the Q: I am in favour of having that
> definition.
> 
>> Anyhow, doesn't need to be in this series.
> 
> True.
> 

How about MICRO_PER_MILLI? Then we won't have to add a new one
for every different type of unit.