| Message ID | 20241225181338.69672-4-l.rubusch@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | iio: accel: adxl345: add FIFO operating with IRQ triggered watermark events | expand |
On Wed, Dec 25, 2024 at 06:13:34PM +0000, Lothar Rubusch wrote: > Add interrupt-names INT1 and INT2 for the two interrupt lines of the > sensor. > > When one of the two interrupt lines is connected, the interrupt as its > interrupt-name, need to be declared in the devicetree. The driver then > configures the sensor to indicate its events on either INT1 or INT2. > > If no interrupt is configured, then no interrupt-name should be > configured, and vice versa. In this case the sensor runs in FIFO BYPASS > mode. This allows sensor measurements, but none of the sensor events. > > Signed-off-by: Lothar Rubusch <l.rubusch@gmail.com> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On Fri, 27 Dec 2024 08:58:00 +0100 Krzysztof Kozlowski <krzk@kernel.org> wrote: > On Wed, Dec 25, 2024 at 06:13:34PM +0000, Lothar Rubusch wrote: > > Add interrupt-names INT1 and INT2 for the two interrupt lines of the > > sensor. > > > > When one of the two interrupt lines is connected, the interrupt as its > > interrupt-name, need to be declared in the devicetree. The driver then > > configures the sensor to indicate its events on either INT1 or INT2. > > > > If no interrupt is configured, then no interrupt-name should be > > configured, and vice versa. In this case the sensor runs in FIFO BYPASS > > mode. This allows sensor measurements, but none of the sensor events. > > > > Signed-off-by: Lothar Rubusch <l.rubusch@gmail.com> > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > Best regards, > Krzysztof > Applied. thanks Jonathan
diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml index bc46ed00f..84d949392 100644 --- a/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml +++ b/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml @@ -37,6 +37,14 @@ properties: interrupts: maxItems: 1 + interrupt-names: + items: + - enum: [INT1, INT2] + +dependencies: + interrupts: [ interrupt-names ] + interrupt-names: [ interrupts ] + required: - compatible - reg @@ -60,6 +68,7 @@ examples: reg = <0x2a>; interrupt-parent = <&gpio0>; interrupts = <0 IRQ_TYPE_LEVEL_HIGH>; + interrupt-names = "INT1"; }; }; - | @@ -78,5 +87,6 @@ examples: spi-cpha; interrupt-parent = <&gpio0>; interrupts = <0 IRQ_TYPE_LEVEL_HIGH>; + interrupt-names = "INT2"; }; };
Add interrupt-names INT1 and INT2 for the two interrupt lines of the sensor. When one of the two interrupt lines is connected, the interrupt as its interrupt-name, need to be declared in the devicetree. The driver then configures the sensor to indicate its events on either INT1 or INT2. If no interrupt is configured, then no interrupt-name should be configured, and vice versa. In this case the sensor runs in FIFO BYPASS mode. This allows sensor measurements, but none of the sensor events. Signed-off-by: Lothar Rubusch <l.rubusch@gmail.com> --- .../devicetree/bindings/iio/accel/adi,adxl345.yaml | 10 ++++++++++ 1 file changed, 10 insertions(+)