Messages in this thread | | | Date | Sun, 6 Sep 2020 16:01:52 +0100 | From | Jonathan Cameron <> | Subject | Re: [PATCH v2 2/3] dt-bindings: iio: gyro: adxrs290: Add interrupts support |
| |
On Thu, 3 Sep 2020 18:39:49 +0530 Nishant Malpani <nish.malpani25@gmail.com> wrote:
> Include 'interrupts' property and provide a suitable example for using > a GPIO interrupt line. > > Signed-off-by: Nishant Malpani <nish.malpani25@gmail.com>
One little point inline. Otherwise looks fine to me
Thanks,
Jonathan
> --- > > Changes in v2: > > - remove 'interrupts' property from the required properties list > - rewrite commit message > --- > .../devicetree/bindings/iio/gyroscope/adi,adxrs290.yaml | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/Documentation/devicetree/bindings/iio/gyroscope/adi,adxrs290.yaml b/Documentation/devicetree/bindings/iio/gyroscope/adi,adxrs290.yaml > index 61adb2c2454b..a5f209e8bbef 100644 > --- a/Documentation/devicetree/bindings/iio/gyroscope/adi,adxrs290.yaml > +++ b/Documentation/devicetree/bindings/iio/gyroscope/adi,adxrs290.yaml > @@ -28,6 +28,9 @@ properties: > > spi-cpha: true > > + interrupts: > + maxItems: 1 > + > required: > - compatible > - reg > @@ -39,6 +42,8 @@ additionalProperties: false > > examples: > - | > + #include <dt-bindings/gpio/gpio.h>
Shouldn't need that header. I don't see anything being used that is defined in gpio.h
> + #include <dt-bindings/interrupt-controller/irq.h> > spi { > #address-cells = <1>; > #size-cells = <0>; > @@ -48,6 +53,8 @@ examples: > spi-max-frequency = <5000000>; > spi-cpol; > spi-cpha; > + interrupt-parent = <&gpio>; > + interrupts = <25 IRQ_TYPE_EDGE_RISING>; > }; > }; > ...
| |