Re: [PATCH 6/6] iio: adc: ad7173: Add support for AD411x devices

From: Ceclan, Dumitru
Date: Tue May 14 2024 - 03:29:49 EST


On 01/04/2024 22:45, David Lechner wrote:
>> unsigned int clock;
>> unsigned int id;
>> char *name;
>> + bool has_current_inputs;
> Maybe more future-proof to have num_current_inputs instead of bool?

At first I agreed with this, but the way that the current inputs are
mapped to reg values does not really offer a way to extend them
without changing completely the numbering scheme. If that happens,
changing this field will be the least bit to need changing.