Hi Jacopo, On Wed, Dec 7, 2016 at 9:29 AM, jacopo@xxxxxxxxxx <jacopo@xxxxxxxxxx> wrote: > On 06/12/2016 22:00, Peter Meerwald-Stadler wrote: >> On Tue, 6 Dec 2016, Jacopo Mondi wrote: >>> Add IIO driver for Maxim MAX11100 single-channel ADC. >>> Support raw_read mode only. >> >> the read_raw() is supposed to return millivolts (after application of >> offset and scale); maybe need _SCALE? > > How can I return millivolts here? They vary in function of the supplied Vref > as ( V = val * Vref / (2 ^ 16 - 1)) where "val" is digital value output by > the ADC. > Since Vref can range from 3.8V to 5.25V, while it's typically 4.096V, isn't > it more appropriate to let userspace perform conversion to millivolts, as it > knows what Vref value is supplied to the ADC? Specify Vref in DT? Which finally gives a good reason to add a DT binding document for MAX11100 ;-) Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds