Messages in this thread | | | Date | Tue, 14 May 2024 10:28:21 +0300 | Subject | Re: [PATCH 6/6] iio: adc: ad7173: Add support for AD411x devices | From | "Ceclan, Dumitru" <> |
| |
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.
| |