Re: [PATCH v6 1/6] dt-bindings: iio: imu: icm42600: Add ICM-42670-P

[Krzysztof Kozlowski <krzk@kernel.org>]

On 12/10/2026 12:43, Kanak Shilledar wrote:
> Hi Krzystztof,
> 
> On Mon, 2026-10-12 at 10:58 +0200, Krzysztof Kozlowski wrote:
>> On Fri, Oct 09, 2026 at 04:12:06PM +0200, Kanak Shilledar wrote:
>>>    It has a configurable host interface that supports I3C, I2C and
>>> SPI serial
>>> -  communication, features a 2kB FIFO and 2 programmable interrupts
>>> with
>>> +  communication, features up to 2.25kB FIFO and 2 programmable
>>> interrupts with
>>>    ultra-low-power wake-on-motion support to minimize system power
>>> consumption.
>>>  
>>>    Other industry-leading features include InvenSense on-chip APEX
>>> Motion
>>> @@ -28,6 +29,7 @@ properties:
>>>    compatible:
>>>      oneOf:
>>>        - enum:
>>> +          - invensense,icm42370p
>>>            - invensense,icm42600
>>>            - invensense,icm42602
>>>            - invensense,icm42605
>>
>> Why aren't the interrupts required?
> 
> The interrupts are only required if using the device is used in
> buffered mode. Which will be added later and the dt-binding will be
> updated when the buffer support is added. Currently, the device works
> without any interrupts.
> 

That's misleading or confusing. Either the device works or does not
without interrupt hooked up. Consult hardware datasheet if in doubts.

Driver support is different thing here and you cannot make it required
later, because that would be unjustified ABI break.

Best regards,
Krzysztof