[PATCH V13 2/9] dt-bindings: iio: imu: icm42600: Add icm42607

[Jean-Baptiste Maneyrol]

Hello Chris and Jonathan,

concerning dt bindings, my initial understanding was that we had a file per
driver. But here, Chris is doing a new driver for icm42607 while adding new
bindings here.

Does it means we don't have 1 binding file per driver, and there is no need
to create a new binding file for inv_icm42607 driver?

Despite the naming, icm42607 chips are a complete new design very different
than all other icm42600 chips. It using similar IPs for things like the FIFO,
but all other parts are different. Especially, it doesn't use banks for
registers access but indirect access delegated to the chip internals for
accessing certain registers.

Thanks,
JB

>From: Chris Morgan <macromorgan at hotmail.com>
>
>Add the ICM42607 and ICM42607P inertial measurement unit.
>
>This device is functionally very similar to the icm42600 series with a
>very different register layout. The driver does not require an
>interrupt for these specific chip revisions.
>
>Signed-off-by: Chris Morgan <macromorgan at hotmail.com>
>Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski at oss.qualcomm.com>
>---
> .../bindings/iio/imu/invensense,icm42600.yaml  | 18 +++++++++++++++++-
> 1 file changed, 17 insertions(+), 1 deletion(-)
>
>diff --git a/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml b/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml
>index 9b2af104f186..81b6e85decd5 100644
>--- a/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml
>+++ b/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml
>@@ -30,6 +30,8 @@ properties:
>       - invensense,icm42600
>       - invensense,icm42602
>       - invensense,icm42605
>+      - invensense,icm42607
>+      - invensense,icm42607p
>       - invensense,icm42622
>       - invensense,icm42631
>       - invensense,icm42686
>@@ -67,10 +69,24 @@ properties:
> required:
>   - compatible
>   - reg
>-  - interrupts
> 
> allOf:
>   - $ref: /schemas/spi/spi-peripheral-props.yaml#
>+  - if:
>+      properties:
>+        compatible:
>+          contains:
>+            enum:
>+              - invensense,icm42600
>+              - invensense,icm42602
>+              - invensense,icm42605
>+              - invensense,icm42622
>+              - invensense,icm42631
>+              - invensense,icm42686
>+              - invensense,icm42688
>+    then:
>+      required:
>+        - interrupts
> 
> unevaluatedProperties: false
> 
>-- 
>2.43.0