Re: [PATCH 10/13] ARM: dts: Add initial device tree support for Exynos5420

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Mark,

On 06/11/2013 07:41 PM, Mark Rutland wrote:
On Tue, Jun 11, 2013 at 02:35:38PM +0100, Chander Kashyap wrote:
On 6 June 2013 22:04, Mark Rutland <mark.rutland@xxxxxxx> wrote:
Hi,

diff --git a/arch/arm/boot/dts/exynos5420-smdk5420.dts b/arch/arm/boot/dts/exynos5420-smdk5420.dts
new file mode 100644
index 0000000..b14e775
--- /dev/null
+++ b/arch/arm/boot/dts/exynos5420-smdk5420.dts
@@ -0,0 +1,40 @@
+/*
+ * SAMSUNG SMDK5420 board device tree source
+ *
+ * Copyright (c) 2013 Samsung Electronics Co., Ltd.
+ *           http://www.samsung.com
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+*/
+
+/dts-v1/;
+/include/ "exynos5420.dtsi"
+
+/ {
+     model = "Samsung SMDK5420 board based on EXYNOS5420";
+     compatible = "samsung,smdk5420", "samsung,exynos5420";
+
+     memory {
+             reg =   <0x20000000 0x10000000
+                      0x30000000 0x10000000
+                      0x40000000 0x10000000
+                      0x50000000 0x10000000
+                      0x60000000 0x10000000
+                      0x70000000 0x10000000
+                      0x80000000 0x10000000
+                      0x90000000 0x10000000>;
+     };

As there are only 28 bits for memory bank size in memory.h.
Hence each bank is split into 256MB.

I don't think a kernel implementation choice should affect the dts.

If the kernel needs to carve up the memory, it should do it itself - it knows
what it needs. Splitting the banks manually as above is papering over the
problem.

I agree this looks a bit odd to split the memory regions into 256MiB banks. But we have had legacy systems which had a hole in the memory range, and we used sparse mem in the past for the same. If we modify this bank size now, we will break the functionality of those systems.

Also, with LPAE systems I changed this bank size to 2GiB in another patch. I think someone was working on cleaning up sparse mem for exynos(I was quite busy internally and didn't follow up much on mailing list). So I would like to keep this bank size as 256MiB for non-LPAE systems at the moment, until such a cleanup gets merged.

Copy: kgene - exynos maintainer

Mr. Kim, let us know your opinion on this.

Regards,
Subash



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

--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux