2023-01-18 18:52:42

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 1/5] dt-bindings: iio: drop unneeded quotes

Cleanup by removing unneeded quotes from refs and redundant blank lines.
No functional impact except adjusting to preferred coding style.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
.../devicetree/bindings/iio/accel/memsensing,msa311.yaml | 5 ++---
Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml | 2 +-
Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml | 2 +-
.../devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml | 2 +-
Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml | 4 ++--
.../devicetree/bindings/iio/adc/ingenic,adc.yaml | 4 ++--
.../devicetree/bindings/iio/adc/microchip,mcp3911.yaml | 4 ++--
.../devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml | 2 +-
.../devicetree/bindings/iio/adc/samsung,exynos-adc.yaml | 2 +-
.../devicetree/bindings/iio/adc/st,stm32-adc.yaml | 8 ++++----
.../devicetree/bindings/iio/adc/ti,ads131e08.yaml | 2 +-
Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml | 2 +-
.../devicetree/bindings/iio/dac/lltc,ltc1660.yaml | 4 ++--
.../devicetree/bindings/iio/dac/lltc,ltc2632.yaml | 4 ++--
.../devicetree/bindings/iio/dac/st,stm32-dac.yaml | 4 ++--
Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml | 2 +-
.../devicetree/bindings/iio/temperature/ti,tmp117.yaml | 6 +++---
17 files changed, 29 insertions(+), 30 deletions(-)

diff --git a/Documentation/devicetree/bindings/iio/accel/memsensing,msa311.yaml b/Documentation/devicetree/bindings/iio/accel/memsensing,msa311.yaml
index 23528dcaa073..d530ec041fe7 100644
--- a/Documentation/devicetree/bindings/iio/accel/memsensing,msa311.yaml
+++ b/Documentation/devicetree/bindings/iio/accel/memsensing,msa311.yaml
@@ -1,9 +1,8 @@
# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
-
%YAML 1.2
---
-$id: "http://devicetree.org/schemas/iio/accel/memsensing,msa311.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/iio/accel/memsensing,msa311.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#

title: MEMSensing digital 3-Axis accelerometer

diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
index 75a7184a4735..35ed04350e28 100644
--- a/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
@@ -61,7 +61,7 @@ required:

patternProperties:
"^channel@([0-9]|1[0-5])$":
- $ref: "adc.yaml"
+ $ref: adc.yaml
type: object
description: |
Represents the external channels which are connected to the ADC.
diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
index 1bfbeed6f299..7cc4ddc4e9b7 100644
--- a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
@@ -43,7 +43,7 @@ required:

patternProperties:
"^channel@[0-7]$":
- $ref: "adc.yaml"
+ $ref: adc.yaml
type: object
description: |
Represents the external channels which are connected to the ADC.
diff --git a/Documentation/devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml b/Documentation/devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml
index 31f840d59303..4817b840977a 100644
--- a/Documentation/devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml
@@ -41,7 +41,7 @@ properties:
description: Startup time expressed in ms, it depends on SoC.

atmel,trigger-edge-type:
- $ref: '/schemas/types.yaml#/definitions/uint32'
+ $ref: /schemas/types.yaml#/definitions/uint32
description:
One of possible edge types for the ADTRG hardware trigger pin.
When the specific edge type is detected, the conversion will
diff --git a/Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml b/Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml
index 77605f17901c..9c57eb13f892 100644
--- a/Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml
@@ -1,8 +1,8 @@
# SPDX-License-Identifier: GPL-2.0
%YAML 1.2
---
-$id: "http://devicetree.org/schemas/iio/adc/avia-hx711.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/iio/adc/avia-hx711.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#

title: AVIA HX711 ADC chip for weight cells

diff --git a/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml b/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml
index 517e8b1fcb73..b71c951e6d02 100644
--- a/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml
@@ -2,8 +2,8 @@
# Copyright 2019-2020 Artur Rojek
%YAML 1.2
---
-$id: "http://devicetree.org/schemas/iio/adc/ingenic,adc.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/iio/adc/ingenic,adc.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#

title: Ingenic JZ47xx ADC controller IIO

diff --git a/Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml b/Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
index 2c93fb41f172..f7b3fde4115a 100644
--- a/Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
@@ -2,8 +2,8 @@
# Copyright 2019 Marcus Folkesson <[email protected]>
%YAML 1.2
---
-$id: "http://devicetree.org/schemas/iio/adc/microchip,mcp3911.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/iio/adc/microchip,mcp3911.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#

title: Microchip MCP3911 Dual channel analog front end (ADC)

diff --git a/Documentation/devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml b/Documentation/devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml
index 8b743742a5f9..ba86c7b7d622 100644
--- a/Documentation/devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml
@@ -69,7 +69,7 @@ required:

patternProperties:
"^channel@[0-7]$":
- $ref: "adc.yaml"
+ $ref: adc.yaml
type: object
description: |
Represents the external channels which are connected to the ADC.
diff --git a/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml b/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml
index 81c87295912c..e27d094cfa05 100644
--- a/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml
@@ -52,7 +52,7 @@ properties:
vdd-supply: true

samsung,syscon-phandle:
- $ref: '/schemas/types.yaml#/definitions/phandle'
+ $ref: /schemas/types.yaml#/definitions/phandle
description:
Phandle to the PMU system controller node (to access the ADC_PHY
register on Exynos3250/4x12/5250/5420/5800).
diff --git a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
index 1c340c95df16..995cbf8cefc6 100644
--- a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
@@ -1,8 +1,8 @@
# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
%YAML 1.2
---
-$id: "http://devicetree.org/schemas/iio/adc/st,stm32-adc.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/iio/adc/st,stm32-adc.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#

title: STMicroelectronics STM32 ADC

@@ -80,7 +80,7 @@ properties:
description:
Phandle to system configuration controller. It can be used to control the
analog circuitry on stm32mp1.
- $ref: "/schemas/types.yaml#/definitions/phandle-array"
+ $ref: /schemas/types.yaml#/definitions/phandle-array

interrupt-controller: true

@@ -341,7 +341,7 @@ patternProperties:
patternProperties:
"^channel@([0-9]|1[0-9])$":
type: object
- $ref: "adc.yaml"
+ $ref: adc.yaml
description: Represents the external channels which are connected to the ADC.

properties:
diff --git a/Documentation/devicetree/bindings/iio/adc/ti,ads131e08.yaml b/Documentation/devicetree/bindings/iio/adc/ti,ads131e08.yaml
index 55c2c73626f4..890f125d422c 100644
--- a/Documentation/devicetree/bindings/iio/adc/ti,ads131e08.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/ti,ads131e08.yaml
@@ -77,7 +77,7 @@ required:

patternProperties:
"^channel@([0-7])$":
- $ref: "adc.yaml"
+ $ref: adc.yaml
type: object
description: |
Represents the external channels which are connected to the ADC.
diff --git a/Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml b/Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml
index bdf3bba2d750..32c52f9fe18b 100644
--- a/Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml
@@ -41,7 +41,7 @@ required:

