Re: [PATCH v2 03/14] dt-bindings: iio: accel: bosch,bma220 change irq type

Krzysztof Kozlowski
2025-09-11T07:33:28Z

On Wed, Sep 10, 2025 at 10:57:08AM +0300, Petre Rodan wrote:
> Set the interrupt type to rising edge instead of high level.
>
> Quoting from the datasheet:
>
> "If at least one of the configured conditions applies, an interrupt
> (logic ‘1’) is issued through the INT pin of the sensor."

I don't see how this explains/suggests raising edge.

>
> The old code did not request an irq line via devm_request_threaded_irq()
> so there is no backward compatibility that would be affected by this
> change.

This sentence is irrelevant. You are changing only example, which has
zero impact.

Rewrite the subject and commit msg to accurately say this - it is only
example - because now you suggest you actually change something...

Best regards,
Krzysztof