Re: [PATCH 5/9] iio: mma8452: add devicetree documentation

[Martin Kepplinger <martink@posteo.de>]

Am 2015-07-06 um 10:06 schrieb Martin Fuzzey:
> On 05/07/15 13:48, Jonathan Cameron wrote:
>> On 04/07/15 14:55, Martin Kepplinger wrote:
>>> Signed-off-by: Martin Kepplinger
>>> <martin.kepplinger@theobroma-systems.com>
>>> Signed-off-by: Christoph Muellner
>>> <christoph.muellner@theobroma-systems.com>
>> This is trivial enough I'll take it without a device tree ack (after a
>> few days)
>> but convention is to CC all device tree patches to the maintainers of
>> devicetree
>> bindings as well as the devicetree list.
> 
> Currently the binding is covered by
> Documentation/devicetree/bindings/i2c/trivial-devices.txt
> Shouldn't that one be removed now that a full binding document is being
> created?
> 

True, thanks very much! Included for the new version of the patches.

> Regards,
> 
> Martin
> 
>> Jonathan
>>> ---
>>>   .../devicetree/bindings/iio/accel/mma8452.txt       | 21
>>> +++++++++++++++++++++
>>>   1 file changed, 21 insertions(+)
>>>   create mode 100644
>>> Documentation/devicetree/bindings/iio/accel/mma8452.txt
>>>
>>> diff --git a/Documentation/devicetree/bindings/iio/accel/mma8452.txt
>>> b/Documentation/devicetree/bindings/iio/accel/mma8452.txt
>>> new file mode 100644
>>> index 0000000..8d98e05
>>> --- /dev/null
>>> +++ b/Documentation/devicetree/bindings/iio/accel/mma8452.txt
>>> @@ -0,0 +1,21 @@
>>> +Freescale MMA8452Q, MMA8453Q, MMA8652FC or MMA8653FC triaxial
>>> accelerometer
>>> +
>>> +Required properties:
>>> +
>>> +  - compatible: should be "fsl,mma8653", "fsl,mma8652",
>>> "fsl,mma8453" or
>>> +    "fsl,mma8452" respectively.
>>> +  - reg: the I2C address of the chip
>>> +
>>> +Optional properties:
>>> +
>>> +  - interrupt-parent: should be the phandle for the interrupt
>>> controller
>>> +  - interrupts: interrupt mapping for GPIO IRQ
>>> +
>>> +Example:
>>> +
>>> +    mma8653fc@1d {
>>> +        compatible = "fsl,mma8653";
>>> +        reg = <0x1d>;
>>> +        interrupt-parent = <&gpio1>;
>>> +        interrupts = <5 0>;
>>> +    };
>>>
>