Commit bed90316 authored by Marek Szyprowski's avatar Marek Szyprowski Committed by Krzysztof Kozlowski

arm64: dts: exynos: Revert "Remove unneeded address space mapping for soc node"

Commit ef72171b ("arm64: dts: exynos: Remove unneeded address space
mapping for soc node") changed the address and size cells in root node from
2 to 1, but /memory nodes for the affected boards were not updated. This
went unnoticed on Exynos5433-based TM2(e) boards, because they use u-boot,
which updates /memory node to the correct values. On the other hand, the
mentioned commit broke boot on Exynos7-based Espresso board, which
bootloader doesn't touch /memory node at all.

This patch reverts commit ef72171b ("arm64: dts: exynos: Remove
unneeded address space mapping for soc node"), so Exynos5433 and Exynos7
SoCs again matches other ARM64 platforms with 64bit mappings in root
node.
Reported-by: default avatarAlim Akhtar <alim.akhtar@samsung.com>
Fixes: ef72171b ("arm64: dts: exynos: Remove unneeded address space mapping for soc node")
Signed-off-by: default avatarMarek Szyprowski <m.szyprowski@samsung.com>
Cc: <stable@vger.kernel.org> # 5.3.x: 72ddcf6a arm64: dts: exynos: Move GPU under /soc node for Exynos5433
Cc: <stable@vger.kernel.org> # 5.3.x: ede87c3a arm64: dts: exynos: Move GPU under /soc node for Exynos7
Cc: <stable@vger.kernel.org> # 4.18.x
Tested-by: default avatarAlim Akhtar <alim.akhtar@samsung.com>
Signed-off-by: default avatarKrzysztof Kozlowski <krzk@kernel.org>
parent ede87c3a
...@@ -18,8 +18,8 @@ ...@@ -18,8 +18,8 @@
/ { / {
compatible = "samsung,exynos5433"; compatible = "samsung,exynos5433";
#address-cells = <1>; #address-cells = <2>;
#size-cells = <1>; #size-cells = <2>;
interrupt-parent = <&gic>; interrupt-parent = <&gic>;
...@@ -260,7 +260,7 @@ soc: soc { ...@@ -260,7 +260,7 @@ soc: soc {
compatible = "simple-bus"; compatible = "simple-bus";
#address-cells = <1>; #address-cells = <1>;
#size-cells = <1>; #size-cells = <1>;
ranges; ranges = <0x0 0x0 0x0 0x18000000>;
chipid@10000000 { chipid@10000000 {
compatible = "samsung,exynos4210-chipid"; compatible = "samsung,exynos4210-chipid";
......
...@@ -12,8 +12,8 @@ ...@@ -12,8 +12,8 @@
/ { / {
compatible = "samsung,exynos7"; compatible = "samsung,exynos7";
interrupt-parent = <&gic>; interrupt-parent = <&gic>;
#address-cells = <1>; #address-cells = <2>;
#size-cells = <1>; #size-cells = <2>;
aliases { aliases {
pinctrl0 = &pinctrl_alive; pinctrl0 = &pinctrl_alive;
...@@ -87,7 +87,7 @@ soc: soc { ...@@ -87,7 +87,7 @@ soc: soc {
compatible = "simple-bus"; compatible = "simple-bus";
#address-cells = <1>; #address-cells = <1>;
#size-cells = <1>; #size-cells = <1>;
ranges; ranges = <0 0 0 0x18000000>;
chipid@10000000 { chipid@10000000 {
compatible = "samsung,exynos4210-chipid"; compatible = "samsung,exynos4210-chipid";
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment