From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jonathan Cameron Subject: Re: [PATCH v4 08/12] dt-bindings: iio: resolver: Document AD2S1200 bindings Date: Sun, 20 May 2018 12:05:53 +0100 Message-ID: <20180520120553.4b2526c9@archlinux> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: "devel" To: David Veenstra Cc: devel@driverdev.osuosl.org, devicetree@vger.kernel.org, lars@metafoo.de, Michael.Hennerich@analog.com, linux-iio@vger.kernel.org, robh+dt@kernel.org, pmeerw@pmeerw.net, knaack.h@gmx.de List-Id: devicetree@vger.kernel.org On Fri, 18 May 2018 20:22:50 +0200 David Veenstra wrote: > Add documentation for the device tree bindings of the AD2S1200 resolver. > > Signed-off-by: David Veenstra Nothing fundamentally wrong it in here, but a few things to tidy up. > --- > Changes in v4: > - Added vendor prefix to gpio function name. > - Added unit address. > - Changed commit subject to be more inline with other dt-bindings > commit . > > .../bindings/iio/resolver/ad2s1200.txt | 16 ++++++++++++++++ > 1 file changed, 16 insertions(+) > create mode 100644 Documentation/devicetree/bindings/iio/resolver/ad2s1200.txt > > diff --git a/Documentation/devicetree/bindings/iio/resolver/ad2s1200.txt b/Documentation/devicetree/bindings/iio/resolver/ad2s1200.txt > new file mode 100644 > index 000000000000..bbf54260c911 > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/resolver/ad2s1200.txt > @@ -0,0 +1,16 @@ > +Analog Devices AD2S1200 and AD2S1205 Resolver-to-Digital Converter > + > +Required properties: > + - compatible : should be "adi,ad2s1200" or "adi,ad2s1205" One line per entry. Saves us having to change this substantially when adding more parts in future. > + - reg : the SPI chip select number of the device > + - adi,sample-gpios : The GPIO pin connected to the SAMPLE line of the AD2S1200 > + - adi,rdvel-gpios : The GPIO pin connected to the RDVEL line of the AD2S1200 > + > +Example: > + > + resolver@0 { @4 if you are going to have reg = <4> below. > + compatible = "adi,ad2s1200"; > + reg = <4>; > + adi,sample-gpios = <&gpio 5 GPIO_ACTIVE_HIGH>; > + adi,rdvel-gpios = <&gpio 6 GPIO_ACTIVE_HIGH>; > + };