From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jonathan Cameron Subject: Re: [PATCH 2/2] Documentation: DT: Add bmi160 imu binding Date: Sun, 6 Nov 2016 12:41:14 +0000 Message-ID: References: <20161103112527.29623-1-m.niestroj@grinn-global.com> <20161103112527.29623-2-m.niestroj@grinn-global.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20161103112527.29623-2-m.niestroj-z3quKL4iOrmQ6ZAhV5LmOA@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Marcin Niestroj Cc: Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald-Stadler , Daniel Baluta , Gregor Boirie , Sanchayan Maity , Rob Herring , Mark Rutland , linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org On 03/11/16 11:25, Marcin Niestroj wrote: > This adds documentation for Bosch BMI160 Inertial Measurement Unit > device-tree bindings. > > Signed-off-by: Marcin Niestroj Unless I missed it in the previous patch we should also have of tables added to the i2c and spi files (which is why the various tests haven't been screaming at me that this device doesn't have documented bindings). Otherwise, the use of interrupt names to indicate which pin on the chip is a little unusual (if you cribbed this from somewhere I've forgotten about then do say so!), so will want a devicetree bindings maintainer input on this. Thanks, Jonathan > --- > .../devicetree/bindings/iio/imu/bmi160.txt | 34 ++++++++++++++++++++++ > 1 file changed, 34 insertions(+) > create mode 100644 Documentation/devicetree/bindings/iio/imu/bmi160.txt > > diff --git a/Documentation/devicetree/bindings/iio/imu/bmi160.txt b/Documentation/devicetree/bindings/iio/imu/bmi160.txt > new file mode 100644 > index 0000000..b02ef3e > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/imu/bmi160.txt > @@ -0,0 +1,34 @@ > +Bosch BMI160 - Inertial Measurement Unit with Accelerometer, Gyroscope > +and externally connectable Magnetometer > + > +https://www.bosch-sensortec.com/bst/products/all_products/bmi160 > + > +Required properties: > + - compatible : should be "bosch,bmi160" > + - reg : the I2C address or SPI chip select number of the sensor > + - spi-max-frequency : set maximum clock frequency (only for SPI) > + > +Optional properties: > + - interrupt-parent : should be the phandle of the interrupt controller > + - interrupts : interrupt mapping for GPIO IRQ, must be IRQ_TYPE_LEVEL_LOW > + - interrupt-names : set to "INT2" if using INT2 pin > + > +Examples: > + > +bmi160@68 { > + compatible = "bosch,bmi160"; > + reg = <0x68>; > + > + interrupt-parent = <&gpio4>; > + interrupts = <12 IRQ_TYPE_LEVEL_LOW>; > +}; > + > +bmi160@0 { > + compatible = "bosch,bmi160"; > + reg = <0>; > + spi-max-frequency = <10000000>; > + > + interrupt-parent = <&gpio2>; > + interrupts = <12 IRQ_TYPE_LEVEL_LOW>; > + interrupt-names = "INT2"; > +}; > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html