On 02/04/17 10:36, Jonathan Cameron wrote: > On 27/03/17 16:12, 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> > Good. Will pick up once the binding is sorted. > > Kick me if I don't! Oops. Missed the units being micro ohms. They want to be ohms to be consistent within the ABI. We let various odd units go in right back at the beginning to try and remain compatible with hwmon. Turned out to be a bad idea. Also the ina2xx uses ohms and you should stay consistent with that. 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..5020d00 >> --- /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 micro 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 micro 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. >> > > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html >
