On Wed, 27 Jul 2022 16:01:48 +0200 Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> wrote: > BMG160 has two interrupt pins to which interrupts can be freely mapped. > Correct the schema to express such case and fix warnings like: > > qcom/msm8916-alcatel-idol347.dtb: gyroscope@68: interrupts: [[97, 1], [98, 1]] is too long > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> We may need more than this. What if only INT2 is wired? I'd expect such a device's binding to include interrupt-names to cover that case. We'd also need a bunch of driver code to route the resulting interrupts. I think the snag is that adding such support will break existing bindings using the below. > --- > .../devicetree/bindings/iio/gyroscope/bosch,bmg160.yaml | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/Documentation/devicetree/bindings/iio/gyroscope/bosch,bmg160.yaml b/Documentation/devicetree/bindings/iio/gyroscope/bosch,bmg160.yaml > index b6bbc312a7cf..ebf98bc2eb7f 100644 > --- a/Documentation/devicetree/bindings/iio/gyroscope/bosch,bmg160.yaml > +++ b/Documentation/devicetree/bindings/iio/gyroscope/bosch,bmg160.yaml > @@ -24,6 +24,9 @@ properties: > > interrupts: > minItems: 1 > + items: > + - description: INT1 pin > + - description: INT2 pin > description: > Should be configured with type IRQ_TYPE_EDGE_RISING. >
