ARM: tegra: define CONFIG_SYS_BOOTMAPSZ
authorStephen Warren <swarren@nvidia.com>
Thu, 20 Sep 2012 09:29:03 +0000 (09:29 +0000)
committerTom Warren <twarren@nvidia.com>
Mon, 29 Oct 2012 16:07:04 +0000 (09:07 -0700)
This define indicates the size of the memory region where it is safe
to place data passed to the Linux kernel (ATAGs, DTB, initrd). The
value needs to be:

a) Less than or equal to RAM size.
b) Small enough that the area is not within the kernel's highmem region,
   since the kernel cannot access ATAGs/DTB/initrd from highmem.
c) Large enough to hold the kernel+DTB+initrd.

256M seems large enough for (c) in most circumstances, and small enough
to satisfy (a) and (b) across any possible Tegra board. Note that the
user can override this value via environment variable "bootm_mapsize"
if needed.

The advantage of defining BOOTMAPSZ is that we no longer need to define
variable fdt_high in the default environment. Previously, we defined
this to prevent the DTB from being relocated to the very end of RAM,
which on most Tegra systems is within highmem, and hence which would
cause boot failures. A user can still define this variable themselves
if they want the FDT to be either left in-place wherever loaded, or
copied to some other specific location. Similarly, there should no
longer be a strict requirement for the user to define initrd_high if
using an initrd.

Signed-off-by: Stephen Warren <swarren@nvidia.com>
Signed-off-by: Tom Warren <twarren@nvidia.com>
include/configs/tegra-common-post.h
include/configs/tegra20-common.h

index 9698c23dad286f5dd07d15ad0d5cb09ba30ca5e1..5d1d3ddcb7c202e5a1b01e06e6aef05cd6985f29 100644 (file)
 #define CONFIG_EXTRA_ENV_SETTINGS \
        TEGRA_DEVICE_SETTINGS \
        "fdt_load=0x01000000\0" \
-       "fdt_high=01100000\0" \
        BOOTCMDS_COMMON
 
 /* overrides for SPL build here */
index dc7444d1cc77206a1b010bfdadcae345ec31b15e..70c5cfbba87f232c127c274e9c57c407f53027d7 100644 (file)
 #define CONFIG_SYS_TEXT_BASE   0x0010c000
 #define CONFIG_SYS_SDRAM_BASE  PHYS_SDRAM_1
 
+#define CONFIG_SYS_BOOTMAPSZ   (256 << 20) /* 256M */
+
 #define CONFIG_SYS_INIT_RAM_ADDR       CONFIG_STACKBASE
 #define CONFIG_SYS_INIT_RAM_SIZE       CONFIG_SYS_MALLOC_LEN
 #define CONFIG_SYS_INIT_SP_ADDR                (CONFIG_SYS_INIT_RAM_ADDR + \