On 06/04/17 15:20, Jacopo Mondi wrote: > Add documentation for max9611 driver. > Document attributes describing value of shunt resistor installed between > RS+ and RS- voltage sense inputs. > > Signed-off-by: Jacopo Mondi <jacopo+renesas@xxxxxxxxxx> Looks good to me. There is the small matter that when we generalize this we will probably loose some detail, but such is life. Applied to the togreg branch of iio.git and pushed out as testing. Thanks, Jonathan > --- > Documentation/ABI/testing/sysfs-bus-iio-adc-max9611 | 17 +++++++++++++++++ > 1 file changed, 17 insertions(+) > create mode 100644 Documentation/ABI/testing/sysfs-bus-iio-adc-max9611 > > diff --git a/Documentation/ABI/testing/sysfs-bus-iio-adc-max9611 b/Documentation/ABI/testing/sysfs-bus-iio-adc-max9611 > new file mode 100644 > index 0000000..6d2d2b0 > --- /dev/null > +++ b/Documentation/ABI/testing/sysfs-bus-iio-adc-max9611 > @@ -0,0 +1,17 @@ > +What: /sys/bus/iio/devices/iio:deviceX/in_power_shunt_resistor > +Date: March 2017 > +KernelVersion: 4.12 > +Contact: linux-iio@xxxxxxxxxxxxxxx > +Description: The value of the shunt resistor used to compute power drain on > + common input voltage pin (RS+). In Ohms. > + > +What: /sys/bus/iio/devices/iio:deviceX/in_current_shunt_resistor > +Date: March 2017 > +KernelVersion: 4.12 > +Contact: linux-iio@xxxxxxxxxxxxxxx > +Description: The value of the shunt resistor used to compute current flowing > + between RS+ and RS- voltage sense inputs. In Ohms. > + > +These attributes describe a single physical component, exposed as two distinct > +attributes as it is used to calculate two different values: power load and > +current flowing between RS+ and RS- inputs. >