riscv: dts: microchip: mpfs: Group tuples in interrupt properties
authorGeert Uytterhoeven <geert@linux-m68k.org>
Fri, 17 Dec 2021 12:49:28 +0000 (13:49 +0100)
committerPalmer Dabbelt <palmer@rivosinc.com>
Sun, 9 Jan 2022 18:11:28 +0000 (10:11 -0800)
To improve human readability and enable automatic validation, the tuples
in the various properties containing interrupt specifiers should be
grouped.

Fix this by grouping the tuples of "interrupts" and
"interrupts-extended" properties using angle brackets.

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
Tested-by: Conor Dooley <conor.dooley@microchip.com>
Signed-off-by: Palmer Dabbelt <palmer@rivosinc.com>
arch/riscv/boot/dts/microchip/microchip-mpfs.dtsi

index d9c1dee..869aaf0 100644 (file)
                        cache-size = <2097152>;
                        cache-unified;
                        interrupt-parent = <&plic>;
-                       interrupts = <1 2 3>;
+                       interrupts = <1>, <2>, <3>;
                        reg = <0x0 0x2010000 0x0 0x1000>;
                };
 
                clint@2000000 {
                        compatible = "sifive,fu540-c000-clint", "sifive,clint0";
                        reg = <0x0 0x2000000 0x0 0xC000>;
-                       interrupts-extended = <&cpu0_intc 3 &cpu0_intc 7
-                                               &cpu1_intc 3 &cpu1_intc 7
-                                               &cpu2_intc 3 &cpu2_intc 7
-                                               &cpu3_intc 3 &cpu3_intc 7
-                                               &cpu4_intc 3 &cpu4_intc 7>;
+                       interrupts-extended = <&cpu0_intc 3>, <&cpu0_intc 7>,
+                                             <&cpu1_intc 3>, <&cpu1_intc 7>,
+                                             <&cpu2_intc 3>, <&cpu2_intc 7>,
+                                             <&cpu3_intc 3>, <&cpu3_intc 7>,
+                                             <&cpu4_intc 3>, <&cpu4_intc 7>;
                };
 
                plic: interrupt-controller@c000000 {
                        #address-cells = <0>;
                        #interrupt-cells = <1>;
                        interrupt-controller;
-                       interrupts-extended = <&cpu0_intc 11
-                                       &cpu1_intc 11 &cpu1_intc 9
-                                       &cpu2_intc 11 &cpu2_intc 9
-                                       &cpu3_intc 11 &cpu3_intc 9
-                                       &cpu4_intc 11 &cpu4_intc 9>;
+                       interrupts-extended = <&cpu0_intc 11>,
+                                             <&cpu1_intc 11>, <&cpu1_intc 9>,
+                                             <&cpu2_intc 11>, <&cpu2_intc 9>,
+                                             <&cpu3_intc 11>, <&cpu3_intc 9>,
+                                             <&cpu4_intc 11>, <&cpu4_intc 9>;
                        riscv,ndev = <186>;
                };
 
                        compatible = "sifive,fu540-c000-pdma";
                        reg = <0x0 0x3000000 0x0 0x8000>;
                        interrupt-parent = <&plic>;
-                       interrupts = <23 24 25 26 27 28 29 30>;
+                       interrupts = <23>, <24>, <25>, <26>, <27>, <28>, <29>,
+                                    <30>;
                        #dma-cells = <1>;
                };
 
                        compatible = "microchip,mpfs-sd4hc", "cdns,sd4hc";
                        reg = <0x0 0x20008000 0x0 0x1000>;
                        interrupt-parent = <&plic>;
-                       interrupts = <88 89>;
+                       interrupts = <88>, <89>;
                        clocks = <&clkcfg 6>;
                        max-frequency = <200000000>;
                        status = "disabled";
                        compatible = "cdns,macb";
                        reg = <0x0 0x20110000 0x0 0x2000>;
                        interrupt-parent = <&plic>;
-                       interrupts = <64 65 66 67>;
+                       interrupts = <64>, <65>, <66>, <67>;
                        local-mac-address = [00 00 00 00 00 00];
                        clocks = <&clkcfg 4>, <&clkcfg 2>;
                        clock-names = "pclk", "hclk";
                        compatible = "cdns,macb";
                        reg = <0x0 0x20112000 0x0 0x2000>;
                        interrupt-parent = <&plic>;
-                       interrupts = <70 71 72 73>;
+                       interrupts = <70>, <71>, <72>, <73>;
                        local-mac-address = [00 00 00 00 00 00];
                        clocks = <&clkcfg 5>, <&clkcfg 2>;
                        status = "disabled";