[PATCH] dt-bindings: iio: gyro: invensense,mpu3050: change irq maxItems

Rodrigo Gobbi posted 1 patch 10 months, 3 weeks ago
.../devicetree/bindings/iio/gyroscope/invensense,mpu3050.yaml   | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] dt-bindings: iio: gyro: invensense,mpu3050: change irq maxItems
Posted by Rodrigo Gobbi 10 months, 3 weeks ago
The mpu3050 datasheet describes that this IC only supports one INT pin,
which means one item with two cells inside binding.
Change max to match this description.

Signed-off-by: Rodrigo Gobbi <rodrigo.gobbi.7@gmail.com>
---
The mpu3050 supports only one INT pin as described at [1] at
"Pin Out and Signal Description" (pg 14), so maxItems could be 1 at max (one item with two cells).
Validated with dtbs_check/dt_binding_check and looks fine. There is a different case at [2],
which is using interrupts-extended but as far as I understand, it`s still max=1.
Tks and regards.

[1] https://invensense.tdk.com/wp-content/uploads/2015/02/PS-MPU-3050A-00-v2-7.pdf
[2] https://github.com/torvalds/linux/blob/015a99fa76650e7d6efa3e36f20c0f5b346fe9ce/arch/arm/boot/dts/qcom/qcom-apq8060-dragonboard.dts#L248
---
 .../devicetree/bindings/iio/gyroscope/invensense,mpu3050.yaml   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/iio/gyroscope/invensense,mpu3050.yaml b/Documentation/devicetree/bindings/iio/gyroscope/invensense,mpu3050.yaml
index d1a6103fc37a..f3242dc0e7e6 100644
--- a/Documentation/devicetree/bindings/iio/gyroscope/invensense,mpu3050.yaml
+++ b/Documentation/devicetree/bindings/iio/gyroscope/invensense,mpu3050.yaml
@@ -21,7 +21,7 @@ properties:
   vlogic-supply: true
 
   interrupts:
-    minItems: 1
+    maxItems: 1
     description:
       Interrupt mapping for the trigger interrupt from the internal oscillator.
 
-- 
2.49.0
Re: [PATCH] dt-bindings: iio: gyro: invensense,mpu3050: change irq maxItems
Posted by Krzysztof Kozlowski 10 months, 3 weeks ago
On 27/05/2025 23:51, Rodrigo Gobbi wrote:
> The mpu3050 datasheet describes that this IC only supports one INT pin,
> which means one item with two cells inside binding.
> Change max to match this description.
Fixes: 749787477ae4 ("dt-bindings:iio:gyro:invensense,mpu3050: txt to yaml format conversion.")


Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof
Re: [PATCH] dt-bindings: iio: gyro: invensense,mpu3050: change irq maxItems
Posted by Jonathan Cameron 10 months, 3 weeks ago
On Wed, 28 May 2025 07:55:40 +0200
Krzysztof Kozlowski <krzk@kernel.org> wrote:

> On 27/05/2025 23:51, Rodrigo Gobbi wrote:
> > The mpu3050 datasheet describes that this IC only supports one INT pin,
> > which means one item with two cells inside binding.
> > Change max to match this description.  
> Fixes: 749787477ae4 ("dt-bindings:iio:gyro:invensense,mpu3050: txt to yaml format conversion.")
> 
> 
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Applied to the fixes-togreg-for-6.16 branch of iio.git.

> 
> Best regards,
> Krzysztof