linux-kernel - RE: [PATCH v2 1/8] dt-bindings: iio: imu: Add inv

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID:
 <FR2PPF4571F02BCD22C5EF19953DF32A8688C30A@FR2PPF4571F02BC.DEUP281.PROD.OUTLOOK.COM>
Date: Tue, 19 Aug 2025 14:53:50 +0000
From: Remi Buisson <Remi.Buisson@....com>
To: Krzysztof Kozlowski <krzk@...nel.org>,
        Jonathan Cameron
	<jic23@...nel.org>,
        David Lechner <dlechner@...libre.com>,
        Nuno Sá <nuno.sa@...log.com>,
        Andy Shevchenko
	<andy@...nel.org>, Rob Herring <robh@...nel.org>,
        Krzysztof Kozlowski
	<krzk+dt@...nel.org>,
        Conor Dooley <conor+dt@...nel.org>
CC: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-iio@...r.kernel.org" <linux-iio@...r.kernel.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>
Subject: RE: [PATCH v2 1/8] dt-bindings: iio: imu: Add inv_icm45600

>
>
>From: Krzysztof Kozlowski <krzk@...nel.org> 
>Sent: Tuesday, August 19, 2025 4:29 PM
>To: Remi Buisson <Remi.Buisson@....com>; Jonathan Cameron <jic23@...nel.org>; David Lechner <dlechner@...libre.com>; Nuno Sá <nuno.sa@...log.com>; Andy Shevchenko <andy@...nel.org>; Rob Herring <robh@...nel.org>; Krzysztof Kozlowski <krzk+dt@...nel.org>; Conor Dooley <conor+dt@...nel.org>
>Cc: linux-kernel@...r.kernel.org; linux-iio@...r.kernel.org; devicetree@...r.kernel.org
>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
>