* [PATCH] dt-bindings: iio: Correct indentation and style in DTS example
@ 2025-01-07 12:58 Krzysztof Kozlowski
2025-01-07 14:06 ` Nuno Sá
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2025-01-07 12:58 UTC (permalink / raw)
To: Lars-Peter Clausen, Michael Hennerich, Jonathan Cameron,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Puranjay Mohan,
Joshua Felmeden, Sankar Velliangiri, linux-iio, devicetree,
linux-kernel
Cc: Krzysztof Kozlowski
DTS example in the bindings should be indented with 2- or 4-spaces and
aligned with opening '- |', so correct any differences like 3-spaces or
mixtures 2- and 4-spaces in one binding.
No functional changes here, but saves some comments during reviews of
new patches built on existing code.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
.../bindings/iio/dac/adi,ad5380.yaml | 18 +++++++++---------
.../iio/humidity/sciosense,ens210.yaml | 12 ++++++------
.../iio/temperature/maxim,max31865.yaml | 18 +++++++++---------
.../bindings/iio/temperature/ti,tmp117.yaml | 6 +++---
4 files changed, 27 insertions(+), 27 deletions(-)
diff --git a/Documentation/devicetree/bindings/iio/dac/adi,ad5380.yaml b/Documentation/devicetree/bindings/iio/dac/adi,ad5380.yaml
index 9eb9928500e2..3e323f1a5458 100644
--- a/Documentation/devicetree/bindings/iio/dac/adi,ad5380.yaml
+++ b/Documentation/devicetree/bindings/iio/dac/adi,ad5380.yaml
@@ -55,18 +55,18 @@ examples:
#address-cells = <1>;
#size-cells = <0>;
dac@0 {
- reg = <0>;
- compatible = "adi,ad5390-5";
- vref-supply = <&dacvref>;
+ reg = <0>;
+ compatible = "adi,ad5390-5";
+ vref-supply = <&dacvref>;
};
};
- |
i2c {
- #address-cells = <1>;
- #size-cells = <0>;
- dac@42 {
- reg = <0x42>;
- compatible = "adi,ad5380-3";
- };
+ #address-cells = <1>;
+ #size-cells = <0>;
+ dac@42 {
+ reg = <0x42>;
+ compatible = "adi,ad5380-3";
+ };
};
...
diff --git a/Documentation/devicetree/bindings/iio/humidity/sciosense,ens210.yaml b/Documentation/devicetree/bindings/iio/humidity/sciosense,ens210.yaml
index ed0ea938f7f8..1e25cf781cf1 100644
--- a/Documentation/devicetree/bindings/iio/humidity/sciosense,ens210.yaml
+++ b/Documentation/devicetree/bindings/iio/humidity/sciosense,ens210.yaml
@@ -43,13 +43,13 @@ additionalProperties: false
examples:
- |
i2c {
- #address-cells = <1>;
- #size-cells = <0>;
+ #address-cells = <1>;
+ #size-cells = <0>;
- temperature-sensor@43 {
- compatible = "sciosense,ens210";
- reg = <0x43>;
- };
+ temperature-sensor@43 {
+ compatible = "sciosense,ens210";
+ reg = <0x43>;
+ };
};
...
diff --git a/Documentation/devicetree/bindings/iio/temperature/maxim,max31865.yaml b/Documentation/devicetree/bindings/iio/temperature/maxim,max31865.yaml
index 7cc365e0ebc8..7c0c6ab6fc69 100644
--- a/Documentation/devicetree/bindings/iio/temperature/maxim,max31865.yaml
+++ b/Documentation/devicetree/bindings/iio/temperature/maxim,max31865.yaml
@@ -40,15 +40,15 @@ unevaluatedProperties: false
examples:
- |
spi {
- #address-cells = <1>;
- #size-cells = <0>;
+ #address-cells = <1>;
+ #size-cells = <0>;
- temperature-sensor@0 {
- compatible = "maxim,max31865";
- reg = <0>;
- spi-max-frequency = <400000>;
- spi-cpha;
- maxim,3-wire;
- };
+ temperature-sensor@0 {
+ compatible = "maxim,max31865";
+ reg = <0>;
+ spi-max-frequency = <400000>;
+ spi-cpha;
+ maxim,3-wire;
+ };
};
...
diff --git a/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml b/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml
index 58aa1542776b..fbba5e934861 100644
--- a/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml
+++ b/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml
@@ -44,8 +44,8 @@ examples:
#size-cells = <0>;
tmp117@48 {
- compatible = "ti,tmp117";
- reg = <0x48>;
- vcc-supply = <&pmic_reg_3v3>;
+ compatible = "ti,tmp117";
+ reg = <0x48>;
+ vcc-supply = <&pmic_reg_3v3>;
};
};
--
2.43.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] dt-bindings: iio: Correct indentation and style in DTS example
2025-01-07 12:58 [PATCH] dt-bindings: iio: Correct indentation and style in DTS example Krzysztof Kozlowski
@ 2025-01-07 14:06 ` Nuno Sá
2025-01-10 15:42 ` Rob Herring (Arm)
2025-01-12 11:54 ` Jonathan Cameron
2 siblings, 0 replies; 4+ messages in thread
From: Nuno Sá @ 2025-01-07 14:06 UTC (permalink / raw)
To: Krzysztof Kozlowski, Lars-Peter Clausen, Michael Hennerich,
Jonathan Cameron, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Puranjay Mohan, Joshua Felmeden, Sankar Velliangiri, linux-iio,
devicetree, linux-kernel
On Tue, 2025-01-07 at 13:58 +0100, Krzysztof Kozlowski wrote:
> DTS example in the bindings should be indented with 2- or 4-spaces and
> aligned with opening '- |', so correct any differences like 3-spaces or
> mixtures 2- and 4-spaces in one binding.
>
> No functional changes here, but saves some comments during reviews of
> new patches built on existing code.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Acked-by: Nuno Sa <nuno.sa@analog.com>
> .../bindings/iio/dac/adi,ad5380.yaml | 18 +++++++++---------
> .../iio/humidity/sciosense,ens210.yaml | 12 ++++++------
> .../iio/temperature/maxim,max31865.yaml | 18 +++++++++---------
> .../bindings/iio/temperature/ti,tmp117.yaml | 6 +++---
> 4 files changed, 27 insertions(+), 27 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/iio/dac/adi,ad5380.yaml
> b/Documentation/devicetree/bindings/iio/dac/adi,ad5380.yaml
> index 9eb9928500e2..3e323f1a5458 100644
> --- a/Documentation/devicetree/bindings/iio/dac/adi,ad5380.yaml
> +++ b/Documentation/devicetree/bindings/iio/dac/adi,ad5380.yaml
> @@ -55,18 +55,18 @@ examples:
> #address-cells = <1>;
> #size-cells = <0>;
> dac@0 {
> - reg = <0>;
> - compatible = "adi,ad5390-5";
> - vref-supply = <&dacvref>;
> + reg = <0>;
> + compatible = "adi,ad5390-5";
> + vref-supply = <&dacvref>;
> };
> };
> - |
> i2c {
> - #address-cells = <1>;
> - #size-cells = <0>;
> - dac@42 {
> - reg = <0x42>;
> - compatible = "adi,ad5380-3";
> - };
> + #address-cells = <1>;
> + #size-cells = <0>;
> + dac@42 {
> + reg = <0x42>;
> + compatible = "adi,ad5380-3";
> + };
> };
> ...
> diff --git
> a/Documentation/devicetree/bindings/iio/humidity/sciosense,ens210.yaml
> b/Documentation/devicetree/bindings/iio/humidity/sciosense,ens210.yaml
> index ed0ea938f7f8..1e25cf781cf1 100644
> --- a/Documentation/devicetree/bindings/iio/humidity/sciosense,ens210.yaml
> +++ b/Documentation/devicetree/bindings/iio/humidity/sciosense,ens210.yaml
> @@ -43,13 +43,13 @@ additionalProperties: false
> examples:
> - |
> i2c {
> - #address-cells = <1>;
> - #size-cells = <0>;
> + #address-cells = <1>;
> + #size-cells = <0>;
>
> - temperature-sensor@43 {
> - compatible = "sciosense,ens210";
> - reg = <0x43>;
> - };
> + temperature-sensor@43 {
> + compatible = "sciosense,ens210";
> + reg = <0x43>;
> + };
> };
> ...
>
> diff --git
> a/Documentation/devicetree/bindings/iio/temperature/maxim,max31865.yaml
> b/Documentation/devicetree/bindings/iio/temperature/maxim,max31865.yaml
> index 7cc365e0ebc8..7c0c6ab6fc69 100644
> --- a/Documentation/devicetree/bindings/iio/temperature/maxim,max31865.yaml
> +++ b/Documentation/devicetree/bindings/iio/temperature/maxim,max31865.yaml
> @@ -40,15 +40,15 @@ unevaluatedProperties: false
> examples:
> - |
> spi {
> - #address-cells = <1>;
> - #size-cells = <0>;
> + #address-cells = <1>;
> + #size-cells = <0>;
>
> - temperature-sensor@0 {
> - compatible = "maxim,max31865";
> - reg = <0>;
> - spi-max-frequency = <400000>;
> - spi-cpha;
> - maxim,3-wire;
> - };
> + temperature-sensor@0 {
> + compatible = "maxim,max31865";
> + reg = <0>;
> + spi-max-frequency = <400000>;
> + spi-cpha;
> + maxim,3-wire;
> + };
> };
> ...
> diff --git a/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml
> b/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml
> index 58aa1542776b..fbba5e934861 100644
> --- a/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml
> +++ b/Documentation/devicetree/bindings/iio/temperature/ti,tmp117.yaml
> @@ -44,8 +44,8 @@ examples:
> #size-cells = <0>;
>
> tmp117@48 {
> - compatible = "ti,tmp117";
> - reg = <0x48>;
> - vcc-supply = <&pmic_reg_3v3>;
> + compatible = "ti,tmp117";
> + reg = <0x48>;
> + vcc-supply = <&pmic_reg_3v3>;
> };
> };
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] dt-bindings: iio: Correct indentation and style in DTS example
2025-01-07 12:58 [PATCH] dt-bindings: iio: Correct indentation and style in DTS example Krzysztof Kozlowski
2025-01-07 14:06 ` Nuno Sá
@ 2025-01-10 15:42 ` Rob Herring (Arm)
2025-01-12 11:54 ` Jonathan Cameron
2 siblings, 0 replies; 4+ messages in thread
From: Rob Herring (Arm) @ 2025-01-10 15:42 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lars-Peter Clausen, devicetree, Jonathan Cameron, Joshua Felmeden,
Conor Dooley, Krzysztof Kozlowski, Michael Hennerich, linux-iio,
Sankar Velliangiri, linux-kernel, Puranjay Mohan
On Tue, 07 Jan 2025 13:58:47 +0100, Krzysztof Kozlowski wrote:
> DTS example in the bindings should be indented with 2- or 4-spaces and
> aligned with opening '- |', so correct any differences like 3-spaces or
> mixtures 2- and 4-spaces in one binding.
>
> No functional changes here, but saves some comments during reviews of
> new patches built on existing code.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> .../bindings/iio/dac/adi,ad5380.yaml | 18 +++++++++---------
> .../iio/humidity/sciosense,ens210.yaml | 12 ++++++------
> .../iio/temperature/maxim,max31865.yaml | 18 +++++++++---------
> .../bindings/iio/temperature/ti,tmp117.yaml | 6 +++---
> 4 files changed, 27 insertions(+), 27 deletions(-)
>
Acked-by: Rob Herring (Arm) <robh@kernel.org>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] dt-bindings: iio: Correct indentation and style in DTS example
2025-01-07 12:58 [PATCH] dt-bindings: iio: Correct indentation and style in DTS example Krzysztof Kozlowski
2025-01-07 14:06 ` Nuno Sá
2025-01-10 15:42 ` Rob Herring (Arm)
@ 2025-01-12 11:54 ` Jonathan Cameron
2 siblings, 0 replies; 4+ messages in thread
From: Jonathan Cameron @ 2025-01-12 11:54 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lars-Peter Clausen, Michael Hennerich, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Puranjay Mohan,
Joshua Felmeden, Sankar Velliangiri, linux-iio, devicetree,
linux-kernel
On Tue, 7 Jan 2025 13:58:47 +0100
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote:
> DTS example in the bindings should be indented with 2- or 4-spaces and
> aligned with opening '- |', so correct any differences like 3-spaces or
> mixtures 2- and 4-spaces in one binding.
>
> No functional changes here, but saves some comments during reviews of
> new patches built on existing code.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Applied and pushed out as testing for 0-day to take a look.
This has probably just missed this cycle :(
Jonathan
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-01-12 11:55 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-01-07 12:58 [PATCH] dt-bindings: iio: Correct indentation and style in DTS example Krzysztof Kozlowski
2025-01-07 14:06 ` Nuno Sá
2025-01-10 15:42 ` Rob Herring (Arm)
2025-01-12 11:54 ` Jonathan Cameron
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox