Re: [PATCH v9 1/7] dt-bindings: iio: imu: icm42600: Drop interrupt requirement

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Low] dt-bindings: iio: imu: icm42600: inaccurate Fixes trailer
--

commit bb25ff93be540fe1b4383c714ff7cd075c29001b
Author: Kanak Shilledar <kanak.shilledar@axis.com>

> dt-bindings: iio: imu: icm42600: Drop interrupt requirement
>
> The devicetree documentation for the invensense,icm426xx devices was
> documented wrongly. As per the discussion in the thread[1], the IMUs can
> work in polling mode, making the interrupts as not required.
>
> Fix the dt-binding to not have interrupt property as mandatory. The
> driver will need to be fixed to not fail if interrupts are missing from
> the device tree. This will be done in a separate patch.
>
> Link: https://lore.kernel.org/linux-iio/20261018034134.2bf170a0@jic23-hlaptop/ [1]
> Fixes: 22addefdabdf ("dt-bindings: iio: imu: icm42600: Add icm42607")
> Signed-off-by: Kanak Shilledar <kanak.shilledar@axis.com>

[Severity: Low]
Commit 22addefdabdf ("dt-bindings: iio: imu: icm42600: Add icm42607") only
moved the existing top-level interrupts requirement into an allOf block to
exempt icm42607.  Should the Fixes tag instead point to the commit that
originally made interrupts required:

  Fixes: 45924b8fd2c9 ("dt-bindings: iio: imu: Add inv_icm42600 documentation")

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261112-b4-inv_icm42370p-v9-0-875c828a281e@axis.com?part=1