linux-kernel - Re: [PATCH] dt-bindings: iio: Correct indentation and style in DTS example

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <d2c31c3cc5542629b7ceb2347464260822fe3202.camel@gmail.com>
Date: Tue, 07 Jan 2025 14:06:35 +0000
From: Nuno Sá <noname.nuno@...il.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>, Lars-Peter Clausen
	 <lars@...afoo.de>, Michael Hennerich <Michael.Hennerich@...log.com>, 
 Jonathan Cameron	 <jic23@...nel.org>, Rob Herring <robh@...nel.org>,
 Krzysztof Kozlowski	 <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, Puranjay Mohan	 <puranjay@...nel.org>, Joshua
 Felmeden <jfelmeden@...goodpenguin.co.uk>,  Sankar Velliangiri	
 <navin@...umiz.com>, linux-iio@...r.kernel.org, devicetree@...r.kernel.org,
 	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] dt-bindings: iio: Correct indentation and style in DTS
 example

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@...aro.org>
> ---

Acked-by: Nuno Sa <nuno.sa@...log.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>;
>          };
>      };