  `On 2018-04-10 17:28, Peter Rosin wrote:> +Example:> +The system voltage is circa 12V, but divided down with a 22/200> +voltage divider to adjust it to the ADC range.> +> +SYSV        ADC       GND> +  +          +         +> +  |  .-----. | .----.  |> +  '--| 200 |-+-| 22 |--'> +     '-----'   '----'> +> +sysv {> +	compatible = "voltage-divider";> +	io-channels = <&maxadc 1>;> +> +	/* Multiply the ADC voltage by 222/22 to get the system voltage. */> +	numerator = <222>; /* 200 + 22 */> +	denominator = <22>;> +};While I already got a reviewed-by from Rob, and maybe I shouldn't bestirring the pot, but I had an umpteenth look and I now think thisone looks a bit odd. It shows a bit that it originates from when thecompatible was the very generic "io-channel-unit-converter" in v1of the series. What I mean is that a voltage divider presumable alwaysgets you a lower voltage. Therefore, one would assume that thedenominator should be larger than the numerator. The fact that thistranslates into the inverted fraction when calculating backwardsthrough the voltage divider should probably not affect the binding.So, in the above example, I think it would make more sense to have	numerator = <22>;	denominator = <222>; /* 200 + 22 */(and then, naturally, adjust the driver to invert the fraction)Comments?Cheers,Peter`   