patternProperties:
"^channel@[0-7]$":
- $ref: "adc.yaml"
+ $ref: adc.yaml
type: object

properties:
diff --git a/Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml b/Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml
index 133b0f867992..c9f51d00fa8f 100644
--- a/Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml
+++ b/Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml
@@ -2,8 +2,8 @@
# Copyright 2019 Marcus Folkesson <[email protected]>
%YAML 1.2
---
-$id: "http://devicetree.org/schemas/iio/dac/lltc,ltc1660.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/iio/dac/lltc,ltc1660.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#

title: Linear Technology Micropower octal 8-Bit and 10-Bit DACs

diff --git a/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml b/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml
index b1eb77335d05..c9e3be3b5754 100644
--- a/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml
+++ b/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml
@@ -1,8 +1,8 @@
# SPDX-License-Identifier: GPL-2.0 OR BSD-2-Clause
%YAML 1.2
---
-$id: "http://devicetree.org/schemas/iio/dac/lltc,ltc2632.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/iio/dac/lltc,ltc2632.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#

title: Linear Technology LTC263x 12-/10-/8-Bit Rail-to-Rail DAC

diff --git a/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml b/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml
index 0f1bf1110122..04045b932bd2 100644
--- a/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml
+++ b/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml
@@ -1,8 +1,8 @@
# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
%YAML 1.2
---
-$id: "http://devicetree.org/schemas/iio/dac/st,stm32-dac.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/iio/dac/st,stm32-dac.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#

title: STMicroelectronics STM32 DAC

diff --git a/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml b/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
index 68b481c63318..decf022335d8 100644
--- a/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
+++ b/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
@@ -63,7 +63,7 @@ properties:
description: if defined provides VDD IO power to the sensor.

st,drdy-int-pin:
- $ref: '/schemas/types.yaml#/definitions/uint32'
+ $ref: /schemas/types.yaml#/definitions/uint32
description: |
The pin on the package that will be used to signal data ready
enum:
diff --git a/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml b/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml
index 347bc16a4671..c4f1c69f9330 100644
--- a/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml
+++ b/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml
@@ -1,10 +1,10 @@
# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
%YAML 1.2
---
-$id: "http://devicetree.org/schemas/iio/temperature/ti,tmp117.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/iio/temperature/ti,tmp117.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#

-title: "TI TMP117 - Digital temperature sensor with integrated NV memory"
+title: TI TMP117 - Digital temperature sensor with integrated NV memory

description: |
TI TMP117 - Digital temperature sensor with integrated NV memory that supports
--
2.34.1


2023-01-18 18:54:35

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 4/5] dt-bindings: iio: use lowercase hex in examples

Use lowercase hex in addresses in examples.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
.../devicetree/bindings/iio/accel/kionix,kxcjk1013.yaml | 2 +-
.../devicetree/bindings/iio/adc/samsung,exynos-adc.yaml | 2 +-
Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml | 2 +-
.../devicetree/bindings/iio/potentiometer/adi,ad5272.yaml | 2 +-
.../devicetree/bindings/iio/temperature/adi,ltc2983.yaml | 6 +++---
5 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/Documentation/devicetree/bindings/iio/accel/kionix,kxcjk1013.yaml b/Documentation/devicetree/bindings/iio/accel/kionix,kxcjk1013.yaml
index 714e48e613de..6ddb03f61bd9 100644
--- a/Documentation/devicetree/bindings/iio/accel/kionix,kxcjk1013.yaml
+++ b/Documentation/devicetree/bindings/iio/accel/kionix,kxcjk1013.yaml
@@ -44,7 +44,7 @@ examples:

accel@f {
compatible = "kionix,kxtf9";
- reg = <0x0F>;
+ reg = <0xf>;
mount-matrix = "0", "1", "0",
"1", "0", "0",
"0", "0", "1";
diff --git a/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml b/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml
index e27d094cfa05..dca6cfe1e88e 100644
--- a/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml
@@ -150,7 +150,7 @@ examples:

adc@126c0000 {
compatible = "samsung,exynos3250-adc";
- reg = <0x126C0000 0x100>;
+ reg = <0x126c0000 0x100>;
interrupts = <0 137 0>;
#io-channel-cells = <1>;

diff --git a/Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml b/Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml
index 88298bc43b81..79da0323c327 100644
--- a/Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml
+++ b/Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml
@@ -46,7 +46,7 @@ examples:

dac@4c {
compatible = "ti,dac5571";
- reg = <0x4C>;
+ reg = <0x4c>;
vref-supply = <&vdd_supply>;
};
};
diff --git a/Documentation/devicetree/bindings/iio/potentiometer/adi,ad5272.yaml b/Documentation/devicetree/bindings/iio/potentiometer/adi,ad5272.yaml
index 0ebb6725a1af..b8d7083c97f8 100644
--- a/Documentation/devicetree/bindings/iio/potentiometer/adi,ad5272.yaml
+++ b/Documentation/devicetree/bindings/iio/potentiometer/adi,ad5272.yaml
@@ -44,7 +44,7 @@ examples:

potentiometer@2f {
compatible = "adi,ad5272-020";
- reg = <0x2F>;
+ reg = <0x2f>;
reset-gpios = <&gpio3 6 GPIO_ACTIVE_LOW>;
};
};
diff --git a/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml b/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml
index b69813f281da..3e5b7b47bcdc 100644
--- a/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml
+++ b/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml
@@ -519,9 +519,9 @@ examples:
reg = <12>;
adi,sensor-type = <26>; //Steinhart
adi,rsense-handle = <&rsense2>;
- adi,custom-steinhart = <0x00F371EC 0x12345678
- 0x2C0F8733 0x10018C66 0xA0FEACCD
- 0x90021D99>; //6 entries
+ adi,custom-steinhart = <0x00f371ec 0x12345678
+ 0x2c0f8733 0x10018c66 0xa0feaccd
+ 0x90021d99>; //6 entries
};

thermocouple@20 {
--
2.34.1

2023-01-18 18:57:18

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 2/5] dt-bindings: iio: minor whitespace cleanups

Drop redundant blank lines and add such when needed.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
Documentation/devicetree/bindings/iio/dac/adi,ad5380.yaml | 1 +
.../bindings/iio/proximity/google,cros-ec-mkbp-proximity.yaml | 1 -
2 files changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/iio/dac/adi,ad5380.yaml b/Documentation/devicetree/bindings/iio/dac/adi,ad5380.yaml
index ff50c72c62b5..9eb9928500e2 100644
--- a/Documentation/devicetree/bindings/iio/dac/adi,ad5380.yaml
+++ b/Documentation/devicetree/bindings/iio/dac/adi,ad5380.yaml
@@ -12,6 +12,7 @@ maintainers:

