[PATCH v2 2/4] dt-bindings: iio: accel: bma255: Add bosch,bmc156_accel

Stephan Gerhold
2021-08-02T15:57:25Z

BMC156 is very smilar to BMC150, but it has only one accelerometer
interrupt pin. It would make sense if only INT1 was exposed but someone
at Bosch decided to only have an INT2 pin.

In this case, it does not make sense if the first interrupt pin is
treated as INT1 (since that pin does not exist). Add a note to the
bindings that the first interrupt pin is treated as INT2 for BMC156.

Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Reviewed-by: Rob Herring <robh@kernel.org>
Signed-off-by: Stephan Gerhold <stephan@gerhold.net>
---
.../bindings/iio/accel/bosch,bma255.yaml | 18 ++++++++++++++++++
1 file changed, 18 insertions(+)

diff --git a/Documentation/devicetree/bindings/iio/accel/bosch,bma255.yaml b/Documentation/devicetree/bindings/iio/accel/bosch,bma255.yaml
index 253b2051d0b1..478e75ae0885 100644
--- a/Documentation/devicetree/bindings/iio/accel/bosch,bma255.yaml
+++ b/Documentation/devicetree/bindings/iio/accel/bosch,bma255.yaml
@@ -26,6 +26,7 @@ properties:
- bosch,bma255
- bosch,bma280
- bosch,bmc150_accel
+ - bosch,bmc156_accel
- bosch,bmi055_accel

# bma180 driver in Linux
@@ -50,6 +51,9 @@ properties:
the one connected to the INT2 pin (if available). The type should be
IRQ_TYPE_EDGE_RISING.

+ BMC156 does not have an INT1 pin, therefore the first interrupt pin is
+ always treated as INT2.
+
interrupt-names:
minItems: 1
maxItems: 2
@@ -85,6 +89,20 @@ examples:
interrupt-names = "INT1";
};
};
+ - |
+ #include <dt-bindings/interrupt-controller/irq.h>
+ i2c {
+ #address-cells = <1>;
+ #size-cells = <0>;
+ accelerometer@10 {
+ compatible = "bosch,bmc156_accel";
+ reg = <0x10>;
+ vddio-supply = <&vddio>;
+ vdd-supply = <&vdd>;
+ interrupts = <116 IRQ_TYPE_EDGE_RISING>;
+ interrupt-names = "INT2";
+ };
+ };
- |
# include <dt-bindings/interrupt-controller/irq.h>
spi {
--
2.32.0