Revert "fdt: fdtdec_setup_memory_banksize() use livetree"
authorMichal Simek <michal.simek@xilinx.com>
Wed, 3 Oct 2018 13:53:52 +0000 (15:53 +0200)
committerTom Rini <trini@konsulko.com>
Mon, 15 Oct 2018 20:22:49 +0000 (16:22 -0400)
This reverts commit c35a7d375ec8f0a8ee343ae4868be3242172632e.
This commit is breaking SPL on zc706.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Tested-by: Patrice Chotard <patrice.chotard@st.com> [on STM32F746-disco]
lib/fdtdec.c

index b1b39254bfdf4a7fc5c93f7a6c6ba6c500b2ec06..a420ba18854fb48be8cc18bd313e96c84d64e15e 100644 (file)
@@ -15,7 +15,6 @@
 #include <serial.h>
 #include <asm/sections.h>
 #include <linux/ctype.h>
-#include <linux/ioport.h>
 #include <linux/lzo.h>
 
 DECLARE_GLOBAL_DATA_PTR;
@@ -1099,34 +1098,41 @@ int fdtdec_setup_mem_size_base(void)
 
 #if defined(CONFIG_NR_DRAM_BANKS)
 
-static ofnode get_next_memory_node(ofnode mem)
+static int get_next_memory_node(const void *blob, int mem)
 {
        do {
-               mem = ofnode_by_prop_value(mem, "device_type", "memory", 7);
-       } while (ofnode_valid(mem) && !ofnode_is_available(mem));
+               mem = fdt_node_offset_by_prop_value(gd->fdt_blob, mem,
+                                                   "device_type", "memory", 7);
+       } while (!fdtdec_get_is_enabled(blob, mem));
 
        return mem;
 }
 
 int fdtdec_setup_memory_banksize(void)
 {
-       int bank, reg = 0;
-       struct resource res;
-       ofnode mem;
+       int bank, ret, mem, reg = 0;
+       struct fdt_resource res;
 
-       mem = get_next_memory_node(ofnode_null());
-       if (!ofnode_valid(mem))
-               goto missing_node;
+       mem = get_next_memory_node(gd->fdt_blob, -1);
+       if (mem < 0) {
+               debug("%s: Missing /memory node\n", __func__);
+               return -EINVAL;
+       }
 
        for (bank = 0; bank < CONFIG_NR_DRAM_BANKS; bank++) {
-               while (ofnode_read_resource(mem, reg++, &res)) {
+               ret = fdt_get_resource(gd->fdt_blob, mem, "reg", reg++, &res);
+               if (ret == -FDT_ERR_NOTFOUND) {
                        reg = 0;
-                       mem = get_next_memory_node(mem);
-                       if (!ofnode_valid(mem)) {
-                               if (bank)
-                                       return 0;
-                               goto missing_node;
-                       }
+                       mem = get_next_memory_node(gd->fdt_blob, mem);
+                       if (mem == -FDT_ERR_NOTFOUND)
+                               break;
+
+                       ret = fdt_get_resource(gd->fdt_blob, mem, "reg", reg++, &res);
+                       if (ret == -FDT_ERR_NOTFOUND)
+                               break;
+               }
+               if (ret != 0) {
+                       return -EINVAL;
                }
 
                gd->bd->bi_dram[bank].start = (phys_addr_t)res.start;
@@ -1140,10 +1146,6 @@ int fdtdec_setup_memory_banksize(void)
        }
 
        return 0;
-
-missing_node:
-       debug("%s: Missing /memory node\n", __func__);
-       return -EINVAL;
 }
 #endif