From mboxrd@z Thu Jan  1 00:00:00 1970
From: Martin Kepplinger <martink-1KBjaw7Xf1+zQB+pC5nmwQ@public.gmane.org>
Subject: Re: [PATCH 5/9] iio: mma8452: add devicetree documentation
Date: Mon, 06 Jul 2015 10:13:10 +0200
Message-ID: <559A3896.9080102@posteo.de>
References: <1436018110-3903-1-git-send-email-martink@posteo.de> <1436018110-3903-6-git-send-email-martink@posteo.de> <5599199B.6030006@kernel.org> <559A3715.3040206@parkeon.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 7bit
Return-path: <linux-iio-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
In-Reply-To: <559A3715.3040206-mB3Nsq4MPf1BDgjK7y7TUQ@public.gmane.org>
Sender: linux-iio-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
To: Martin Fuzzey <mfuzzey-mB3Nsq4MPf1BDgjK7y7TUQ@public.gmane.org>, Jonathan Cameron <jic23-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>, knaack.h-Mmb7MZpHnFY@public.gmane.org, lars-Qo5EllUWu/uELgA04lAiVw@public.gmane.org, pmeerw-jW+XmwGofnusTnJN9+BGXg@public.gmane.org, roberta.dobrescu-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, christoph.muellner-SN7IsUiht6C/RdPyistoZJqQE7yCjDx5@public.gmane.org
Cc: linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Martin Kepplinger <martin.kepplinger-SN7IsUiht6C/RdPyistoZJqQE7yCjDx5@public.gmane.org>
List-Id: devicetree@vger.kernel.org

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-SN7IsUiht6C/RdPyistoZJqQE7yCjDx5@public.gmane.org>
>>> Signed-off-by: Christoph Muellner
>>> <christoph.muellner-SN7IsUiht6C/RdPyistoZJqQE7yCjDx5@public.gmane.org>
>> 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>;
>>> +    };
>>>
>