diff mbox

[01/23] ARM: am335x: Add missing memory node in sl50 board

Message ID 1472639737-17586-2-git-send-email-javier@osg.samsung.com (mailing list archive)
State New, archived
Headers show

Commit Message

Javier Martinez Canillas Aug. 31, 2016, 10:35 a.m. UTC
The skeleton.dtsi file was removed in ARM64 for different reasons as
explained in commit ("3ebee5a2e141 arm64: dts: kill skeleton.dtsi").

These also applies to ARM and it will also allow to get rid of the
following DTC warnings in the future:

"Node /memory has a reg or ranges property, but no unit name"

But the sl50 board doesn't have a, so removing the  skeleton.dtsi
inclusion from am33xx.dtsi will cause a change in the compiled DTB.

The board has 512 MiB of RAM and its starting address is 0x80000000,
so add a proper memory device node in the DTS.

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

---

 arch/arm/boot/dts/am335x-sl50.dts | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Enric Balletbo Serra Aug. 31, 2016, 1:09 p.m. UTC | #1
2016-08-31 12:35 GMT+02:00 Javier Martinez Canillas <javier@osg.samsung.com>:
> The skeleton.dtsi file was removed in ARM64 for different reasons as
> explained in commit ("3ebee5a2e141 arm64: dts: kill skeleton.dtsi").
>
> These also applies to ARM and it will also allow to get rid of the
> following DTC warnings in the future:
>
> "Node /memory has a reg or ranges property, but no unit name"
>
> But the sl50 board doesn't have a, so removing the  skeleton.dtsi
> inclusion from am33xx.dtsi will cause a change in the compiled DTB.
>
> The board has 512 MiB of RAM and its starting address is 0x80000000,
> so add a proper memory device node in the DTS.
>
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
>
> ---
>
>  arch/arm/boot/dts/am335x-sl50.dts | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/arch/arm/boot/dts/am335x-sl50.dts b/arch/arm/boot/dts/am335x-sl50.dts
> index f802adb23ff9..e3be7a6bf8f1 100644
> --- a/arch/arm/boot/dts/am335x-sl50.dts
> +++ b/arch/arm/boot/dts/am335x-sl50.dts
> @@ -19,6 +19,11 @@
>                 };
>         };
>
> +       memory {
> +               device_type = "memory";
> +               reg = <0x80000000 0x20000000>; /* 512 MB */
> +       };
> +
>         chosen {
>                 stdout-path = &uart0;
>         };
> --
> 2.7.4
>

Acked-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>

Thanks,
 Enric
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/boot/dts/am335x-sl50.dts b/arch/arm/boot/dts/am335x-sl50.dts
index f802adb23ff9..e3be7a6bf8f1 100644
--- a/arch/arm/boot/dts/am335x-sl50.dts
+++ b/arch/arm/boot/dts/am335x-sl50.dts
@@ -19,6 +19,11 @@ 
 		};
 	};
 
+	memory {
+		device_type = "memory";
+		reg = <0x80000000 0x20000000>; /* 512 MB */
+	};
+
 	chosen {
 		stdout-path = &uart0;
 	};