On Mon, Mar 20, 2017 at 09:24:34PM -0600, Sam Povilus wrote: > Adding the ability for the ads7828 and ads7830 to use the device tree to > get their optional parameters, instead of using platform devices. This > allows people using custom boards to also use the ads7828 in a non-default > manner. > > v2: conforming to coding style > v3: changing from "_" to "-" for device tree entries > v4: using regulator subsystem for voltage source, renaming differential > input property > v5: checking for invalid vref values sooner and returning error if they are > bad, not warning user. > > Signed-off-by: Sam Povilus <kernel.development@xxxxxxxx> > --- > .../devicetree/bindings/hwmon/ads7828.txt | 26 ++++++++++++++++++++++ > .../devicetree/bindings/i2c/trivial-devices.txt | 2 -- > drivers/hwmon/ads7828.c | 21 +++++++++++++++++ > 3 files changed, 47 insertions(+), 2 deletions(-) > create mode 100644 Documentation/devicetree/bindings/hwmon/ads7828.txt > > diff --git a/Documentation/devicetree/bindings/hwmon/ads7828.txt b/Documentation/devicetree/bindings/hwmon/ads7828.txt > new file mode 100644 > index 000000000000..5a0c4a7c6144 > --- /dev/null > +++ b/Documentation/devicetree/bindings/hwmon/ads7828.txt > @@ -0,0 +1,26 @@ > +ads7828 properties > + > +Required properties: > +- compatible: Should be one of > + ti,ads7828 > + ti,ads7830 > +- reg: I2C address > + > +Optional properties: > + > +- ti,differential-input > + Set to use the device in differential mode. > +- vref What does this point to? Should be vref-supply? > + The external reference on the device is set to this regulators output. If it > + does not exists the internal reference will be used and output by the ads78xx > + on the "external vref" pin. > + > + Example ADS7828 node: > + > + ads7828: ads@48 { adc@... > + comatible = "ti,ads7828"; > + reg = <0x48>; > + vref = <&vref>; Add a vref node to the example. > + ti,differential-input; > + }; > + -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html