From: Remi Buisson <remi.buisson@tdk.com>
Document the ICM-456xxx devices devicetree bindings.
Describe custom sysfs API for controlling the power modes.
Signed-off-by: Remi Buisson <remi.buisson@tdk.com>
---
.../ABI/testing/sysfs-bus-iio-inv_icm45600 | 37 ++++++
.../bindings/iio/imu/invensense,icm45600.yaml | 136 +++++++++++++++++++++
2 files changed, 173 insertions(+)
diff --git a/Documentation/ABI/testing/sysfs-bus-iio-inv_icm45600 b/Documentation/ABI/testing/sysfs-bus-iio-inv_icm45600
new file mode 100644
index 0000000000000000000000000000000000000000..8d2d9b68ad9e35fe0d6c157e984afc327eab92ec
--- /dev/null
+++ b/Documentation/ABI/testing/sysfs-bus-iio-inv_icm45600
@@ -0,0 +1,37 @@
+What: /sys/bus/iio/devices/iio:deviceX/in_accel_power_mode
+KernelVersion: 6.16
+Contact: linux-iio@vger.kernel.org
+Description:
+ Accelerometer power mode. Setting this attribute will set the
+ requested power mode to use if the ODR support it. If ODR
+ support only 1 mode, power mode will be enforced.
+ Reading this attribute will return the current accelerometer
+ power mode if the sensor is on, or the requested value if the
+ sensor is off. The value between real and requested value can
+ be different for ODR supporting only 1 mode.
+
+What: /sys/bus/iio/devices/iio:deviceX/in_accel_power_mode_available
+KernelVersion: 6.16
+Contact: linux-iio@vger.kernel.org
+Description:
+ List of available accelerometer power modes that can be set in
+ in_accel_power_mode attribute.
+
+What: /sys/bus/iio/devices/iio:deviceX/in_anglvel_power_mode
+KernelVersion: 6.16
+Contact: linux-iio@vger.kernel.org
+Description:
+ Gyroscope power mode. Setting this attribute will set the
+ requested power mode to use if the ODR support it. If ODR
+ support only 1 mode, power mode will be enforced.
+ Reading this attribute will return the current gyroscope
+ power mode if the sensor is on, or the requested value if the
+ sensor is off. The value between real and requested value can
+ be different for ODR supporting only 1 mode.
+
+What: /sys/bus/iio/devices/iio:deviceX/in_anglvel_power_mode_available
+KernelVersion: 6.16
+Contact: linux-iio@vger.kernel.org
+Description:
+ List of available gyroscope power modes that can be set in
+ in_anglvel_power_mode attribute.
diff --git a/Documentation/devicetree/bindings/iio/imu/invensense,icm45600.yaml b/Documentation/devicetree/bindings/iio/imu/invensense,icm45600.yaml
new file mode 100644
index 0000000000000000000000000000000000000000..51455f0b5cb90abdd823f154e45891ad364296e6
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/imu/invensense,icm45600.yaml
@@ -0,0 +1,136 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/imu/invensense,icm45600.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: InvenSense ICM-456xx Inertial Measurement Unit
+
+maintainers:
+ - Remi Buisson <remi.buisson@tdk.com>
+
+description: |
+ 6-axis MotionTracking device that combines a 3-axis gyroscope and a 3-axis
+ accelerometer.
+
+ It has a configurable host interface that supports I3C, I2C and SPI serial
+ communication, features up to 8kB 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
+ Processing engine for gesture recognition, activity classification, and
+ pedometer, along with programmable digital filters, and an embedded
+ temperature sensor.
+
+ https://invensense.tdk.com/wp-content/uploads/documentation/DS-000576_ICM-45605.pdf
+
+properties:
+ compatible:
+ enum:
+ - invensense,icm45605
+ - invensense,icm45686
+ - invensense,icm45688p
+ - invensense,icm45608
+ - invensense,icm45634
+ - invensense,icm45689
+ - invensense,icm45606
+ - invensense,icm45687
+
+ reg:
+ maxItems: 1
+
+ interrupts:
+ minItems: 1
+ maxItems: 2
+
+ interrupt-names:
+ minItems: 1
+ maxItems: 2
+ items:
+ enum:
+ - INT1
+ - INT2
+ description: Choose chip interrupt pin to be used as interrupt input.
+
+ drive-open-drain:
+ type: boolean
+
+ vdd-supply:
+ description: Regulator that provides power to the sensor
+
+ vddio-supply:
+ description: Regulator that provides power to the bus
+
+required:
+ - compatible
+ - reg
+ - interrupts
+ - interrupt-names
+
+allOf:
+ - $ref: /schemas/spi/spi-peripheral-props.yaml#
+
+unevaluatedProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/gpio/gpio.h>
+ #include <dt-bindings/interrupt-controller/irq.h>
+ i2c {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ icm45605@68 {
+ compatible = "invensense,icm45605";
+ reg = <0x68>;
+ interrupt-parent = <&gpio2>;
+ interrupt-names = "INT1";
+ interrupts = <7 IRQ_TYPE_EDGE_RAISING>;
+ vdd-supply = <&vdd>;
+ vddio-supply = <&vddio>;
+ mount-matrix = "1", "0", "0",
+ "0", "1", "0",
+ "0", "0", "1";
+ };
+ };
+ - |
+ #include <dt-bindings/gpio/gpio.h>
+ #include <dt-bindings/interrupt-controller/irq.h>
+ spi {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ icm45605@0 {
+ compatible = "invensense,icm45605";
+ reg = <0>;
+ spi-max-frequency = <24000000>;
+ interrupt-parent = <&gpio1>;
+ interrupt-names = "INT1";
+ interrupts = <6 IRQ_TYPE_EDGE_RAISING>;
+ vdd-supply = <&vdd>;
+ vddio-supply = <&vddio>;
+ mount-matrix = "1", "0", "0",
+ "0", "1", "0",
+ "0", "0", "1";
+ };
+ };
+ - |
+ #include <dt-bindings/gpio/gpio.h>
+ #include <dt-bindings/interrupt-controller/irq.h>
+ i3c {
+ #address-cells = <3>;
+ #size-cells = <0>;
+
+ icm45600@68,46A00000011 {
+ compatible = "invensense,icm45600";
+ reg = <0x68 0x46A 0x84>;
+ interrupt-parent = <&gpio1>;
+ interrupt-names = "INT1";
+ interrupts = <5 IRQ_TYPE_EDGE_RISING>;
+ vdd-supply = <&vdd>;
+ vddio-supply = <&vddio>;
+ mount-matrix = "1", "0", "0",
+ "0", "1", "0",
+ "0", "0", "1";
+ };
+ };
--
2.34.1On Fri, 11 Apr 2025 13:28:39 +0000
Remi Buisson via B4 Relay <devnull+remi.buisson.tdk.com@kernel.org> wrote:
> From: Remi Buisson <remi.buisson@tdk.com>
>
> Document the ICM-456xxx devices devicetree bindings.
> Describe custom sysfs API for controlling the power modes.
>
> Signed-off-by: Remi Buisson <remi.buisson@tdk.com>
> ---
> .../ABI/testing/sysfs-bus-iio-inv_icm45600 | 37 ++++++
> .../bindings/iio/imu/invensense,icm45600.yaml | 136 +++++++++++++++++++++
> 2 files changed, 173 insertions(+)
>
> diff --git a/Documentation/ABI/testing/sysfs-bus-iio-inv_icm45600 b/Documentation/ABI/testing/sysfs-bus-iio-inv_icm45600
> new file mode 100644
> index 0000000000000000000000000000000000000000..8d2d9b68ad9e35fe0d6c157e984afc327eab92ec
> --- /dev/null
> +++ b/Documentation/ABI/testing/sysfs-bus-iio-inv_icm45600
> @@ -0,0 +1,37 @@
As has been noted, wrong place. This goes in the patch with the relevant ABI
being added to the driver.
> +What: /sys/bus/iio/devices/iio:deviceX/in_accel_power_mode
> +KernelVersion: 6.16
> +Contact: linux-iio@vger.kernel.org
> +Description:
> + Accelerometer power mode. Setting this attribute will set the
> + requested power mode to use if the ODR support it. If ODR
> + support only 1 mode, power mode will be enforced.
If the mode id enforced, then the ODR will change? I hope not. ODR should dominate here
though note you should match ABI terms anyway so sampling_frequency.
Also as noted, these already exist in some form in the main docs. Add entries
there.
> + Reading this attribute will return the current accelerometer
> + power mode if the sensor is on, or the requested value if the
> + sensor is off. The value between real and requested value can
> + be different for ODR supporting only 1 mode.
I'd just fail the set. If the ODR is changed in a fashion that requires this
to change, just do it and have this attribute return the new value. All ABI
is allowed when necessary to have side effects on other IIO ABI elements.
We try to keep that as intuitive as possible, but sometimes the hardware
puts very complex requirements on us.
> +
> +What: /sys/bus/iio/devices/iio:deviceX/in_accel_power_mode_available
> +KernelVersion: 6.16
> +Contact: linux-iio@vger.kernel.org
> +Description:
> + List of available accelerometer power modes that can be set in
> + in_accel_power_mode attribute.
> +
> +What: /sys/bus/iio/devices/iio:deviceX/in_anglvel_power_mode
> +KernelVersion: 6.16
> +Contact: linux-iio@vger.kernel.org
> +Description:
> + Gyroscope power mode. Setting this attribute will set the
> + requested power mode to use if the ODR support it. If ODR
> + support only 1 mode, power mode will be enforced.
> + Reading this attribute will return the current gyroscope
> + power mode if the sensor is on, or the requested value if the
> + sensor is off. The value between real and requested value can
> + be different for ODR supporting only 1 mode.
> +
> +What: /sys/bus/iio/devices/iio:deviceX/in_anglvel_power_mode_available
> +KernelVersion: 6.16
> +Contact: linux-iio@vger.kernel.org
> +Description:
> + List of available gyroscope power modes that can be set in
> + in_anglvel_power_mode attribute.
> diff --git a/Documentation/devicetree/bindings/iio/imu/invensense,icm45600.yaml b/Documentation/devicetree/bindings/iio/imu/invensense,icm45600.yaml
> new file mode 100644
> index 0000000000000000000000000000000000000000..51455f0b5cb90abdd823f154e45891ad364296e6
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/imu/invensense,icm45600.yaml
> @@ -0,0 +1,136 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/iio/imu/invensense,icm45600.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: InvenSense ICM-456xx Inertial Measurement Unit
I'd go with Invensense ICM-45600 and similar Inertial Measurement Units.
We have been bitten too many times in the past by wild cards and manufacturers
who love to put something in a numbering scheme for marketing reasons (or just
because they feel like it) rather than because they are in any way related
at the hardware level.
> +
> +maintainers:
> + - Remi Buisson <remi.buisson@tdk.com>
> +
> +description: |
> + 6-axis MotionTracking device that combines a 3-axis gyroscope and a 3-axis
> + accelerometer.
> +
> + It has a configurable host interface that supports I3C, I2C and SPI serial
> + communication, features up to 8kB 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
> + Processing engine for gesture recognition, activity classification, and
> + pedometer, along with programmable digital filters, and an embedded
> + temperature sensor.
> +
> + https://invensense.tdk.com/wp-content/uploads/documentation/DS-000576_ICM-45605.pdf
> +
> +properties:
> + compatible:
> + enum:
> + - invensense,icm45605
Alpha numeric order for these.
> + - invensense,icm45686
> + - invensense,icm45688p
> + - invensense,icm45608
> + - invensense,icm45634
> + - invensense,icm45689
> + - invensense,icm45606
> + - invensense,icm45687
> +
> + reg:
> + maxItems: 1
> +
> + interrupts:
> + minItems: 1
> + maxItems: 2
> +
> + interrupt-names:
> + minItems: 1
> + maxItems: 2
> + items:
> + enum:
> + - INT1
> + - INT2
> + description: Choose chip interrupt pin to be used as interrupt input.
> +
> + drive-open-drain:
> + type: boolean
> +
> + vdd-supply:
> + description: Regulator that provides power to the sensor
For really 'standard' supplies like these it is also acceptable to just
do
vdd-supply: true
vddio-supply: true
When description adds little there is no need to give one.
> +
> + vddio-supply:
> + description: Regulator that provides power to the bus
> +
I think you need
mount-matrix: true
as well.
> +required:
> + - compatible
> + - reg
> + - interrupts
> + - interrupt-names
> +
> +allOf:
> + - $ref: /schemas/spi/spi-peripheral-props.yaml#
> +
> +unevaluatedProperties: false
> +
> +examples:
> + - |
> + #include <dt-bindings/gpio/gpio.h>
> + #include <dt-bindings/interrupt-controller/irq.h>
> + i2c {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + icm45605@68 {
> + compatible = "invensense,icm45605";
> + reg = <0x68>;
> + interrupt-parent = <&gpio2>;
> + interrupt-names = "INT1";
> + interrupts = <7 IRQ_TYPE_EDGE_RAISING>;
> + vdd-supply = <&vdd>;
> + vddio-supply = <&vddio>;
> + mount-matrix = "1", "0", "0",
> + "0", "1", "0",
> + "0", "0", "1";
> + };
> + };
> + - |
> + #include <dt-bindings/gpio/gpio.h>
> + #include <dt-bindings/interrupt-controller/irq.h>
> + spi {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + icm45605@0 {
> + compatible = "invensense,icm45605";
> + reg = <0>;
> + spi-max-frequency = <24000000>;
> + interrupt-parent = <&gpio1>;
> + interrupt-names = "INT1";
> + interrupts = <6 IRQ_TYPE_EDGE_RAISING>;
> + vdd-supply = <&vdd>;
> + vddio-supply = <&vddio>;
> + mount-matrix = "1", "0", "0",
> + "0", "1", "0",
> + "0", "0", "1";
> + };
> + };
> + - |
> + #include <dt-bindings/gpio/gpio.h>
> + #include <dt-bindings/interrupt-controller/irq.h>
> + i3c {
> + #address-cells = <3>;
> + #size-cells = <0>;
> +
> + icm45600@68,46A00000011 {
> + compatible = "invensense,icm45600";
> + reg = <0x68 0x46A 0x84>;
> + interrupt-parent = <&gpio1>;
> + interrupt-names = "INT1";
> + interrupts = <5 IRQ_TYPE_EDGE_RISING>;
> + vdd-supply = <&vdd>;
> + vddio-supply = <&vddio>;
> + mount-matrix = "1", "0", "0",
> + "0", "1", "0",
> + "0", "0", "1";
> + };
> + };
>
On 4/11/25 8:28 AM, Remi Buisson via B4 Relay wrote: > From: Remi Buisson <remi.buisson@tdk.com> > > Document the ICM-456xxx devices devicetree bindings. > Describe custom sysfs API for controlling the power modes. > > Signed-off-by: Remi Buisson <remi.buisson@tdk.com> > --- IMHO, it is more logical to have the dt-bindings patch first in the series before the code that uses it. > .../ABI/testing/sysfs-bus-iio-inv_icm45600 | 37 ++++++ > .../bindings/iio/imu/invensense,icm45600.yaml | 136 +++++++++++++++++++++ > 2 files changed, 173 insertions(+) > > diff --git a/Documentation/ABI/testing/sysfs-bus-iio-inv_icm45600 b/Documentation/ABI/testing/sysfs-bus-iio-inv_icm45600 > new file mode 100644 > index 0000000000000000000000000000000000000000..8d2d9b68ad9e35fe0d6c157e984afc327eab92ec > --- /dev/null > +++ b/Documentation/ABI/testing/sysfs-bus-iio-inv_icm45600 ABI documentation is separate from dt-bindings and needs to go in a different patch. Also, it looks like /sys/.../iio:deviceX/in_accelY_power_mode is already a standard attribute in Documentation/ABI/testing/sysfs-bus-iio so we could add to that instead of creating a new file. And there is Documentation/ABI/testing/sysfs-bus-iio-inv_icm42600 that has the same attribute essentially. So it would be good to delete this file and consolidate everything in the main file. > diff --git a/Documentation/devicetree/bindings/iio/imu/invensense,icm45600.yaml b/Documentation/devicetree/bindings/iio/imu/invensense,icm45600.yaml > new file mode 100644 > index 0000000000000000000000000000000000000000..51455f0b5cb90abdd823f154e45891ad364296e6 > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/imu/invensense,icm45600.yaml > @@ -0,0 +1,136 @@ ... > +allOf: > + - $ref: /schemas/spi/spi-peripheral-props.yaml# Since this can be connected to different buses, I don't think we want to always ref this. It gets included implicitly for all child nodes on a spi controller node anyway.
On Fri, 11 Apr 2025 16:18:39 -0500 David Lechner <dlechner@baylibre.com> wrote: > On 4/11/25 8:28 AM, Remi Buisson via B4 Relay wrote: > > From: Remi Buisson <remi.buisson@tdk.com> > > > > Document the ICM-456xxx devices devicetree bindings. > > Describe custom sysfs API for controlling the power modes. > > > > Signed-off-by: Remi Buisson <remi.buisson@tdk.com> > > --- > > IMHO, it is more logical to have the dt-bindings patch first in the series > before the code that uses it. > > > .../ABI/testing/sysfs-bus-iio-inv_icm45600 | 37 ++++++ > > .../bindings/iio/imu/invensense,icm45600.yaml | 136 +++++++++++++++++++++ > > 2 files changed, 173 insertions(+) > > > > diff --git a/Documentation/ABI/testing/sysfs-bus-iio-inv_icm45600 b/Documentation/ABI/testing/sysfs-bus-iio-inv_icm45600 > > new file mode 100644 > > index 0000000000000000000000000000000000000000..8d2d9b68ad9e35fe0d6c157e984afc327eab92ec > > --- /dev/null > > +++ b/Documentation/ABI/testing/sysfs-bus-iio-inv_icm45600 > > > ABI documentation is separate from dt-bindings and needs to go in a > different patch. > > Also, it looks like /sys/.../iio:deviceX/in_accelY_power_mode is > already a standard attribute in Documentation/ABI/testing/sysfs-bus-iio > so we could add to that instead of creating a new file. Just a quick side note. Those powermode interfaces are hard for userspace to reason about, so where possible it is better to set power mode in response to more explicit demands such as sampling frequency. I can't remember when I got persuaded to let that one in. I checked - long ago and we still only have 2 users :) > > And there is Documentation/ABI/testing/sysfs-bus-iio-inv_icm42600 > that has the same attribute essentially. So it would be good to > delete this file and consolidate everything in the main file. > > > diff --git a/Documentation/devicetree/bindings/iio/imu/invensense,icm45600.yaml b/Documentation/devicetree/bindings/iio/imu/invensense,icm45600.yaml > > new file mode 100644 > > index 0000000000000000000000000000000000000000..51455f0b5cb90abdd823f154e45891ad364296e6 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/iio/imu/invensense,icm45600.yaml > > @@ -0,0 +1,136 @@ > > ... > > > +allOf: > > + - $ref: /schemas/spi/spi-peripheral-props.yaml# > > Since this can be connected to different buses, I don't think we want > to always ref this. It gets included implicitly for all child nodes on a > spi controller node anyway. >
On Fri, Apr 11, 2025 at 01:28:39PM +0000, Remi Buisson wrote:
> Document the ICM-456xxx devices devicetree bindings.
> Describe custom sysfs API for controlling the power modes.
You can drop 'documentation' from the subject.
>
> Signed-off-by: Remi Buisson <remi.buisson@tdk.com>
> ---
> .../ABI/testing/sysfs-bus-iio-inv_icm45600 | 37 ++++++
This goes in the patch adding the sysfs files.
> .../bindings/iio/imu/invensense,icm45600.yaml | 136 +++++++++++++++++++++
> 2 files changed, 173 insertions(+)
> diff --git a/Documentation/devicetree/bindings/iio/imu/invensense,icm45600.yaml b/Documentation/devicetree/bindings/iio/imu/invensense,icm45600.yaml
> new file mode 100644
> index 0000000000000000000000000000000000000000..51455f0b5cb90abdd823f154e45891ad364296e6
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/imu/invensense,icm45600.yaml
> @@ -0,0 +1,136 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/iio/imu/invensense,icm45600.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: InvenSense ICM-456xx Inertial Measurement Unit
> +
> +maintainers:
> + - Remi Buisson <remi.buisson@tdk.com>
> +
> +description: |
> + 6-axis MotionTracking device that combines a 3-axis gyroscope and a 3-axis
> + accelerometer.
> +
> + It has a configurable host interface that supports I3C, I2C and SPI serial
> + communication, features up to 8kB 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
> + Processing engine for gesture recognition, activity classification, and
> + pedometer, along with programmable digital filters, and an embedded
> + temperature sensor.
> +
> + https://invensense.tdk.com/wp-content/uploads/documentation/DS-000576_ICM-45605.pdf
> +
> +properties:
> + compatible:
> + enum:
> + - invensense,icm45605
> + - invensense,icm45686
> + - invensense,icm45688p
> + - invensense,icm45608
> + - invensense,icm45634
> + - invensense,icm45689
> + - invensense,icm45606
> + - invensense,icm45687
sort in alpanumeric order
> +
> + reg:
> + maxItems: 1
> +
> + interrupts:
> + minItems: 1
> + maxItems: 2
> +
> + interrupt-names:
> + minItems: 1
> + maxItems: 2
> + items:
> + enum:
> + - INT1
> + - INT2
> + description: Choose chip interrupt pin to be used as interrupt input.
> +
> + drive-open-drain:
> + type: boolean
> +
> + vdd-supply:
> + description: Regulator that provides power to the sensor
> +
> + vddio-supply:
> + description: Regulator that provides power to the bus
> +
> +required:
> + - compatible
> + - reg
> + - interrupts
> + - interrupt-names
> +
> +allOf:
> + - $ref: /schemas/spi/spi-peripheral-props.yaml#
> +
> +unevaluatedProperties: false
> +
> +examples:
> + - |
> + #include <dt-bindings/gpio/gpio.h>
> + #include <dt-bindings/interrupt-controller/irq.h>
> + i2c {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + icm45605@68 {
> + compatible = "invensense,icm45605";
> + reg = <0x68>;
> + interrupt-parent = <&gpio2>;
> + interrupt-names = "INT1";
> + interrupts = <7 IRQ_TYPE_EDGE_RAISING>;
> + vdd-supply = <&vdd>;
> + vddio-supply = <&vddio>;
> + mount-matrix = "1", "0", "0",
> + "0", "1", "0",
> + "0", "0", "1";
> + };
> + };
> + - |
> + #include <dt-bindings/gpio/gpio.h>
> + #include <dt-bindings/interrupt-controller/irq.h>
> + spi {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + icm45605@0 {
> + compatible = "invensense,icm45605";
> + reg = <0>;
> + spi-max-frequency = <24000000>;
> + interrupt-parent = <&gpio1>;
> + interrupt-names = "INT1";
> + interrupts = <6 IRQ_TYPE_EDGE_RAISING>;
> + vdd-supply = <&vdd>;
> + vddio-supply = <&vddio>;
> + mount-matrix = "1", "0", "0",
> + "0", "1", "0",
> + "0", "0", "1";
> + };
> + };
> + - |
> + #include <dt-bindings/gpio/gpio.h>
> + #include <dt-bindings/interrupt-controller/irq.h>
> + i3c {
> + #address-cells = <3>;
> + #size-cells = <0>;
> +
> + icm45600@68,46A00000011 {
> + compatible = "invensense,icm45600";
> + reg = <0x68 0x46A 0x84>;
> + interrupt-parent = <&gpio1>;
> + interrupt-names = "INT1";
> + interrupts = <5 IRQ_TYPE_EDGE_RISING>;
> + vdd-supply = <&vdd>;
> + vddio-supply = <&vddio>;
> + mount-matrix = "1", "0", "0",
> + "0", "1", "0",
> + "0", "0", "1";
> + };
> + };
>
> --
> 2.34.1
>
On Fri, 11 Apr 2025 13:28:39 +0000, Remi Buisson wrote: > Document the ICM-456xxx devices devicetree bindings. > Describe custom sysfs API for controlling the power modes. > > Signed-off-by: Remi Buisson <remi.buisson@tdk.com> > --- > .../ABI/testing/sysfs-bus-iio-inv_icm45600 | 37 ++++++ > .../bindings/iio/imu/invensense,icm45600.yaml | 136 +++++++++++++++++++++ > 2 files changed, 173 insertions(+) > My bot found errors running 'make dt_binding_check' on your patch: yamllint warnings/errors: dtschema/dtc warnings/errors: Error: Documentation/devicetree/bindings/iio/imu/invensense,icm45600.example.dts:35.33-34 syntax error FATAL ERROR: Unable to parse input tree make[2]: *** [scripts/Makefile.dtbs:131: Documentation/devicetree/bindings/iio/imu/invensense,icm45600.example.dtb] Error 1 make[2]: *** Waiting for unfinished jobs.... make[1]: *** [/builds/robherring/dt-review-ci/linux/Makefile:1525: dt_binding_check] Error 2 make: *** [Makefile:248: __sub-make] Error 2 doc reference errors (make refcheckdocs): [WARNING] /sys/bus/iio/devices/iio:deviceX/in_accel_power_mode is defined 2 times: ./Documentation/ABI/testing/sysfs-bus-iio-inv_icm42600:1; ./Documentation/ABI/testing/sysfs-bus-iio-inv_icm45600:1 [WARNING] /sys/bus/iio/devices/iio:deviceX/in_accel_power_mode_available is defined 2 times: ./Documentation/ABI/testing/sysfs-bus-iio-inv_icm42600:13; ./Documentation/ABI/testing/sysfs-bus-iio-inv_icm45600:13 See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20250411-add_newport_driver-v1-7-15082160b019@tdk.com The base for the series is generally the latest rc1. A different dependency should be noted in *this* patch. If you already ran 'make dt_binding_check' and didn't see the above error(s), then make sure 'yamllint' is installed and dt-schema is up to date: pip3 install dtschema --upgrade Please check and re-submit after running the above command yourself. Note that DT_SCHEMA_FILES can be set to your schema file to speed up checking your schema. However, it must be unset to test all examples with your schema.
© 2016 - 2025 Red Hat, Inc.