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

From	Reza Arbab <arbab@linux.vnet.ibm.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v4 2/5] drivers/of: do not add memory for unavailable nodes
Date	2016-10-24 20:30 +0200
Message-ID	<svSat-2UD-7@gated-at.bofh.it> (permalink)
References	<splKh-4z5-5@gated-at.bofh.it> <splKi-4z5-11@gated-at.bofh.it> <suBv3-1O9-1@gated-at.bofh.it> <svKFX-6hE-33@gated-at.bofh.it>
Organization	IBM Linux Technology Center

Show all headers | View raw

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 which drops 1/5 and 2/5. That would be the 
only difference here.

>That would have the advantage that existing kernels already understand
>it.
>
>Another problem with using "status" is we could have device trees out
>there that have status = disabled and we don't know about it, and by
>changing the kernel to use that property we break people's systems.
>Though for memory nodes my guess is that's not true, but you never know ...

-- 
Reza Arbab

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH v4 2/5] drivers/of: do not add memory for unavailable nodes Alistair Popple <apopple@au1.ibm.com> - 2016-10-21 08:30 +0200
  Re: [PATCH v4 2/5] drivers/of: do not add memory for unavailable  nodes Reza Arbab <arbab@linux.vnet.ibm.com> - 2016-10-23 04:00 +0200
  Re: [PATCH v4 2/5] drivers/of: do not add memory for unavailable nodes Michael Ellerman <mpe@ellerman.id.au> - 2016-10-24 12:30 +0200
    Re: [PATCH v4 2/5] drivers/of: do not add memory for unavailable  nodes Reza Arbab <arbab@linux.vnet.ibm.com> - 2016-10-24 20:30 +0200

csiph-web