RE: [PATCH v2 1/8] dt-bindings: iio: imu: Add inv_icm45600

[Remi Buisson]

>
>
>From: Krzysztof Kozlowski <krzk@xxxxxxxxxx> 
>Sent: Tuesday, August 19, 2025 4:29 PM
>To: Remi Buisson <Remi.Buisson@xxxxxxx>; Jonathan Cameron <jic23@xxxxxxxxxx>; David Lechner <dlechner@xxxxxxxxxxxx>; Nuno Sá <nuno.sa@xxxxxxxxxx>; Andy Shevchenko <andy@xxxxxxxxxx>; Rob Herring <robh@xxxxxxxxxx>; Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>; Conor Dooley <conor+dt@xxxxxxxxxx>
>Cc: linux-kernel@xxxxxxxxxxxxxxx; linux-iio@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx
>Subject: Re: [PATCH v2 1/8] dt-bindings: iio: imu: Add inv_icm45600
>On 19/08/2025 12:18, Remi Buisson wrote:
>>>> +  interrupt-names:
>>>> +    minItems: 1
>>>> +    maxItems: 2
>>>> +    items:
>>>> +      enum:
>>>> +        - INT1
>>>> +        - INT2
>>>
>> Hello Krzysztof,
>> Could you please help me understand the proposed simplification?
>>> This can be simpler
>>>
>>> minItems: 1
>>> items:
>>>  - enum: [ int1, int2 ]
>> This should be the one-lined "original" list.
>>>  - const: int 2
>> I don't understand what this "int 2" does.
>
>space was a typo, obviously
>
>> Why do we need a const here ?
>
>Because you cannot have here int1. Order of items is fixed.

Thanks for your prompt reply.
I'll fix it in next patch version.

Remi
>
>
>
>Best regards,
>Krzysztof
>