Message ID | 20210428082203.3587022-1-sean@geanix.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [RFC,1/4] dt-bindings: iio: accel: fxls8962af: add interrupt options | expand |
On Wed, 28 Apr 2021 10:22:00 +0200 Sean Nyekjaer <sean@geanix.com> wrote: > This in done for supporting hw buffered sampling > > Signed-off-by: Sean Nyekjaer <sean@geanix.com> Please squash into the original binding patch. Bindings should reflect the hardware features rather than what the driver happens to be currently taking advantage of. Occasionally we'll do it in multiple parts because we aren't sure what the binding will look like, but in this case you've posted this before the other has been applied so just merge them together. Thanks, Jonathan > --- > This series depends on "iio: accel: add support for > FXLS8962AF/FXLS8964AF accelerometers" > > .../bindings/iio/accel/nxp,fxls8962af.yaml | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml b/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml > index c7be7a1679ab..e0e5542377df 100644 > --- a/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml > +++ b/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml > @@ -32,6 +32,16 @@ properties: > interrupts: > maxItems: 1 > > + interrupt-names: > + maxItems: 1 > + items: > + enum: > + - INT1 > + - INT2 > + > + drive-open-drain: > + type: boolean > + > required: > - compatible > - reg > @@ -51,6 +61,7 @@ examples: > reg = <0x62>; > interrupt-parent = <&gpio0>; > interrupts = <0 IRQ_TYPE_LEVEL_HIGH>; > + interrupt-names = "INT1"; > }; > }; > - | > @@ -66,5 +77,6 @@ examples: > spi-max-frequency = <4000000>; > interrupt-parent = <&gpio0>; > interrupts = <0 IRQ_TYPE_LEVEL_HIGH>; > + interrupt-names = "INT1"; > }; > };
On Wed, Apr 28, 2021 at 10:22:00AM +0200, Sean Nyekjaer wrote: > This in done for supporting hw buffered sampling > > Signed-off-by: Sean Nyekjaer <sean@geanix.com> > --- > This series depends on "iio: accel: add support for > FXLS8962AF/FXLS8964AF accelerometers" > > .../bindings/iio/accel/nxp,fxls8962af.yaml | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml b/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml > index c7be7a1679ab..e0e5542377df 100644 > --- a/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml > +++ b/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml > @@ -32,6 +32,16 @@ properties: > interrupts: > maxItems: 1 > > + interrupt-names: > + maxItems: 1 > + items: > + enum: > + - INT1 > + - INT2 You can simplify this to: interrupt-names: enum: - INT1 - INT2 > + > + drive-open-drain: > + type: boolean > + > required: > - compatible > - reg > @@ -51,6 +61,7 @@ examples: > reg = <0x62>; > interrupt-parent = <&gpio0>; > interrupts = <0 IRQ_TYPE_LEVEL_HIGH>; > + interrupt-names = "INT1"; > }; > }; > - | > @@ -66,5 +77,6 @@ examples: > spi-max-frequency = <4000000>; > interrupt-parent = <&gpio0>; > interrupts = <0 IRQ_TYPE_LEVEL_HIGH>; > + interrupt-names = "INT1"; > }; > }; > -- > 2.31.0 >
diff --git a/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml b/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml index c7be7a1679ab..e0e5542377df 100644 --- a/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml +++ b/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml @@ -32,6 +32,16 @@ properties: interrupts: maxItems: 1 + interrupt-names: + maxItems: 1 + items: + enum: + - INT1 + - INT2 + + drive-open-drain: + type: boolean + required: - compatible - reg @@ -51,6 +61,7 @@ examples: reg = <0x62>; interrupt-parent = <&gpio0>; interrupts = <0 IRQ_TYPE_LEVEL_HIGH>; + interrupt-names = "INT1"; }; }; - | @@ -66,5 +77,6 @@ examples: spi-max-frequency = <4000000>; interrupt-parent = <&gpio0>; interrupts = <0 IRQ_TYPE_LEVEL_HIGH>; + interrupt-names = "INT1"; }; };
This in done for supporting hw buffered sampling Signed-off-by: Sean Nyekjaer <sean@geanix.com> --- This series depends on "iio: accel: add support for FXLS8962AF/FXLS8964AF accelerometers" .../bindings/iio/accel/nxp,fxls8962af.yaml | 12 ++++++++++++ 1 file changed, 12 insertions(+)