ipq40xx: tidy up dts-v1 statements
authorAdrian Schmutzler <freifunk@adrianschmutzler.de>
Fri, 25 Sep 2020 19:10:41 +0000 (21:10 +0200)
committerAdrian Schmutzler <freifunk@adrianschmutzler.de>
Fri, 25 Sep 2020 19:10:41 +0000 (21:10 +0200)
The "/dts-v1/;" identifier is supposed to be present once at the
top of a device tree file after the includes have been processed.

Therefore, adding it to a DTS _and_ a DTSI file is actually wrong,
as it will be present twice then (though the compiler does not
complain about it).

In ipq40xx, the dts-v1 statement is already included in
qcom-ipq4019.dtsi, so we don't have to add it anywhere at all.
However, based on the conditions stated above, this requires
qcom-ipq4019.dtsi to be included as the first file in any DTS(I).

Consequently, this patch removes all cases of dts-v1 for the
ipq40xx target, and moves the includes accordingly where necessary.

While at it, remove a few obviously unneeded includes on the way.

Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
target/linux/ipq40xx/files/arch/arm/boot/dts/qcom-ipq4018-ex6100v2.dts
target/linux/ipq40xx/files/arch/arm/boot/dts/qcom-ipq4018-ex6150v2.dts
target/linux/ipq40xx/files/arch/arm/boot/dts/qcom-ipq4018-wrtq-329acn.dts
target/linux/ipq40xx/files/arch/arm/boot/dts/qcom-ipq4019-ea8300.dts
target/linux/ipq40xx/files/arch/arm/boot/dts/qcom-ipq4019-map-ac2200.dts
target/linux/ipq40xx/files/arch/arm/boot/dts/qcom-ipq4019-mr8300.dts
target/linux/ipq40xx/files/arch/arm/boot/dts/qcom-ipq4019-u4019-32m.dts
target/linux/ipq40xx/files/arch/arm/boot/dts/qcom-ipq4019-xx8300.dtsi

index 2b93dcbb99ef4ae3163d1116a14cdb1e00ea73f5..1495c64da9591f5f6623b74338244cbcfb6501bd 100644 (file)
@@ -16,8 +16,6 @@
  */
 
 #include "qcom-ipq4018-ex61x0v2.dtsi"
-#include <dt-bindings/gpio/gpio.h>
-#include <dt-bindings/input/input.h>
 
 / {
        model = "Netgear EX6100v2";
index f1c44c1ae8b8dc8e97e7cd162e4c0f568eb9276d..ce24466e5487ac342756c28eca7ce1811044da4d 100644 (file)
@@ -16,8 +16,6 @@
  */
 
 #include "qcom-ipq4018-ex61x0v2.dtsi"
-#include <dt-bindings/gpio/gpio.h>
-#include <dt-bindings/input/input.h>
 
 / {
        model = "Netgear EX6150v2";
index faac71489c009f8f9f16b991424ab0a23cf8e269..2465348a821ece25ba56be86adcb92ab71fc1fbc 100644 (file)
@@ -1,12 +1,10 @@
 // SPDX-License-Identifier: GPL-2.0-or-later OR MIT
-/dts-v1/;
 
+#include "qcom-ipq4019.dtsi"
 #include <dt-bindings/gpio/gpio.h>
 #include <dt-bindings/input/input.h>
 #include <dt-bindings/soc/qcom,tcsr.h>
 
-#include "qcom-ipq4019.dtsi"
-
 / {
        model = "Luma Home WRTQ-329ACN";
        compatible = "luma,wrtq-329acn";
index 9c1e0f04bbea500d506e64aa1be14bf2f96802e7..d8acf36c4a59b936baacfbdd4595261f662c26fd 100644 (file)
@@ -1,7 +1,5 @@
 // SPDX-License-Identifier: GPL-2.0-or-later OR MIT
 
-/dts-v1/;
-
 #include "qcom-ipq4019-xx8300.dtsi"
 
 / {
index 06d0277b0164a2e09028f2960fee0ce54845b9f6..92aaa927e2a6170fccbcb654c392a2a30ec69a25 100644 (file)
@@ -1,5 +1,4 @@
 // SPDX-License-Identifier: GPL-2.0-or-later OR MIT
-/dts-v1/;
 
 #include "qcom-ipq4019.dtsi"
 #include <dt-bindings/gpio/gpio.h>
index 4818a2df598187879593ea1e77b63a1433a35937..a09730a9e888730ae0a51cbf0d00f9de1a041da2 100644 (file)
@@ -1,7 +1,5 @@
 // SPDX-License-Identifier: GPL-2.0-or-later OR MIT
 
-/dts-v1/;
-
 #include "qcom-ipq4019-xx8300.dtsi"
 
 / {
index 63fbfb65c1724a7dd8f5add47f231837f4443243..811c82b9c53db06eaa423131a880ff85d2572041 100644 (file)
@@ -1,7 +1,5 @@
 // SPDX-License-Identifier: GPL-2.0-or-later OR MIT
 
-/dts-v1/;
-
 #include "qcom-ipq4019-u4019.dtsi"
 #include <dt-bindings/gpio/gpio.h>
 #include <dt-bindings/input/input.h>
index 77d495e670f06ced57bd669c47a15b875b952ba8..8f971e505c3c7d106b151fb1c47cbb0d9c0ee265 100644 (file)
@@ -8,8 +8,6 @@
  *
  */
 
-/dts-v1/;
-
 #include "qcom-ipq4019.dtsi"
 #include <dt-bindings/gpio/gpio.h>
 #include <dt-bindings/input/input.h>