Re: [PATCH v4 2/5] drivers/of: do not add memory for unavailable nodes

2016-10-24 Thread Reza Arbab
On Mon, Oct 24, 2016 at 09:24:04PM +1100, Michael Ellerman wrote: The code already looks for "linux,usable-memory" in preference to "reg". Can you use that instead? Yes, we could set the size of "linux,usable-memory" to zero instead of setting status to "disabled". I'll send a v5 of this set

Re: [PATCH v4 2/5] drivers/of: do not add memory for unavailable nodes

2016-10-24 Thread Michael Ellerman
Alistair Popple writes: > Hi Reza, > > On Thu, 6 Oct 2016 01:36:32 PM Reza Arbab wrote: >> Respect the standard dt "status" property when scanning memory nodes in >> early_init_dt_scan_memory(), so that if the node is unavailable, no >> memory will be added. > > What happens if a kernel without t

Re: [PATCH v4 2/5] drivers/of: do not add memory for unavailable nodes

2016-10-22 Thread Reza Arbab
Hi Alistair, On Fri, Oct 21, 2016 at 05:22:54PM +1100, Alistair Popple wrote: From what I can tell it seems that kernels without this patch will try and use this memory even if it is marked in the device-tree as status="disabled" which could lead to problems for older kernels when we start exp

Re: [PATCH v4 2/5] drivers/of: do not add memory for unavailable nodes

2016-10-20 Thread Alistair Popple
Hi Reza, On Thu, 6 Oct 2016 01:36:32 PM Reza Arbab wrote: > Respect the standard dt "status" property when scanning memory nodes in > early_init_dt_scan_memory(), so that if the node is unavailable, no > memory will be added. What happens if a kernel without this patch is booted on a system with

Re: [PATCH v4 2/5] drivers/of: do not add memory for unavailable nodes

2016-10-11 Thread Rob Herring
On Thu, Oct 6, 2016 at 1:36 PM, Reza Arbab wrote: > Respect the standard dt "status" property when scanning memory nodes in > early_init_dt_scan_memory(), so that if the node is unavailable, no > memory will be added. > > The use case at hand is accelerator or device memory, which may be > unusabl

[PATCH v4 2/5] drivers/of: do not add memory for unavailable nodes

2016-10-06 Thread Reza Arbab
Respect the standard dt "status" property when scanning memory nodes in early_init_dt_scan_memory(), so that if the node is unavailable, no memory will be added. The use case at hand is accelerator or device memory, which may be unusable until post-boot initialization of the memory link. Such a no