ARM: tegra: Rename GPU node on Tegra124

In order to be able to pass DT schema validation, change the GPU nodes'
unit-address to the standard notation. Previously this was using a "0,"
prefix that originated from a time when the top-level device tree node
contained #address-cells = <2>.

Note that this technically breaks backwards-compatibility with certain
older versions of the U-Boot bootloader because early versions used a
hard-coded DT path lookup to find the GPU node and perform some fixups
on it. However, this was changed to a compatible string based lookup in
April 2016, so it's reasonable to expect people to update U-Boot on the
systems that they want to use this updated kernel DTB with.

Signed-off-by: Thierry Reding <treding@nvidia.com>

diff --git a/arch/arm/boot/dts/tegra124-apalis-v1.2.dtsi b/arch/arm/boot/dts/tegra124-apalis-v1.2.dtsi
index 4ba4d52..f00ef4d 100644 (file)
--- a/arch/arm/boot/dts/tegra124-apalis-v1.2.dtsi
+++ b/arch/arm/boot/dts/tegra124-apalis-v1.2.dtsi
@@ -57,7 +57,7 @@
                };
        };
 
-       gpu@0,57000000 {
+       gpu@57000000 {
                /*
                 * Node left disabled on purpose - the bootloader will enable
                 * it after having set the VPR up

diff --git a/arch/arm/boot/dts/tegra124-apalis.dtsi b/arch/arm/boot/dts/tegra124-apalis.dtsi
index 3760744..e6d2581 100644 (file)
--- a/arch/arm/boot/dts/tegra124-apalis.dtsi
+++ b/arch/arm/boot/dts/tegra124-apalis.dtsi
@@ -56,7 +56,7 @@
                };
        };
 
-       gpu@0,57000000 {
+       gpu@57000000 {
                /*
                 * Node left disabled on purpose - the bootloader will enable
                 * it after having set the VPR up

diff --git a/arch/arm/boot/dts/tegra124-jetson-tk1.dts b/arch/arm/boot/dts/tegra124-jetson-tk1.dts
index c3edb33..f42191f 100644 (file)
--- a/arch/arm/boot/dts/tegra124-jetson-tk1.dts
+++ b/arch/arm/boot/dts/tegra124-jetson-tk1.dts
@@ -72,7 +72,7 @@
                status = "okay";
        };
 
-       gpu@0,57000000 {
+       gpu@57000000 {
                /*
                 * Node left disabled on purpose - the bootloader will enable
                 * it after having set the VPR up

diff --git a/arch/arm/boot/dts/tegra124-nyan.dtsi b/arch/arm/boot/dts/tegra124-nyan.dtsi
index 2d045d1..998b129 100644 (file)
--- a/arch/arm/boot/dts/tegra124-nyan.dtsi
+++ b/arch/arm/boot/dts/tegra124-nyan.dtsi
@@ -61,7 +61,7 @@
                };
        };
 
-       gpu@0,57000000 {
+       gpu@57000000 {
                status = "okay";
 
                vdd-supply = <&vdd_gpu>;

diff --git a/arch/arm/boot/dts/tegra124-venice2.dts b/arch/arm/boot/dts/tegra124-venice2.dts
index 5f17a2f..7985d1c 100644 (file)
--- a/arch/arm/boot/dts/tegra124-venice2.dts
+++ b/arch/arm/boot/dts/tegra124-venice2.dts
@@ -51,7 +51,7 @@
                };
        };
 
-       gpu@0,57000000 {
+       gpu@57000000 {
                /*
                 * Node left disabled on purpose - the bootloader will enable
                 * it after having set the VPR up

diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi
index f4ac0c3..d0ec1a3 100644 (file)
--- a/arch/arm/boot/dts/tegra124.dtsi
+++ b/arch/arm/boot/dts/tegra124.dtsi
@@ -223,12 +223,7 @@
                interrupt-parent = <&gic>;
        };
 
-       /*
-        * Please keep the following 0, notation in place as a former mainline
-        * U-Boot version was looking for that particular notation in order to
-        * perform required fix-ups on that GPU node.
-        */
-       gpu@0,57000000 {
+       gpu@57000000 {
                compatible = "nvidia,gk20a";
                reg = <0x0 0x57000000 0x0 0x01000000>,
                      <0x0 0x58000000 0x0 0x01000000>;