diff mbox

[v3,2/2] iio: light: opt3001: Add device tree binding documentation

Message ID 1435876079-26287-3-git-send-email-dannenberg@ti.com
State Needs Review / ACK, archived
Headers show

Checks

Context Check Description
robh/checkpatch warning total: 1 errors, 0 warnings, 0 lines checked
robh/patch-applied success

Commit Message

Andreas Dannenberg July 2, 2015, 10:27 p.m. UTC
Signed-off-by: Andreas Dannenberg <dannenberg@ti.com>
---
 .../devicetree/bindings/iio/light/opt3001.txt          | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/iio/light/opt3001.txt

Comments

Jonathan Cameron July 5, 2015, 2:09 p.m. UTC | #1
On 02/07/15 23:27, Andreas Dannenberg wrote:
> Signed-off-by: Andreas Dannenberg <dannenberg@ti.com>
Utterly standard binding so unless someone shouts, I'll pick this
up with the driver.

Thanks,

Jonathan
> ---
>  .../devicetree/bindings/iio/light/opt3001.txt          | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/iio/light/opt3001.txt
> 
> diff --git a/Documentation/devicetree/bindings/iio/light/opt3001.txt b/Documentation/devicetree/bindings/iio/light/opt3001.txt
> new file mode 100644
> index 0000000..d5f0765
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/light/opt3001.txt
> @@ -0,0 +1,18 @@
> +* Texas Instruments OPT3001 Ambient Light Sensor
> +
> +http://www.ti.com/product/opt3001
> +
> +Required properties:
> +  - compatible: should be "ti,opt3001"
> +  - reg: the I2C address of the sensor
> +  - interrupt-parent: should be the phandle for the interrupt controller
> +  - interrupts: interrupt mapping for GPIO IRQ (configure for falling edge)
> +
> +Example:
> +
> +opt3001@44 {
> +	compatible = "ti,opt3001";
> +	reg = <0x44>;
> +	interrupt-parent = <&gpio1>;
> +	interrupts = <28 IRQ_TYPE_EDGE_FALLING>;
> +};
> 

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Jonathan Cameron July 19, 2015, 1:17 p.m. UTC | #2
On 05/07/15 15:09, Jonathan Cameron wrote:
> On 02/07/15 23:27, Andreas Dannenberg wrote:
>> Signed-off-by: Andreas Dannenberg <dannenberg@ti.com>
> Utterly standard binding so unless someone shouts, I'll pick this
> up with the driver.
Applied to the togreg branch of iio.git - initially pushed
out as testing for the autobuilders to play with it.

Thanks,

Jonathan
> 
> Thanks,
> 
> Jonathan
>> ---
>>  .../devicetree/bindings/iio/light/opt3001.txt          | 18 ++++++++++++++++++
>>  1 file changed, 18 insertions(+)
>>  create mode 100644 Documentation/devicetree/bindings/iio/light/opt3001.txt
>>
>> diff --git a/Documentation/devicetree/bindings/iio/light/opt3001.txt b/Documentation/devicetree/bindings/iio/light/opt3001.txt
>> new file mode 100644
>> index 0000000..d5f0765
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/iio/light/opt3001.txt
>> @@ -0,0 +1,18 @@
>> +* Texas Instruments OPT3001 Ambient Light Sensor
>> +
>> +http://www.ti.com/product/opt3001
>> +
>> +Required properties:
>> +  - compatible: should be "ti,opt3001"
>> +  - reg: the I2C address of the sensor
>> +  - interrupt-parent: should be the phandle for the interrupt controller
>> +  - interrupts: interrupt mapping for GPIO IRQ (configure for falling edge)
>> +
>> +Example:
>> +
>> +opt3001@44 {
>> +	compatible = "ti,opt3001";
>> +	reg = <0x44>;
>> +	interrupt-parent = <&gpio1>;
>> +	interrupts = <28 IRQ_TYPE_EDGE_FALLING>;
>> +};
>>
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Andreas Dannenberg July 20, 2015, 6:22 p.m. UTC | #3
Hi Jon,

On 07/19/2015 08:17 AM, Jonathan Cameron wrote:
> On 05/07/15 15:09, Jonathan Cameron wrote:
>> On 02/07/15 23:27, Andreas Dannenberg wrote:
>>> Signed-off-by: Andreas Dannenberg <dannenberg@ti.com>
>> Utterly standard binding so unless someone shouts, I'll pick this
>> up with the driver.
> Applied to the togreg branch of iio.git - initially pushed
> out as testing for the autobuilders to play with it.

Thanks. Found it on the 'testing' branch, pulled the entire branch,
and successfully re-ran all my HW tests without any findings.

Regards,
Andreas
Jonathan Cameron July 23, 2015, 8:02 p.m. UTC | #4
On 20/07/15 19:22, Andreas Dannenberg wrote:
> Hi Jon,
> 
> On 07/19/2015 08:17 AM, Jonathan Cameron wrote:
>> On 05/07/15 15:09, Jonathan Cameron wrote:
>>> On 02/07/15 23:27, Andreas Dannenberg wrote:
>>>> Signed-off-by: Andreas Dannenberg <dannenberg@ti.com>
>>> Utterly standard binding so unless someone shouts, I'll pick this
>>> up with the driver.
>> Applied to the togreg branch of iio.git - initially pushed
>> out as testing for the autobuilders to play with it.
> 
> Thanks. Found it on the 'testing' branch, pulled the entire branch,
> and successfully re-ran all my HW tests without any findings.
> 
> Regards,
> Andreas
That's an impressive level of dedication!

Thanks

Jonathan


--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/iio/light/opt3001.txt b/Documentation/devicetree/bindings/iio/light/opt3001.txt
new file mode 100644
index 0000000..d5f0765
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/light/opt3001.txt
@@ -0,0 +1,18 @@ 
+* Texas Instruments OPT3001 Ambient Light Sensor
+
+http://www.ti.com/product/opt3001
+
+Required properties:
+  - compatible: should be "ti,opt3001"
+  - reg: the I2C address of the sensor
+  - interrupt-parent: should be the phandle for the interrupt controller
+  - interrupts: interrupt mapping for GPIO IRQ (configure for falling edge)
+
+Example:
+
+opt3001@44 {
+	compatible = "ti,opt3001";
+	reg = <0x44>;
+	interrupt-parent = <&gpio1>;
+	interrupts = <28 IRQ_TYPE_EDGE_FALLING>;
+};