Ok will fix in v4. Please review the 2/2 patch when you get a chance :) On Mon, Jul 13, 2015 at 2:42 AM, Jonathan Cameron <jic23@xxxxxxxxxx> wrote: > > > On 13 July 2015 03:20:42 BST, Matt Ranostay <mranostay@xxxxxxxxx> wrote: >>Document compatible string, and required DT properties for APDS9960 >>chipset driver. >> >>Reviewed-by: Marek Vasut <marex@xxxxxxx> >>Signed-off-by: Matt Ranostay <mranostay@xxxxxxxxx> >>--- >>.../devicetree/bindings/iio/light/apds9960.txt | 19 >>+++++++++++++++++++ >> 1 file changed, 19 insertions(+) >>create mode 100644 >>Documentation/devicetree/bindings/iio/light/apds9960.txt >> >>diff --git a/Documentation/devicetree/bindings/iio/light/apds9960.txt >>b/Documentation/devicetree/bindings/iio/light/apds9960.txt >>new file mode 100644 >>index 0000000..7a47d28 >>--- /dev/null >>+++ b/Documentation/devicetree/bindings/iio/light/apds9960.txt >>@@ -0,0 +1,19 @@ >>+* Avago APDS9960 gesture/RGB/ALS/proximity sensor >>+ >>+http://www.avagotech.com/docs/AV02-4191EN >>+ >>+Required properties: >>+ >>+ - compatible: must be "avago,apds9960" >>+ - reg: the I2c address of the sensor >>+ - interrupt-parent: should be the phandle for the interrupt >>controller >>+ - interrupts: interrupt mapping for GPIO IRQ > Usual trick here is to cross reference the interrupt bindings doc to allow all the > complexity without duplicating docs too much. >>+ >>+Example: >>+ >>+apds9960@39 { >>+ compatible = "avago,apds9960"; >>+ reg = <0x39>; >>+ interrupt-parent = <&gpio1>; >>+ interrupts = <16 1>; >>+}; > > -- > Sent from my Android device with K-9 Mail. Please excuse my brevity. -- 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