Re: [PATCH v5] input: add ADC resistor ladder driver

On Mon, Aug 1, 2016 at 12:51 PM, Dmitry Torokhov
[off-list ref] wrote:

On Fri, Jul 29, 2016 at 12:44:33AM +0200, Alexandre Belloni wrote:

quoted

From: Dmitry Torokhov <dmitry.torokhov@gmail.com>

A common way of multiplexing buttons on a single input in cheap devices is
to use a resistor ladder on an ADC. This driver supports that configuration
by polling an ADC channel provided by IIO.

Acked-by: Jonathan Cameron <jic23@kernel.org>
Signed-off-by: Alexandre Belloni <redacted>
---

Changes in v5:
 - Dmitry's changes
 - s/mvolt/millivolt/

Rob, is "millivolt" OK with you?

No, because -microvolt is what is documented in
Documentation/devicetree/bindings/property-units.txt. We could add
millivolt, but I don't really see any reason to as microvolt provides
enough range. I don't want just any units because then it's just at
the whim of the binding author.

Rob