On Fri, 20 Apr 2018 19:05:12 +0200 Jean-Baptiste Maneyrol <jmaneyrol@xxxxxxxxxxxxxx> wrote: > On 20/04/2018 18:54, Martin Kelly wrote: > > Document that the hardware supports falling edge, rising edge, level > > low, and level high interrupt types, rather than just rising edge. > > > > The language used is the same as that in st_lsm6dsx.txt. > > > > Signed-off-by: Martin Kelly <mkelly@xxxxxxxx> Hi Martin, Please pick up an acks / reviewed-by's from earlier versions. The chances are I'll miss them some of the time otherwise. Rob acked this one. Applied with the two acks I know of to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > > --- > > Documentation/devicetree/bindings/iio/imu/inv_mpu6050.txt | 11 ++++++++--- > > 1 file changed, 8 insertions(+), 3 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/iio/imu/inv_mpu6050.txt b/Documentation/devicetree/bindings/iio/imu/inv_mpu6050.txt > > index 2b4514592f83..6b106d5ef298 100644 > > --- a/Documentation/devicetree/bindings/iio/imu/inv_mpu6050.txt > > +++ b/Documentation/devicetree/bindings/iio/imu/inv_mpu6050.txt > > @@ -11,7 +11,12 @@ Required properties: > > "invensense,icm20608" > > - reg : the I2C address of the sensor > > - interrupt-parent : should be the phandle for the interrupt controller > > - - interrupts : interrupt mapping for GPIO IRQ > > + - interrupts: interrupt mapping for IRQ. It should be configured with flags > > + IRQ_TYPE_LEVEL_HIGH, IRQ_TYPE_EDGE_RISING, IRQ_TYPE_LEVEL_LOW or > > + IRQ_TYPE_EDGE_FALLING. > > + > > + Refer to interrupt-controller/interrupts.txt for generic interrupt client node > > + bindings. > > > > Optional properties: > > - mount-matrix: an optional 3x3 mounting rotation matrix > > @@ -24,7 +29,7 @@ Example: > > compatible = "invensense,mpu6050"; > > reg = <0x68>; > > interrupt-parent = <&gpio1>; > > - interrupts = <18 1>; > > + interrupts = <18 IRQ_TYPE_EDGE_RISING>; > > mount-matrix = "-0.984807753012208", /* x0 */ > > "0", /* y0 */ > > "-0.173648177666930", /* z0 */ > > @@ -41,7 +46,7 @@ Example: > > compatible = "invensense,mpu9250"; > > reg = <0x68>; > > interrupt-parent = <&gpio3>; > > - interrupts = <21 1>; > > + interrupts = <21 IRQ_TYPE_LEVEL_HIGH>; > > i2c-gate { > > #address-cells = <1>; > > #size-cells = <0>; > > > > > Acked-by: Jean-Baptiste Maneyrol <jmaneyrol@xxxxxxxxxxxxxx> > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- 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