[PATCH] dt-bindings: timer: Correct indentation and style in DTS example
Niklas Söderlund
niklas.soderlund at ragnatech.se
Tue Jan 7 05:51:33 PST 2025
Hello Krzysztof,
Thanks for your work.
On 2025-01-07 14:10:22 +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 at linaro.org>
For renesas,*
Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas at ragnatech.se>
> ---
> .../bindings/timer/arm,twd-timer.yaml | 6 +--
> .../bindings/timer/renesas,cmt.yaml | 44 +++++++++----------
> .../bindings/timer/renesas,em-sti.yaml | 10 ++---
> .../bindings/timer/renesas,mtu2.yaml | 14 +++---
> .../bindings/timer/renesas,ostm.yaml | 10 ++---
> .../bindings/timer/renesas,tmu.yaml | 22 +++++-----
> .../bindings/timer/renesas,tpu.yaml | 8 ++--
> .../bindings/timer/sifive,clint.yaml | 2 +-
> 8 files changed, 58 insertions(+), 58 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/timer/arm,twd-timer.yaml b/Documentation/devicetree/bindings/timer/arm,twd-timer.yaml
> index 5684df6448ef..eb1127352c7b 100644
> --- a/Documentation/devicetree/bindings/timer/arm,twd-timer.yaml
> +++ b/Documentation/devicetree/bindings/timer/arm,twd-timer.yaml
> @@ -50,7 +50,7 @@ examples:
> #include <dt-bindings/interrupt-controller/arm-gic.h>
>
> timer at 2c000600 {
> - compatible = "arm,arm11mp-twd-timer";
> - reg = <0x2c000600 0x20>;
> - interrupts = <GIC_PPI 13 0xf01>;
> + compatible = "arm,arm11mp-twd-timer";
> + reg = <0x2c000600 0x20>;
> + interrupts = <GIC_PPI 13 0xf01>;
> };
> diff --git a/Documentation/devicetree/bindings/timer/renesas,cmt.yaml b/Documentation/devicetree/bindings/timer/renesas,cmt.yaml
> index 5e09c04da30e..260b05f213e6 100644
> --- a/Documentation/devicetree/bindings/timer/renesas,cmt.yaml
> +++ b/Documentation/devicetree/bindings/timer/renesas,cmt.yaml
> @@ -178,29 +178,29 @@ examples:
> #include <dt-bindings/interrupt-controller/arm-gic.h>
> #include <dt-bindings/power/r8a7790-sysc.h>
> cmt0: timer at ffca0000 {
> - compatible = "renesas,r8a7790-cmt0", "renesas,rcar-gen2-cmt0";
> - reg = <0xffca0000 0x1004>;
> - interrupts = <GIC_SPI 142 IRQ_TYPE_LEVEL_HIGH>,
> - <GIC_SPI 143 IRQ_TYPE_LEVEL_HIGH>;
> - clocks = <&cpg CPG_MOD 124>;
> - clock-names = "fck";
> - power-domains = <&sysc R8A7790_PD_ALWAYS_ON>;
> - resets = <&cpg 124>;
> + compatible = "renesas,r8a7790-cmt0", "renesas,rcar-gen2-cmt0";
> + reg = <0xffca0000 0x1004>;
> + interrupts = <GIC_SPI 142 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 143 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&cpg CPG_MOD 124>;
> + clock-names = "fck";
> + power-domains = <&sysc R8A7790_PD_ALWAYS_ON>;
> + resets = <&cpg 124>;
> };
>
> cmt1: timer at e6130000 {
> - compatible = "renesas,r8a7790-cmt1", "renesas,rcar-gen2-cmt1";
> - reg = <0xe6130000 0x1004>;
> - interrupts = <GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>,
> - <GIC_SPI 121 IRQ_TYPE_LEVEL_HIGH>,
> - <GIC_SPI 122 IRQ_TYPE_LEVEL_HIGH>,
> - <GIC_SPI 123 IRQ_TYPE_LEVEL_HIGH>,
> - <GIC_SPI 124 IRQ_TYPE_LEVEL_HIGH>,
> - <GIC_SPI 125 IRQ_TYPE_LEVEL_HIGH>,
> - <GIC_SPI 126 IRQ_TYPE_LEVEL_HIGH>,
> - <GIC_SPI 127 IRQ_TYPE_LEVEL_HIGH>;
> - clocks = <&cpg CPG_MOD 329>;
> - clock-names = "fck";
> - power-domains = <&sysc R8A7790_PD_ALWAYS_ON>;
> - resets = <&cpg 329>;
> + compatible = "renesas,r8a7790-cmt1", "renesas,rcar-gen2-cmt1";
> + reg = <0xe6130000 0x1004>;
> + interrupts = <GIC_SPI 120 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 121 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 122 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 123 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 124 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 125 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 126 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 127 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&cpg CPG_MOD 329>;
> + clock-names = "fck";
> + power-domains = <&sysc R8A7790_PD_ALWAYS_ON>;
> + resets = <&cpg 329>;
> };
> diff --git a/Documentation/devicetree/bindings/timer/renesas,em-sti.yaml b/Documentation/devicetree/bindings/timer/renesas,em-sti.yaml
> index 233d74d5402c..a7385d865bca 100644
> --- a/Documentation/devicetree/bindings/timer/renesas,em-sti.yaml
> +++ b/Documentation/devicetree/bindings/timer/renesas,em-sti.yaml
> @@ -38,9 +38,9 @@ examples:
> - |
> #include <dt-bindings/interrupt-controller/arm-gic.h>
> timer at e0180000 {
> - compatible = "renesas,em-sti";
> - reg = <0xe0180000 0x54>;
> - interrupts = <GIC_SPI 125 IRQ_TYPE_LEVEL_HIGH>;
> - clocks = <&sti_sclk>;
> - clock-names = "sclk";
> + compatible = "renesas,em-sti";
> + reg = <0xe0180000 0x54>;
> + interrupts = <GIC_SPI 125 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&sti_sclk>;
> + clock-names = "sclk";
> };
> diff --git a/Documentation/devicetree/bindings/timer/renesas,mtu2.yaml b/Documentation/devicetree/bindings/timer/renesas,mtu2.yaml
> index 15d8dddf4ae9..e56c12f03f72 100644
> --- a/Documentation/devicetree/bindings/timer/renesas,mtu2.yaml
> +++ b/Documentation/devicetree/bindings/timer/renesas,mtu2.yaml
> @@ -66,11 +66,11 @@ examples:
> #include <dt-bindings/clock/r7s72100-clock.h>
> #include <dt-bindings/interrupt-controller/arm-gic.h>
> mtu2: timer at fcff0000 {
> - compatible = "renesas,mtu2-r7s72100", "renesas,mtu2";
> - reg = <0xfcff0000 0x400>;
> - interrupts = <GIC_SPI 107 IRQ_TYPE_LEVEL_HIGH>;
> - interrupt-names = "tgi0a";
> - clocks = <&mstp3_clks R7S72100_CLK_MTU2>;
> - clock-names = "fck";
> - power-domains = <&cpg_clocks>;
> + compatible = "renesas,mtu2-r7s72100", "renesas,mtu2";
> + reg = <0xfcff0000 0x400>;
> + interrupts = <GIC_SPI 107 IRQ_TYPE_LEVEL_HIGH>;
> + interrupt-names = "tgi0a";
> + clocks = <&mstp3_clks R7S72100_CLK_MTU2>;
> + clock-names = "fck";
> + power-domains = <&cpg_clocks>;
> };
> diff --git a/Documentation/devicetree/bindings/timer/renesas,ostm.yaml b/Documentation/devicetree/bindings/timer/renesas,ostm.yaml
> index e8c642166462..9ba858f094ab 100644
> --- a/Documentation/devicetree/bindings/timer/renesas,ostm.yaml
> +++ b/Documentation/devicetree/bindings/timer/renesas,ostm.yaml
> @@ -71,9 +71,9 @@ examples:
> #include <dt-bindings/clock/r7s72100-clock.h>
> #include <dt-bindings/interrupt-controller/arm-gic.h>
> ostm0: timer at fcfec000 {
> - compatible = "renesas,r7s72100-ostm", "renesas,ostm";
> - reg = <0xfcfec000 0x30>;
> - interrupts = <GIC_SPI 102 IRQ_TYPE_EDGE_RISING>;
> - clocks = <&mstp5_clks R7S72100_CLK_OSTM0>;
> - power-domains = <&cpg_clocks>;
> + compatible = "renesas,r7s72100-ostm", "renesas,ostm";
> + reg = <0xfcfec000 0x30>;
> + interrupts = <GIC_SPI 102 IRQ_TYPE_EDGE_RISING>;
> + clocks = <&mstp5_clks R7S72100_CLK_OSTM0>;
> + power-domains = <&cpg_clocks>;
> };
> diff --git a/Documentation/devicetree/bindings/timer/renesas,tmu.yaml b/Documentation/devicetree/bindings/timer/renesas,tmu.yaml
> index 75b0e7c70b62..b1229595acfb 100644
> --- a/Documentation/devicetree/bindings/timer/renesas,tmu.yaml
> +++ b/Documentation/devicetree/bindings/timer/renesas,tmu.yaml
> @@ -122,15 +122,15 @@ examples:
> #include <dt-bindings/interrupt-controller/arm-gic.h>
> #include <dt-bindings/power/r8a7779-sysc.h>
> tmu0: timer at ffd80000 {
> - compatible = "renesas,tmu-r8a7779", "renesas,tmu";
> - reg = <0xffd80000 0x30>;
> - interrupts = <GIC_SPI 32 IRQ_TYPE_LEVEL_HIGH>,
> - <GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>,
> - <GIC_SPI 34 IRQ_TYPE_LEVEL_HIGH>,
> - <GIC_SPI 35 IRQ_TYPE_LEVEL_HIGH>;
> - interrupt-names = "tuni0", "tuni1", "tuni2", "ticpi2";
> - clocks = <&mstp0_clks R8A7779_CLK_TMU0>;
> - clock-names = "fck";
> - power-domains = <&sysc R8A7779_PD_ALWAYS_ON>;
> - #renesas,channels = <3>;
> + compatible = "renesas,tmu-r8a7779", "renesas,tmu";
> + reg = <0xffd80000 0x30>;
> + interrupts = <GIC_SPI 32 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 34 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 35 IRQ_TYPE_LEVEL_HIGH>;
> + interrupt-names = "tuni0", "tuni1", "tuni2", "ticpi2";
> + clocks = <&mstp0_clks R8A7779_CLK_TMU0>;
> + clock-names = "fck";
> + power-domains = <&sysc R8A7779_PD_ALWAYS_ON>;
> + #renesas,channels = <3>;
> };
> diff --git a/Documentation/devicetree/bindings/timer/renesas,tpu.yaml b/Documentation/devicetree/bindings/timer/renesas,tpu.yaml
> index 01554dff23d8..7a473b302775 100644
> --- a/Documentation/devicetree/bindings/timer/renesas,tpu.yaml
> +++ b/Documentation/devicetree/bindings/timer/renesas,tpu.yaml
> @@ -49,8 +49,8 @@ additionalProperties: false
> examples:
> - |
> tpu: tpu at ffffe0 {
> - compatible = "renesas,tpu";
> - reg = <0xffffe0 16>, <0xfffff0 12>;
> - clocks = <&pclk>;
> - clock-names = "fck";
> + compatible = "renesas,tpu";
> + reg = <0xffffe0 16>, <0xfffff0 12>;
> + clocks = <&pclk>;
> + clock-names = "fck";
> };
> diff --git a/Documentation/devicetree/bindings/timer/sifive,clint.yaml b/Documentation/devicetree/bindings/timer/sifive,clint.yaml
> index b42d43d2de48..d6a73be66765 100644
> --- a/Documentation/devicetree/bindings/timer/sifive,clint.yaml
> +++ b/Documentation/devicetree/bindings/timer/sifive,clint.yaml
> @@ -76,6 +76,6 @@ examples:
> <&cpu2intc 3>, <&cpu2intc 7>,
> <&cpu3intc 3>, <&cpu3intc 7>,
> <&cpu4intc 3>, <&cpu4intc 7>;
> - reg = <0x2000000 0x10000>;
> + reg = <0x2000000 0x10000>;
> };
> ...
> --
> 2.43.0
>
>
--
Kind Regards,
Niklas Söderlund
More information about the linux-arm-kernel
mailing list