samsung:board: dram_init(): alloc memory to store fdt dram info

For CONFIG_OF_MULTI the dram_init() function gets memory information
from the device tree blob by the call to fdtdec_decode_memory().
The same is done second time in dram_init_banksize() - this was the
simplest because board info structure is not initialized at dram_init()
stage.

This change uses malloc and gd->priv pointer to store the memory banks
parameters and next uses it in dram_init_banksize().

Thanks to this change, the boot time decreases, because, the function
fdtdec_decode_memory() is called only once.

Change-Id: I8f357c6a641d4440f32d2fe24d7e1e2dc5bc4a3d
Signed-off-by: Przemyslaw Marczak <p.marczak@samsung.com>

diff --git a/board/samsung/common/board.c b/board/samsung/common/board.c
index dc729d5..dce5190 100644 (file)
--- a/board/samsung/common/board.c
+++ b/board/samsung/common/board.c
@@ -21,10 +21,12 @@
 #include <asm/arch/pinmux.h>
 #include <asm/arch/power.h>
 #include <asm/arch/system.h>
+#include <malloc.h>
 #include <power/pmic.h>
 #include <asm/arch/sromc.h>
 #include <lcd.h>
 #include <samsung/misc.h>
+#include <samsung/multi-plat.h>
 
 DECLARE_GLOBAL_DATA_PTR;
 
@@ -97,16 +99,19 @@ int dram_init(void)
 {
        int i;
 #ifdef CONFIG_FDTDEC_MEMORY
-       int dram_banks;
+       struct memory_info *mem;
 
-       uint64_t mem_size[CONFIG_NR_DRAM_BANKS];
+       mem = malloc(sizeof(struct memory_info));
 
-       dram_banks = fdtdec_decode_memory(gd->fdt_blob, NULL, mem_size,
+       mem->banks = fdtdec_decode_memory(gd->fdt_blob, &mem->addr[0],
+                                         &mem->size[0],
                                          CONFIG_NR_DRAM_BANKS);
 
-       for (i = 0; i < dram_banks; i++)
-               gd->ram_size += (uint32_t)mem_size[i];
+       for (i = 0; i < mem->banks; i++)
+               gd->ram_size += (uint32_t)mem->size[i];
 
+       /* Save the mem_info for dram_init_banksize() */
+       gd->priv = mem;
 #else
        u32 addr;
 
@@ -122,17 +127,11 @@ void dram_init_banksize(void)
 {
        int i;
 #ifdef CONFIG_FDTDEC_MEMORY
-       int dram_banks;
+       struct memory_info *mem = gd->priv;
 
-       uint64_t mem_addr[CONFIG_NR_DRAM_BANKS];
-       uint64_t mem_size[CONFIG_NR_DRAM_BANKS];
-
-       dram_banks = fdtdec_decode_memory(gd->fdt_blob, mem_addr, mem_size,
-                                         CONFIG_NR_DRAM_BANKS);
-
-       for (i = 0; i < dram_banks; i++) {
-               gd->bd->bi_dram[i].start = (uint32_t)mem_addr[i];
-               gd->bd->bi_dram[i].size = (uint32_t)mem_size[i];
+       for (i = 0; i < mem->banks ; i++) {
+               gd->bd->bi_dram[i].start = (uint32_t)mem->addr[i];
+               gd->bd->bi_dram[i].size = (uint32_t)mem->size[i];
        }
 #else
        u32 addr, size;

diff --git a/include/samsung/multi-plat.h b/include/samsung/multi-plat.h
new file mode 100644 (file)
index 0000000..29f5f50
--- /dev/null
+++ b/include/samsung/multi-plat.h
@@ -0,0 +1,13 @@
+#ifndef __INCLUDE__MULTI_PLAT__H__
+#define __INCLUDE__MULTI_PLAT__H__
+
+#include <common.h>
+
+#ifdef CONFIG_OF_MULTI
+struct memory_info {
+       uint64_t addr[CONFIG_NR_DRAM_BANKS];
+       uint64_t size[CONFIG_NR_DRAM_BANKS];
+       uint32_t banks;
+};
+#endif
+#endif
\ No newline at end of file