On Tue, May 8, 2018 at 6:38 PM, James Hogan <jhogan@xxxxxxxxxx> wrote: > On Tue, Mar 27, 2018 at 08:14:35PM -0500, Rob Herring wrote: >> There's no need to scan /memory nodes twice. The DT core code scans >> nodes and adds memblocks already, so we can just use >> memblock_phys_mem_size() to see if we have any memory already setup. > > Hmm, on MIPS, early_init_dt_add_memory_arch() calls add_memory_region(), > which just modifies boot_mem_map. memblock isn't notified until after > plat_mem_setup() returns, in bootmem_init(). Yes, you're right. I guess first boot_mem_map needs to be converted to memblock. Rob
