On Mon, Apr 10, 2017 at 10:42 AM, Rob Herring <robh@xxxxxxxxxx> wrote: > On Tue, Apr 04, 2017 at 12:20:34PM -0700, John Muir wrote: >> Signed-off-by: John Muir <john@xxxxxxxxx> >> --- >> .../devicetree/bindings/hwmon/max31760.txt | 58 ++++++++++++++++++++++ >> 1 file changed, 58 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/hwmon/max31760.txt >> >> diff --git a/Documentation/devicetree/bindings/hwmon/max31760.txt b/Documentation/devicetree/bindings/hwmon/max31760.txt >> new file mode 100644 >> index 000000000000..43787a77c322 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/hwmon/max31760.txt >> @@ -0,0 +1,58 @@ >> +MAX31760 fan controller >> +----------------------- >> + >> +This device supports I2C only. Many properties of this device are configurable >> +thorugh the hwmon interface. See also Documentation/hwmon/max31760. > > I really think we need to describe the fans as separate nodes and > preferably with a common binding. This is the second fan controller > binding recently[1]. Forgot the link: https://patchwork.kernel.org/patch/9643643/ -- 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