Re: [PATCH v2 2/6] dt-bindings: iio: imu: Add bindings for ADI ADIS16607

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] dt-bindings: commit message claims nonexistent FIFO watermark property
--

commit 46f4e849c03631bf69981fbde007debcca23f628
Author: Radu Sabau <radu.sabau@analog.com>

dt-bindings: iio: imu: Add bindings for ADI ADIS16607

This commit adds device tree bindings for the Analog Devices ADIS16607
Precision MEMS Inertial Measurement Unit. It documents the supported
compatible variants, required SPI properties, and optional properties for
external clocks and reset GPIOs.

>     The binding exposes optional properties for:
>       - An external clock for sync mode (falling back to the internal 8 kHz
>         clock when absent)
>       - A hardware reset GPIO (active low)
>       - FIFO mode for buffered data acquisition with a configurable watermark

[Severity: Medium]
Is this commit message accurate?

The message claims the binding exposes optional properties for a
configurable FIFO watermark, but looking at the properties block in
Documentation/devicetree/bindings/iio/imu/adi,adis16607.yaml, no such
properties are defined.

Because unevaluatedProperties is set to false, if a developer attempts to
configure a FIFO watermark in their device tree node based on this commit
message, it will cause a schema validation failure during dt_binding_check.

Are FIFO watermarks managed dynamically via IIO sysfs rather than statically
in the device tree for this hardware?

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261008-adis16607-v2-0-7c94df80878a@analog.com?part=2