On 07/10/16 01:48, Brian Masney wrote: > Fix the following warnings from checkpatch: > > WARNING: DT compatible string "isil,isl29018" appears un-documented -- > check ./Documentation/devicetree/bindings/ > WARNING: DT compatible string "isil,isl29023" appears un-documented -- > check ./Documentation/devicetree/bindings/ > WARNING: DT compatible string "isil,isl29035" appears un-documented -- > check ./Documentation/devicetree/bindings/ > > Signed-off-by: Brian Masney <masneyb@xxxxxxxxxxxxx> Device tree patches should always be cc'd to the device tree list and maintainers. (done above). As this is very simple I'm happy to take it without an Ack from them (though one would always be welcome and can be added for a week or so as I'm not pushing this out in a non rebasing tree yet). Applied to the togreg branch of iio.git - pushed out as testing for the autobuilders to play with it. Mark and Rob, just noticed we are down to just the two of you. Good luck guys and hope you find more assistance soon if you want it. Thanks, Jonathan > --- > .../devicetree/bindings/iio/light/isl29018.txt | 28 ++++++++++++++++++++++ > 1 file changed, 28 insertions(+) > create mode 100644 Documentation/devicetree/bindings/iio/light/isl29018.txt > > diff --git a/Documentation/devicetree/bindings/iio/light/isl29018.txt b/Documentation/devicetree/bindings/iio/light/isl29018.txt > new file mode 100644 > index 0000000..7e53524 > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/light/isl29018.txt > @@ -0,0 +1,28 @@ > +* ISL 29018/29023/29035 I2C ALS, Proximity, and Infrared sensor > + > +Required properties: > + > + - compatible: Should be one of > + "isil,isl29018" > + "isil,isl29023" > + "isil,isl29035" > + - reg: the I2C address of the device > + > +Optional properties: > + > + - interrupt-parent: should be the phandle for the interrupt controller > + - interrupts: the sole interrupt generated by the device > + > + Refer to interrupt-controller/interrupts.txt for generic interrupt client > + node bindings. > + > + - vcc-supply: phandle to the regulator that provides power to the sensor. > + > +Example: > + > +isl29018@44 { > + compatible = "isil,isl29018"; > + reg = <0x44>; > + interrupt-parent = <&gpio>; > + interrupts = <TEGRA_GPIO(Z, 2) IRQ_TYPE_LEVEL_HIGH>; > +}; > -- 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