On 09/11/16 15:18, Marcin Niestroj wrote: > On 06.11.2016 13:41, Jonathan Cameron wrote: >> 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 <m.niestroj@xxxxxxxxxxxxxxxx> >> 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). > > Ok, I will add them. > >> >> 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. > > I have used interrupt names similar as in other driver. Please see > [1] for it's DT documentation and [2] for implementation. > [1] Documentation/devicetree/bindings/iio/accel/mma8452.txt > [2] drivers/iio/accel/mma8452.c Fair enough. Just put the INT1 option in the docs as requested and this should be fine. Jonathan > >> >> 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@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html