Re: [PATCH 2/4] staging: iio: isl29018: document device tree bindings

Re: [PATCH 2/4] staging: iio: isl29018: document device tree bindings

[Jonathan Cameron]

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-1iNe0GrtECGEi8DpZVb4nw@public.gmane.org>
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>;
> +};
>