Re: [PATCH] dt-bindings: mma8452: Re-word 'interrupt-names' description

From: Martin Kepplinger
Date: Tue Oct 22 2019 - 04:43:15 EST


On 22.10.19 05:56, Andrey Smirnov wrote:
> Current wording in the binding documentation doesn't make it 100%
> clear that only one of "INT1" and "INT2" will ever be used by the
> driver and that specifying both has no advantages. Re-word it to make
> this aspect a bit more explicit.
>
> Signed-off-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx>
> Cc: Chris Healy <cphealy@xxxxxxxxx>
> Cc: Jonathan Cameron <jic23@xxxxxxxxxx>
> Cc: Hartmut Knaack <knaack.h@xxxxxx>
> Cc: Lars-Peter Clausen <lars@xxxxxxxxxx>
> Cc: Peter Meerwald-Stadler <pmeerw@xxxxxxxxxx>
> Cc: linux-iio@xxxxxxxxxxxxxxx
> Cc: devicetree@xxxxxxxxxxxxxxx
> Cc: linux-kernel@xxxxxxxxxxxxxxx
> ---
> Documentation/devicetree/bindings/iio/accel/mma8452.txt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/iio/accel/mma8452.txt b/Documentation/devicetree/bindings/iio/accel/mma8452.txt
> index e132394375a1..b27b6bee9eb6 100644
> --- a/Documentation/devicetree/bindings/iio/accel/mma8452.txt
> +++ b/Documentation/devicetree/bindings/iio/accel/mma8452.txt
> @@ -17,7 +17,7 @@ Optional properties:
>
> - interrupts: interrupt mapping for GPIO IRQ
>
> - - interrupt-names: should contain "INT1" and/or "INT2", the accelerometer's
> + - interrupt-names: should contain "INT1" or "INT2", the accelerometer's
> interrupt line in use.
>
> - vdd-supply: phandle to the regulator that provides vdd power to the accelerometer.
>

Acked-by: Martin Kepplinger <martink@xxxxxxxxx>

thanks,

martin