description: |
DAC devices supporting both SPI and I2C interfaces.
+
properties:
compatible:
enum:
diff --git a/Documentation/devicetree/bindings/iio/proximity/google,cros-ec-mkbp-proximity.yaml b/Documentation/devicetree/bindings/iio/proximity/google,cros-ec-mkbp-proximity.yaml
index 00e3b59641d2..d4e09d2dcd21 100644
--- a/Documentation/devicetree/bindings/iio/proximity/google,cros-ec-mkbp-proximity.yaml
+++ b/Documentation/devicetree/bindings/iio/proximity/google,cros-ec-mkbp-proximity.yaml
@@ -1,7 +1,6 @@
# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
%YAML 1.2
---
-
$id: http://devicetree.org/schemas/iio/proximity/google,cros-ec-mkbp-proximity.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#

--
2.34.1

2023-01-18 19:13:30

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 3/5] dt-bindings: iio: correct node names in examples

Do not use underscores and unneeded suffixes (e.g. i2c0) in node name in
examples.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
.../devicetree/bindings/iio/accel/adi,adis16201.yaml | 2 +-
.../devicetree/bindings/iio/accel/adi,adis16240.yaml | 2 +-
.../devicetree/bindings/iio/accel/adi,adxl313.yaml | 2 +-
.../devicetree/bindings/iio/accel/adi,adxl345.yaml | 4 ++--
.../devicetree/bindings/iio/accel/bosch,bma220.yaml | 2 +-
.../devicetree/bindings/iio/accel/nxp,fxls8962af.yaml | 4 ++--
Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml | 2 +-
Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml | 2 +-
Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml | 2 +-
.../devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml | 2 +-
.../devicetree/bindings/iio/adc/st,stmpe-adc.yaml | 8 +++-----
.../devicetree/bindings/iio/frequency/adf4371.yaml | 2 +-
.../devicetree/bindings/iio/gyroscope/nxp,fxas21002c.yaml | 4 ++--
.../devicetree/bindings/iio/health/ti,afe4403.yaml | 2 +-
.../devicetree/bindings/iio/health/ti,afe4404.yaml | 2 +-
Documentation/devicetree/bindings/iio/humidity/dht11.yaml | 2 +-
.../devicetree/bindings/iio/humidity/ti,hdc2010.yaml | 2 +-
.../devicetree/bindings/iio/imu/adi,adis16460.yaml | 2 +-
.../devicetree/bindings/iio/imu/invensense,icm42600.yaml | 4 ++--
.../devicetree/bindings/iio/imu/nxp,fxos8700.yaml | 4 ++--
.../devicetree/bindings/iio/pressure/asc,dlhl60d.yaml | 2 +-
.../devicetree/bindings/iio/pressure/bmp085.yaml | 2 +-
.../bindings/iio/temperature/maxim,max31865.yaml | 2 +-
23 files changed, 30 insertions(+), 32 deletions(-)

diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adis16201.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adis16201.yaml
index 7332442e5661..b6ba7ad1a8d5 100644
--- a/Documentation/devicetree/bindings/iio/accel/adi,adis16201.yaml
+++ b/Documentation/devicetree/bindings/iio/accel/adi,adis16201.yaml
@@ -41,7 +41,7 @@ unevaluatedProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/irq.h>
- spi0 {
+ spi {
#address-cells = <1>;
#size-cells = <0>;

diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
index f6f97164c2ca..5887021cc90f 100644
--- a/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
+++ b/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
@@ -39,7 +39,7 @@ examples:
- |
#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/interrupt-controller/irq.h>
- spi0 {
+ spi {
#address-cells = <1>;
#size-cells = <0>;

diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adxl313.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adxl313.yaml
index 185b68ffb536..0c5b64cae965 100644
--- a/Documentation/devicetree/bindings/iio/accel/adi,adxl313.yaml
+++ b/Documentation/devicetree/bindings/iio/accel/adi,adxl313.yaml
@@ -59,7 +59,7 @@ examples:
- |
#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/interrupt-controller/irq.h>
- i2c0 {
+ i2c {
#address-cells = <1>;
#size-cells = <0>;

diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
index 346abfb13a3a..07cacc3f6a97 100644
--- a/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
+++ b/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
@@ -49,7 +49,7 @@ examples:
- |
#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/interrupt-controller/irq.h>
- i2c0 {
+ i2c {
#address-cells = <1>;
#size-cells = <0>;

@@ -64,7 +64,7 @@ examples:
- |
#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/interrupt-controller/irq.h>
- spi0 {
+ spi {
#address-cells = <1>;
#size-cells = <0>;

diff --git a/Documentation/devicetree/bindings/iio/accel/bosch,bma220.yaml b/Documentation/devicetree/bindings/iio/accel/bosch,bma220.yaml
index 5dd06f5905b4..ec643de031a3 100644
--- a/Documentation/devicetree/bindings/iio/accel/bosch,bma220.yaml
+++ b/Documentation/devicetree/bindings/iio/accel/bosch,bma220.yaml
@@ -36,7 +36,7 @@ unevaluatedProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/irq.h>
- spi0 {
+ spi {
#address-cells = <1>;
#size-cells = <0>;

diff --git a/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml b/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml
index 65ce8ea14b52..783c7ddfcd90 100644
--- a/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml
+++ b/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml
@@ -50,7 +50,7 @@ unevaluatedProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/irq.h>
- i2c0 {
+ i2c {
#address-cells = <1>;
#size-cells = <0>;

@@ -65,7 +65,7 @@ examples:
};
- |
#include <dt-bindings/interrupt-controller/irq.h>
- spi0 {
+ spi {
#address-cells = <1>;
#size-cells = <0>;

diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
index cc347dade4ef..1d965124c488 100644
--- a/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
@@ -99,7 +99,7 @@ unevaluatedProperties: false

examples:
- |
- spi0 {
+ spi {
#address-cells = <1>;
#size-cells = <0>;

diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
index ac5a47c8f070..b05652dfd4a2 100644
--- a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
@@ -112,7 +112,7 @@ examples:
- |
#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/interrupt-controller/irq.h>
- spi0 {
+ spi {
#address-cells = <1>;
#size-cells = <0>;

diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
index a67ba67dab51..5fcc8dd012f1 100644
--- a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
@@ -72,7 +72,7 @@ additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
- spi0 {
+ spi {
#address-cells = <1>;
#size-cells = <0>;

diff --git a/Documentation/devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml b/Documentation/devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml
index 18aaf6df179d..73def67fbe01 100644
--- a/Documentation/devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml
@@ -50,7 +50,7 @@ additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/irq.h>
- spmi_bus {
+ spmi {
#address-cells = <1>;
#size-cells = <0>;
pmic_iadc: adc@3600 {
diff --git a/Documentation/devicetree/bindings/iio/adc/st,stmpe-adc.yaml b/Documentation/devicetree/bindings/iio/adc/st,stmpe-adc.yaml
index 333744a2159c..474e35c49348 100644
--- a/Documentation/devicetree/bindings/iio/adc/st,stmpe-adc.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/st,stmpe-adc.yaml
@@ -35,10 +35,8 @@ additionalProperties: false

examples:
- |
- stmpe {
- stmpe_adc {
- compatible = "st,stmpe-adc";
- st,norequest-mask = <0x0F>; /* dont use ADC CH3-0 */
- };
+ adc {
+ compatible = "st,stmpe-adc";
+ st,norequest-mask = <0x0f>; /* dont use ADC CH3-0 */
};
...
diff --git a/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml b/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml
index 0144f74a4768..a0a5e5919987 100644
--- a/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml
+++ b/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml
@@ -53,7 +53,7 @@ unevaluatedProperties: false

examples:
- |
- spi0 {
+ spi {
#address-cells = <1>;
#size-cells = <0>;

diff --git a/Documentation/devicetree/bindings/iio/gyroscope/nxp,fxas21002c.yaml b/Documentation/devicetree/bindings/iio/gyroscope/nxp,fxas21002c.yaml
index 2c900e9dddc6..052dccbb5eea 100644
--- a/Documentation/devicetree/bindings/iio/gyroscope/nxp,fxas21002c.yaml
+++ b/Documentation/devicetree/bindings/iio/gyroscope/nxp,fxas21002c.yaml
@@ -65,7 +65,7 @@ examples:
- |
#include <dt-bindings/interrupt-controller/irq.h>

- i2c0 {
+ i2c {
#address-cells = <1>;
#size-cells = <0>;

@@ -81,7 +81,7 @@ examples:
interrupt-names = "INT1";
};
};
- spi0 {
+ spi {
#address-cells = <1>;
#size-cells = <0>;

diff --git a/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml b/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml
index 6c5ad426a016..12f75ddc4a70 100644
--- a/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml
+++ b/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml
@@ -42,7 +42,7 @@ examples:
#address-cells = <1>;
#size-cells = <0>;

- heart_mon@0 {
+ heart-rate@0 {
compatible = "ti,afe4403";
reg = <0>;
spi-max-frequency = <10000000>;
diff --git a/Documentation/devicetree/bindings/iio/health/ti,afe4404.yaml b/Documentation/devicetree/bindings/iio/health/ti,afe4404.yaml
index c0e815d9999e..b334f3e356ad 100644
--- a/Documentation/devicetree/bindings/iio/health/ti,afe4404.yaml
+++ b/Documentation/devicetree/bindings/iio/health/ti,afe4404.yaml
@@ -39,7 +39,7 @@ examples:
#address-cells = <1>;
#size-cells = <0>;

- heart_mon@58 {
+ heart-rate@58 {
compatible = "ti,afe4404";
reg = <0x58>;
tx-supply = <&vbat>;
diff --git a/Documentation/devicetree/bindings/iio/humidity/dht11.yaml b/Documentation/devicetree/bindings/iio/humidity/dht11.yaml
index 2247481d0203..0103f4238942 100644
--- a/Documentation/devicetree/bindings/iio/humidity/dht11.yaml
+++ b/Documentation/devicetree/bindings/iio/humidity/dht11.yaml
@@ -34,7 +34,7 @@ additionalProperties: false

examples:
- |
- humidity_sensor {
+ humidity-sensor {
compatible = "dht11";
gpios = <&gpio0 6 0>;
};
diff --git a/Documentation/devicetree/bindings/iio/humidity/ti,hdc2010.yaml b/Documentation/devicetree/bindings/iio/humidity/ti,hdc2010.yaml
index 88384b69f917..a36173b0c654 100644
--- a/Documentation/devicetree/bindings/iio/humidity/ti,hdc2010.yaml
+++ b/Documentation/devicetree/bindings/iio/humidity/ti,hdc2010.yaml
@@ -35,7 +35,7 @@ additionalProperties: false

examples:
- |
- i2c0 {
+ i2c {
#address-cells = <1>;
#size-cells = <0>;

diff --git a/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml b/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
index d166dbca18c3..4e43c80e5119 100644
--- a/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
+++ b/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
@@ -42,7 +42,7 @@ examples:
- |
#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/interrupt-controller/irq.h>
- spi0 {
+ spi {
#address-cells = <1>;
#size-cells = <0>;

diff --git a/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml b/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml
index 13c9abdd3131..3ecfb3f086bc 100644
--- a/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml
+++ b/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml
@@ -65,7 +65,7 @@ examples:
- |
#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/interrupt-controller/irq.h>
- i2c0 {
+ i2c {
#address-cells = <1>;
#size-cells = <0>;

@@ -81,7 +81,7 @@ examples:
- |
#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/interrupt-controller/irq.h>
- spi0 {
+ spi {
#address-cells = <1>;
#size-cells = <0>;

diff --git a/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml b/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
index 24416b59b782..db6a145840f7 100644
--- a/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
+++ b/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
@@ -49,7 +49,7 @@ examples:
- |
#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/interrupt-controller/irq.h>
- i2c0 {
+ i2c {
#address-cells = <1>;
#size-cells = <0>;

@@ -65,7 +65,7 @@ examples:
- |
#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/interrupt-controller/irq.h>
- spi0 {
+ spi {
#address-cells = <1>;
#size-cells = <0>;

diff --git a/Documentation/devicetree/bindings/iio/pressure/asc,dlhl60d.yaml b/Documentation/devicetree/bindings/iio/pressure/asc,dlhl60d.yaml
index 1f9fe15b4b3c..9fb8d773efa3 100644
--- a/Documentation/devicetree/bindings/iio/pressure/asc,dlhl60d.yaml
+++ b/Documentation/devicetree/bindings/iio/pressure/asc,dlhl60d.yaml
@@ -39,7 +39,7 @@ examples:
- |
#include <dt-bindings/interrupt-controller/irq.h>

- i2c0 {
+ i2c {
#address-cells = <1>;
#size-cells = <0>;

diff --git a/Documentation/devicetree/bindings/iio/pressure/bmp085.yaml b/Documentation/devicetree/bindings/iio/pressure/bmp085.yaml
index 72cd2c2d3f17..256f537840e4 100644
--- a/Documentation/devicetree/bindings/iio/pressure/bmp085.yaml
+++ b/Documentation/devicetree/bindings/iio/pressure/bmp085.yaml
@@ -60,7 +60,7 @@ examples:
- |
#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/interrupt-controller/irq.h>
- i2c0 {
+ i2c {
#address-cells = <1>;
#size-cells = <0>;
pressure@77 {
diff --git a/Documentation/devicetree/bindings/iio/temperature/maxim,max31865.yaml b/Documentation/devicetree/bindings/iio/temperature/maxim,max31865.yaml
index a2823ed6867b..66394e17fcc8 100644
--- a/Documentation/devicetree/bindings/iio/temperature/maxim,max31865.yaml
+++ b/Documentation/devicetree/bindings/iio/temperature/maxim,max31865.yaml
@@ -43,7 +43,7 @@ examples:
#address-cells = <1>;
#size-cells = <0>;

- temp_sensor@0 {
+ temperature-sensor@0 {
compatible = "maxim,max31865";
reg = <0>;
spi-max-frequency = <400000>;
--
2.34.1

2023-01-19 20:42:19

by Dmitry Rokosov

[permalink] [raw]
Subject: Re: [PATCH 1/5] dt-bindings: iio: drop unneeded quotes

Dear Krzysztof,

Thank you for the patch!
I'm okay with changes to memsensing,msa311.yaml schema.

On Wed, Jan 18, 2023 at 07:44:09PM +0100, Krzysztof Kozlowski wrote:
> Cleanup by removing unneeded quotes from refs and redundant blank lines.
> No functional impact except adjusting to preferred coding style.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> .../devicetree/bindings/iio/accel/memsensing,msa311.yaml | 5 ++---
> Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml | 2 +-
> .../devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml | 4 ++--
> .../devicetree/bindings/iio/adc/ingenic,adc.yaml | 4 ++--
> .../devicetree/bindings/iio/adc/microchip,mcp3911.yaml | 4 ++--
> .../devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml | 2 +-
> .../devicetree/bindings/iio/adc/samsung,exynos-adc.yaml | 2 +-
> .../devicetree/bindings/iio/adc/st,stm32-adc.yaml | 8 ++++----
> .../devicetree/bindings/iio/adc/ti,ads131e08.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml | 2 +-
> .../devicetree/bindings/iio/dac/lltc,ltc1660.yaml | 4 ++--
> .../devicetree/bindings/iio/dac/lltc,ltc2632.yaml | 4 ++--
> .../devicetree/bindings/iio/dac/st,stm32-dac.yaml | 4 ++--
> Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml | 2 +-
> .../devicetree/bindings/iio/temperature/ti,tmp117.yaml | 6 +++---
> 17 files changed, 29 insertions(+), 30 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/iio/accel/memsensing,msa311.yaml b/Documentation/devicetree/bindings/iio/accel/memsensing,msa311.yaml
> index 23528dcaa073..d530ec041fe7 100644
> --- a/Documentation/devicetree/bindings/iio/accel/memsensing,msa311.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/memsensing,msa311.yaml
> @@ -1,9 +1,8 @@
> # SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> -
> %YAML 1.2
> ---
> -$id: "http://devicetree.org/schemas/iio/accel/memsensing,msa311.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/iio/accel/memsensing,msa311.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: MEMSensing digital 3-Axis accelerometer
>

Reviewed-by: Dmitry Rokosov <[email protected]>

[...]

--
Thank you,
Dmitry

2023-01-20 08:52:10

by Claudiu Beznea

[permalink] [raw]
Subject: Re: [PATCH 1/5] dt-bindings: iio: drop unneeded quotes

On 18.01.2023 20:44, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>
> Cleanup by removing unneeded quotes from refs and redundant blank lines.
> No functional impact except adjusting to preferred coding style.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> .../devicetree/bindings/iio/accel/memsensing,msa311.yaml | 5 ++---
> Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml | 2 +-
> .../devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml | 2 +-

For atmel,sama5d2-adc.yaml:
Reviewed-by: Claudiu Beznea <[email protected]>

> Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml | 4 ++--
> .../devicetree/bindings/iio/adc/ingenic,adc.yaml | 4 ++--
> .../devicetree/bindings/iio/adc/microchip,mcp3911.yaml | 4 ++--
> .../devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml | 2 +-
> .../devicetree/bindings/iio/adc/samsung,exynos-adc.yaml | 2 +-
> .../devicetree/bindings/iio/adc/st,stm32-adc.yaml | 8 ++++----
> .../devicetree/bindings/iio/adc/ti,ads131e08.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml | 2 +-
> .../devicetree/bindings/iio/dac/lltc,ltc1660.yaml | 4 ++--
> .../devicetree/bindings/iio/dac/lltc,ltc2632.yaml | 4 ++--
> .../devicetree/bindings/iio/dac/st,stm32-dac.yaml | 4 ++--
> Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml | 2 +-
> .../devicetree/bindings/iio/temperature/ti,tmp117.yaml | 6 +++---
> 17 files changed, 29 insertions(+), 30 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/iio/accel/memsensing,msa311.yaml b/Documentation/devicetree/bindings/iio/accel/memsensing,msa311.yaml
> index 23528dcaa073..d530ec041fe7 100644
> --- a/Documentation/devicetree/bindings/iio/accel/memsensing,msa311.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/memsensing,msa311.yaml
> @@ -1,9 +1,8 @@
> # SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> -
> %YAML 1.2
> ---
> -$id: "http://devicetree.org/schemas/iio/accel/memsensing,msa311.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/iio/accel/memsensing,msa311.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: MEMSensing digital 3-Axis accelerometer
>
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> index 75a7184a4735..35ed04350e28 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> @@ -61,7 +61,7 @@ required:
>
> patternProperties:
> "^channel@([0-9]|1[0-5])$":
> - $ref: "adc.yaml"
> + $ref: adc.yaml
> type: object
> description: |
> Represents the external channels which are connected to the ADC.
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> index 1bfbeed6f299..7cc4ddc4e9b7 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> @@ -43,7 +43,7 @@ required:
>
> patternProperties:
> "^channel@[0-7]$":
> - $ref: "adc.yaml"
> + $ref: adc.yaml
> type: object
> description: |
> Represents the external channels which are connected to the ADC.
> diff --git a/Documentation/devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml b/Documentation/devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml
> index 31f840d59303..4817b840977a 100644
> --- a/Documentation/devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml
> @@ -41,7 +41,7 @@ properties:
> description: Startup time expressed in ms, it depends on SoC.
>
> atmel,trigger-edge-type:
> - $ref: '/schemas/types.yaml#/definitions/uint32'
> + $ref: /schemas/types.yaml#/definitions/uint32
> description:
> One of possible edge types for the ADTRG hardware trigger pin.
> When the specific edge type is detected, the conversion will
> diff --git a/Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml b/Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml
> index 77605f17901c..9c57eb13f892 100644
> --- a/Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml
> @@ -1,8 +1,8 @@
> # SPDX-License-Identifier: GPL-2.0
> %YAML 1.2
> ---
> -$id: "http://devicetree.org/schemas/iio/adc/avia-hx711.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/iio/adc/avia-hx711.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: AVIA HX711 ADC chip for weight cells
>
> diff --git a/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml b/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml
> index 517e8b1fcb73..b71c951e6d02 100644
> --- a/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/ingenic,adc.yaml
> @@ -2,8 +2,8 @@
> # Copyright 2019-2020 Artur Rojek
> %YAML 1.2
> ---
> -$id: "http://devicetree.org/schemas/iio/adc/ingenic,adc.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/iio/adc/ingenic,adc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Ingenic JZ47xx ADC controller IIO
>
> diff --git a/Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml b/Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
> index 2c93fb41f172..f7b3fde4115a 100644
> --- a/Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
> @@ -2,8 +2,8 @@
> # Copyright 2019 Marcus Folkesson <[email protected]>
> %YAML 1.2
> ---
> -$id: "http://devicetree.org/schemas/iio/adc/microchip,mcp3911.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/iio/adc/microchip,mcp3911.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Microchip MCP3911 Dual channel analog front end (ADC)
>
> diff --git a/Documentation/devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml b/Documentation/devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml
> index 8b743742a5f9..ba86c7b7d622 100644
> --- a/Documentation/devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml
> @@ -69,7 +69,7 @@ required:
>
> patternProperties:
> "^channel@[0-7]$":
> - $ref: "adc.yaml"
> + $ref: adc.yaml
> type: object
> description: |
> Represents the external channels which are connected to the ADC.
> diff --git a/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml b/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml
> index 81c87295912c..e27d094cfa05 100644
> --- a/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml
> @@ -52,7 +52,7 @@ properties:
> vdd-supply: true
>
> samsung,syscon-phandle:
> - $ref: '/schemas/types.yaml#/definitions/phandle'
> + $ref: /schemas/types.yaml#/definitions/phandle
> description:
> Phandle to the PMU system controller node (to access the ADC_PHY
> register on Exynos3250/4x12/5250/5420/5800).
> diff --git a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
> index 1c340c95df16..995cbf8cefc6 100644
> --- a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml
> @@ -1,8 +1,8 @@
> # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> %YAML 1.2
> ---
> -$id: "http://devicetree.org/schemas/iio/adc/st,stm32-adc.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/iio/adc/st,stm32-adc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: STMicroelectronics STM32 ADC
>
> @@ -80,7 +80,7 @@ properties:
> description:
> Phandle to system configuration controller. It can be used to control the
> analog circuitry on stm32mp1.
> - $ref: "/schemas/types.yaml#/definitions/phandle-array"
> + $ref: /schemas/types.yaml#/definitions/phandle-array
>
> interrupt-controller: true
>
> @@ -341,7 +341,7 @@ patternProperties:
> patternProperties:
> "^channel@([0-9]|1[0-9])$":
> type: object
> - $ref: "adc.yaml"
> + $ref: adc.yaml
> description: Represents the external channels which are connected to the ADC.
>
> properties:
> diff --git a/Documentation/devicetree/bindings/iio/adc/ti,ads131e08.yaml b/Documentation/devicetree/bindings/iio/adc/ti,ads131e08.yaml
> index 55c2c73626f4..890f125d422c 100644
> --- a/Documentation/devicetree/bindings/iio/adc/ti,ads131e08.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/ti,ads131e08.yaml
> @@ -77,7 +77,7 @@ required:
>
> patternProperties:
> "^channel@([0-7])$":
> - $ref: "adc.yaml"
> + $ref: adc.yaml
> type: object
> description: |
> Represents the external channels which are connected to the ADC.
> diff --git a/Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml b/Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml
> index bdf3bba2d750..32c52f9fe18b 100644
> --- a/Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml
> @@ -41,7 +41,7 @@ required:
>
> patternProperties:
> "^channel@[0-7]$":
> - $ref: "adc.yaml"
> + $ref: adc.yaml
> type: object
>
> properties:
> diff --git a/Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml b/Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml
> index 133b0f867992..c9f51d00fa8f 100644
> --- a/Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml
> +++ b/Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml
> @@ -2,8 +2,8 @@
> # Copyright 2019 Marcus Folkesson <[email protected]>
> %YAML 1.2
> ---
> -$id: "http://devicetree.org/schemas/iio/dac/lltc,ltc1660.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/iio/dac/lltc,ltc1660.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Linear Technology Micropower octal 8-Bit and 10-Bit DACs
>
> diff --git a/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml b/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml
> index b1eb77335d05..c9e3be3b5754 100644
> --- a/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml
> +++ b/Documentation/devicetree/bindings/iio/dac/lltc,ltc2632.yaml
> @@ -1,8 +1,8 @@
> # SPDX-License-Identifier: GPL-2.0 OR BSD-2-Clause
> %YAML 1.2
> ---
> -$id: "http://devicetree.org/schemas/iio/dac/lltc,ltc2632.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/iio/dac/lltc,ltc2632.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Linear Technology LTC263x 12-/10-/8-Bit Rail-to-Rail DAC
>
> diff --git a/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml b/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml
> index 0f1bf1110122..04045b932bd2 100644
> --- a/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml
> +++ b/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml
> @@ -1,8 +1,8 @@
> # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> %YAML 1.2
> ---
> -$id: "http://devicetree.org/schemas/iio/dac/st,stm32-dac.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/iio/dac/st,stm32-dac.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: STMicroelectronics STM32 DAC
>
> diff --git a/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml b/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
> index 68b481c63318..decf022335d8 100644
> --- a/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
> @@ -63,7 +63,7 @@ properties:
> description: if defined provides VDD IO power to the sensor.
>
> st,drdy-int-pin:
> - $ref: '/schemas/types.yaml#/definitions/uint32'
> + $ref: /schemas/types.yaml#/definitions/uint32
> description: |
> The pin on the package that will be used to signal data ready
> enum:
> diff --git a/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml b/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml
> index 347bc16a4671..c4f1c69f9330 100644
> --- a/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml
> +++ b/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml
> @@ -1,10 +1,10 @@
> # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> %YAML 1.2
> ---
> -$id: "http://devicetree.org/schemas/iio/temperature/ti,tmp117.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/iio/temperature/ti,tmp117.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
> -title: "TI TMP117 - Digital temperature sensor with integrated NV memory"
> +title: TI TMP117 - Digital temperature sensor with integrated NV memory
>
> description: |
> TI TMP117 - Digital temperature sensor with integrated NV memory that supports
> --
> 2.34.1
>

2023-01-20 22:25:06

by Puranjay Mohan

[permalink] [raw]
Subject: Re: [PATCH 1/5] dt-bindings: iio: drop unneeded quotes

On Wed, Jan 18, 2023 at 7:44 PM Krzysztof Kozlowski
<[email protected]> wrote:
>
> Cleanup by removing unneeded quotes from refs and redundant blank lines.
> No functional impact except adjusting to preferred coding style.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> .../devicetree/bindings/iio/accel/memsensing,msa311.yaml | 5 ++---
> Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml | 2 +-
> .../devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml | 4 ++--
> .../devicetree/bindings/iio/adc/ingenic,adc.yaml | 4 ++--
> .../devicetree/bindings/iio/adc/microchip,mcp3911.yaml | 4 ++--
> .../devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml | 2 +-
> .../devicetree/bindings/iio/adc/samsung,exynos-adc.yaml | 2 +-
> .../devicetree/bindings/iio/adc/st,stm32-adc.yaml | 8 ++++----
> .../devicetree/bindings/iio/adc/ti,ads131e08.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml | 2 +-
> .../devicetree/bindings/iio/dac/lltc,ltc1660.yaml | 4 ++--
> .../devicetree/bindings/iio/dac/lltc,ltc2632.yaml | 4 ++--
> .../devicetree/bindings/iio/dac/st,stm32-dac.yaml | 4 ++--
> Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml | 2 +-
> .../devicetree/bindings/iio/temperature/ti,tmp117.yaml | 6 +++---

For ti,tmp117.yaml
Reviewed-by: Puranjay Mohan <[email protected]>

2023-01-21 17:58:28

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH 3/5] dt-bindings: iio: correct node names in examples

On Wed, 18 Jan 2023 19:44:11 +0100
Krzysztof Kozlowski <[email protected]> wrote:

> Do not use underscores and unneeded suffixes (e.g. i2c0) in node name in
> examples.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---

> diff --git a/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml b/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml
> index 6c5ad426a016..12f75ddc4a70 100644
> --- a/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml
> +++ b/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml
> @@ -42,7 +42,7 @@ examples:
> #address-cells = <1>;
> #size-cells = <0>;
>
> - heart_mon@0 {
> + heart-rate@0 {

These are both heart-rate and pulse oximeters so measure more than just
the rate (oxygen saturation in the blood). Reality is they actually
measure light absorption over time, but you can calculate an estimate
of both rate and oxygen saturation from that.

I don't really mind simplifying that to heart-rate, but wanted to
call this out for possible discussion.


> compatible = "ti,afe4403";
> reg = <0>;
> spi-max-frequency = <10000000>;
> diff --git a/Documentation/devicetree/bindings/iio/health/ti,afe4404.yaml b/Documentation/devicetree/bindings/iio/health/ti,afe4404.yaml
> index c0e815d9999e..b334f3e356ad 100644
> --- a/Documentation/devicetree/bindings/iio/health/ti,afe4404.yaml
> +++ b/Documentation/devicetree/bindings/iio/health/ti,afe4404.yaml
> @@ -39,7 +39,7 @@ examples:
> #address-cells = <1>;
> #size-cells = <0>;
>
> - heart_mon@58 {
> + heart-rate@58 {
> compatible = "ti,afe4404";
> reg = <0x58>;
> tx-supply = <&vbat>;

The rest are all fine. If no one comments I'll pick this up as it is in a few
days.

Jonathan

2023-01-21 18:55:27

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 3/5] dt-bindings: iio: correct node names in examples

On 21/01/2023 18:17, Jonathan Cameron wrote:
> On Wed, 18 Jan 2023 19:44:11 +0100
> Krzysztof Kozlowski <[email protected]> wrote:
>
>> Do not use underscores and unneeded suffixes (e.g. i2c0) in node name in
>> examples.
>>
>> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>> ---
>
>> diff --git a/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml b/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml
>> index 6c5ad426a016..12f75ddc4a70 100644
>> --- a/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml
>> +++ b/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml
>> @@ -42,7 +42,7 @@ examples:
>> #address-cells = <1>;
>> #size-cells = <0>;
>>
>> - heart_mon@0 {
>> + heart-rate@0 {
>
> These are both heart-rate and pulse oximeters so measure more than just
> the rate (oxygen saturation in the blood). Reality is they actually
> measure light absorption over time, but you can calculate an estimate
> of both rate and oxygen saturation from that.
>
> I don't really mind simplifying that to heart-rate, but wanted to
> call this out for possible discussion.

They could be heart-mon. The fix is mostly around the underscore. I
don't have any arguments for changing it to heart-rate, thus we can go
with whatever you prefer.

Best regards,
Krzysztof

2023-01-22 16:47:42

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH 3/5] dt-bindings: iio: correct node names in examples

On Sat, 21 Jan 2023 19:31:23 +0100
Krzysztof Kozlowski <[email protected]> wrote:

> On 21/01/2023 18:17, Jonathan Cameron wrote:
> > On Wed, 18 Jan 2023 19:44:11 +0100
> > Krzysztof Kozlowski <[email protected]> wrote:
> >
> >> Do not use underscores and unneeded suffixes (e.g. i2c0) in node name in
> >> examples.
> >>
> >> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> >> ---
> >
> >> diff --git a/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml b/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml
> >> index 6c5ad426a016..12f75ddc4a70 100644
> >> --- a/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml
> >> +++ b/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml
> >> @@ -42,7 +42,7 @@ examples:
> >> #address-cells = <1>;
> >> #size-cells = <0>;
> >>
> >> - heart_mon@0 {
> >> + heart-rate@0 {
> >
> > These are both heart-rate and pulse oximeters so measure more than just
> > the rate (oxygen saturation in the blood). Reality is they actually
> > measure light absorption over time, but you can calculate an estimate
> > of both rate and oxygen saturation from that.
> >
> > I don't really mind simplifying that to heart-rate, but wanted to
> > call this out for possible discussion.
>
> They could be heart-mon. The fix is mostly around the underscore. I
> don't have any arguments for changing it to heart-rate, thus we can go
> with whatever you prefer.
I'm fine with either and as we have a patch with heart-rate that wins
currently. I'll just let this sit for a little longer than
normal to see if we get any other responses!

Thanks,

Jonathan
>
> Best regards,
> Krzysztof
>


2023-01-23 20:21:20

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH 1/5] dt-bindings: iio: drop unneeded quotes


On Wed, 18 Jan 2023 19:44:09 +0100, Krzysztof Kozlowski wrote:
> Cleanup by removing unneeded quotes from refs and redundant blank lines.
> No functional impact except adjusting to preferred coding style.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> .../devicetree/bindings/iio/accel/memsensing,msa311.yaml | 5 ++---
> Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml | 2 +-
> .../devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml | 4 ++--
> .../devicetree/bindings/iio/adc/ingenic,adc.yaml | 4 ++--
> .../devicetree/bindings/iio/adc/microchip,mcp3911.yaml | 4 ++--
> .../devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml | 2 +-
> .../devicetree/bindings/iio/adc/samsung,exynos-adc.yaml | 2 +-
> .../devicetree/bindings/iio/adc/st,stm32-adc.yaml | 8 ++++----
> .../devicetree/bindings/iio/adc/ti,ads131e08.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml | 2 +-
> .../devicetree/bindings/iio/dac/lltc,ltc1660.yaml | 4 ++--
> .../devicetree/bindings/iio/dac/lltc,ltc2632.yaml | 4 ++--
> .../devicetree/bindings/iio/dac/st,stm32-dac.yaml | 4 ++--
> Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml | 2 +-
> .../devicetree/bindings/iio/temperature/ti,tmp117.yaml | 6 +++---
> 17 files changed, 29 insertions(+), 30 deletions(-)
>

Acked-by: Rob Herring <[email protected]>

2023-01-23 20:22:24

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH 2/5] dt-bindings: iio: minor whitespace cleanups


On Wed, 18 Jan 2023 19:44:10 +0100, Krzysztof Kozlowski wrote:
> Drop redundant blank lines and add such when needed.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> Documentation/devicetree/bindings/iio/dac/adi,ad5380.yaml | 1 +
> .../bindings/iio/proximity/google,cros-ec-mkbp-proximity.yaml | 1 -
> 2 files changed, 1 insertion(+), 1 deletion(-)
>

Acked-by: Rob Herring <[email protected]>

2023-01-23 20:33:49

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH 3/5] dt-bindings: iio: correct node names in examples

On Sun, Jan 22, 2023 at 05:01:05PM +0000, Jonathan Cameron wrote:
> On Sat, 21 Jan 2023 19:31:23 +0100
> Krzysztof Kozlowski <[email protected]> wrote:
>
> > On 21/01/2023 18:17, Jonathan Cameron wrote:
> > > On Wed, 18 Jan 2023 19:44:11 +0100
> > > Krzysztof Kozlowski <[email protected]> wrote:
> > >
> > >> Do not use underscores and unneeded suffixes (e.g. i2c0) in node name in
> > >> examples.
> > >>
> > >> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> > >> ---
> > >
> > >> diff --git a/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml b/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml
> > >> index 6c5ad426a016..12f75ddc4a70 100644
> > >> --- a/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml
> > >> +++ b/Documentation/devicetree/bindings/iio/health/ti,afe4403.yaml
> > >> @@ -42,7 +42,7 @@ examples:
> > >> #address-cells = <1>;
> > >> #size-cells = <0>;
> > >>
> > >> - heart_mon@0 {
> > >> + heart-rate@0 {
> > >
> > > These are both heart-rate and pulse oximeters so measure more than just
> > > the rate (oxygen saturation in the blood). Reality is they actually
> > > measure light absorption over time, but you can calculate an estimate
> > > of both rate and oxygen saturation from that.
> > >
> > > I don't really mind simplifying that to heart-rate, but wanted to
> > > call this out for possible discussion.
> >
> > They could be heart-mon. The fix is mostly around the underscore. I
> > don't have any arguments for changing it to heart-rate, thus we can go
> > with whatever you prefer.
> I'm fine with either and as we have a patch with heart-rate that wins
> currently. I'll just let this sit for a little longer than
> normal to see if we get any other responses!

I don't care all that much until we've documented something. Otherwise,
we may be just changing things twice. We have the list in the spec, but
really I'd like that in schema form. We'd also need to figure out how to
use that. There's always going to be odd things which we don't have any
defined name.

For now, I'd just do 's/_/-/'.

Rob

2023-01-23 20:35:52

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH 4/5] dt-bindings: iio: use lowercase hex in examples


On Wed, 18 Jan 2023 19:44:12 +0100, Krzysztof Kozlowski wrote:
> Use lowercase hex in addresses in examples.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> .../devicetree/bindings/iio/accel/kionix,kxcjk1013.yaml | 2 +-
> .../devicetree/bindings/iio/adc/samsung,exynos-adc.yaml | 2 +-
> Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml | 2 +-
> .../devicetree/bindings/iio/potentiometer/adi,ad5272.yaml | 2 +-
> .../devicetree/bindings/iio/temperature/adi,ltc2983.yaml | 6 +++---
> 5 files changed, 7 insertions(+), 7 deletions(-)
>

Acked-by: Rob Herring <[email protected]>

2023-01-23 20:36:47

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH 3/5] dt-bindings: iio: correct node names in examples


On Wed, 18 Jan 2023 19:44:11 +0100, Krzysztof Kozlowski wrote:
> Do not use underscores and unneeded suffixes (e.g. i2c0) in node name in
> examples.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> .../devicetree/bindings/iio/accel/adi,adis16201.yaml | 2 +-
> .../devicetree/bindings/iio/accel/adi,adis16240.yaml | 2 +-
> .../devicetree/bindings/iio/accel/adi,adxl313.yaml | 2 +-
> .../devicetree/bindings/iio/accel/adi,adxl345.yaml | 4 ++--
> .../devicetree/bindings/iio/accel/bosch,bma220.yaml | 2 +-
> .../devicetree/bindings/iio/accel/nxp,fxls8962af.yaml | 4 ++--
> Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml | 2 +-
> .../devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml | 2 +-
> .../devicetree/bindings/iio/adc/st,stmpe-adc.yaml | 8 +++-----
> .../devicetree/bindings/iio/frequency/adf4371.yaml | 2 +-
> .../devicetree/bindings/iio/gyroscope/nxp,fxas21002c.yaml | 4 ++--
> .../devicetree/bindings/iio/health/ti,afe4403.yaml | 2 +-
> .../devicetree/bindings/iio/health/ti,afe4404.yaml | 2 +-
> Documentation/devicetree/bindings/iio/humidity/dht11.yaml | 2 +-
> .../devicetree/bindings/iio/humidity/ti,hdc2010.yaml | 2 +-
> .../devicetree/bindings/iio/imu/adi,adis16460.yaml | 2 +-
> .../devicetree/bindings/iio/imu/invensense,icm42600.yaml | 4 ++--
> .../devicetree/bindings/iio/imu/nxp,fxos8700.yaml | 4 ++--
> .../devicetree/bindings/iio/pressure/asc,dlhl60d.yaml | 2 +-
> .../devicetree/bindings/iio/pressure/bmp085.yaml | 2 +-
> .../bindings/iio/temperature/maxim,max31865.yaml | 2 +-
> 23 files changed, 30 insertions(+), 32 deletions(-)
>

Acked-by: Rob Herring <[email protected]>

2023-01-24 01:54:42

by Marcelo Schmitt

[permalink] [raw]
Subject: Re: [PATCH 1/5] dt-bindings: iio: drop unneeded quotes

On 01/18, Krzysztof Kozlowski wrote:
> Cleanup by removing unneeded quotes from refs and redundant blank lines.
> No functional impact except adjusting to preferred coding style.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> .../devicetree/bindings/iio/accel/memsensing,msa311.yaml | 5 ++---
> Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml | 2 +-
> .../devicetree/bindings/iio/adc/atmel,sama5d2-adc.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml | 4 ++--
> .../devicetree/bindings/iio/adc/ingenic,adc.yaml | 4 ++--
> .../devicetree/bindings/iio/adc/microchip,mcp3911.yaml | 4 ++--
> .../devicetree/bindings/iio/adc/renesas,rzg2l-adc.yaml | 2 +-
> .../devicetree/bindings/iio/adc/samsung,exynos-adc.yaml | 2 +-
> .../devicetree/bindings/iio/adc/st,stm32-adc.yaml | 8 ++++----
> .../devicetree/bindings/iio/adc/ti,ads131e08.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/ti,tsc2046.yaml | 2 +-
> .../devicetree/bindings/iio/dac/lltc,ltc1660.yaml | 4 ++--
> .../devicetree/bindings/iio/dac/lltc,ltc2632.yaml | 4 ++--
> .../devicetree/bindings/iio/dac/st,stm32-dac.yaml | 4 ++--
> Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml | 2 +-
> .../devicetree/bindings/iio/temperature/ti,tmp117.yaml | 6 +++---
> 17 files changed, 29 insertions(+), 30 deletions(-)
>

For adi,ad7292.yaml
Acked-by: Marcelo Schmitt <[email protected]>

2023-01-24 07:40:28

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 3/5] dt-bindings: iio: correct node names in examples

On 23/01/2023 21:33, Rob Herring wrote:
until we've documented something. Otherwise,
> we may be just changing things twice. We have the list in the spec, but
> really I'd like that in schema form. We'd also need to figure out how to
> use that. There's always going to be odd things which we don't have any
> defined name.
>
> For now, I'd just do 's/_/-/'.

I'll send a v2 with fixes.

Best regards,
Krzysztof