Re: [PATCH V9 02/11] dt-bindings: iio: imu: icm42600: Add icm42607 binding

[Krzysztof Kozlowski <krzk@kernel.org>]

On 01/06/2026 03:44, Chris Morgan wrote:
> On Sat, May 30, 2026 at 09:26:31AM +0200, Krzysztof Kozlowski wrote:
>> On Fri, May 29, 2026 at 10:17:29PM -0500, Chris Morgan wrote:
>>> From: Chris Morgan <macromorgan@hotmail.com>
>>>
>>> Add devicetree binding for the Invensense ICM42607 and Invensense
>>> ICM42607P inertial measurement unit. This unit is a combined
>>> accelerometer, gyroscope, and thermometer available via I2C or SPI.
>>
>> Implement feedback from v3. Subject is still incorrect.
>>
> 
> I'm sorry, that was dropped due to an oversight. I'll correct it.
> 
>>>
>>> This device is functionally very similar to the icm42600 series with a
>>> very different register layout, however unlike the icm42600 driver we
>>> require a interrupt-names property be present.
>>
>> Why do we require it? Who needs it?
>>
> 
> The driver this was based off of, the icm42600 driver, has a fallback
> to select the 1st available interrupt in the event of the named

So some other driver? That's not relevant here if you do not use that.

> interrupt not existing. I was told not to do that here so the named
> interrupt is a requirement for the icm42607.

So icm42607 driver needs it? Then write it explicitly.

Best regards,
Krzysztof

_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip