diff options
| author | Zhen Lei <thunder.leizhen@huawei.com> | 2016-09-01 14:54:53 +0800 |
|---|---|---|
| committer | Will Deacon <will.deacon@arm.com> | 2016-09-09 14:59:08 +0100 |
| commit | 84b14256c18c967afd3cf4ee2df09535587154e0 (patch) | |
| tree | 901440b13550ffccc512aae2b50d517d6a67a48d /drivers/of | |
| parent | 16a82f06c40301045e4c05297ea93b85595dbbc4 (diff) | |
| download | linux-84b14256c18c967afd3cf4ee2df09535587154e0.tar.bz2 | |
of/numa: fix a memory@ node can only contains one memory block
For a normal memory@ devicetree node, its reg property can contains more
memory blocks.
Because we don't known how many memory blocks maybe contained, so we try
from index=0, increase 1 until error returned(the end).
Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Will Deacon <will.deacon@arm.com>
Diffstat (limited to 'drivers/of')
| -rw-r--r-- | drivers/of/of_numa.c | 29 |
1 files changed, 10 insertions, 19 deletions
diff --git a/drivers/of/of_numa.c b/drivers/of/of_numa.c index fb71b4ef1eeb..7b3fbdc0b278 100644 --- a/drivers/of/of_numa.c +++ b/drivers/of/of_numa.c @@ -63,13 +63,9 @@ static int __init of_numa_parse_memory_nodes(void) struct device_node *np = NULL; struct resource rsrc; u32 nid; - int r = 0; - - for (;;) { - np = of_find_node_by_type(np, "memory"); - if (!np) - break; + int i, r; + for_each_node_by_type(np, "memory") { r = of_property_read_u32(np, "numa-node-id", &nid); if (r == -EINVAL) /* @@ -78,23 +74,18 @@ static int __init of_numa_parse_memory_nodes(void) * "numa-node-id" property */ continue; - else if (r) - /* some other error */ - break; - r = of_address_to_resource(np, 0, &rsrc); - if (r) { - pr_err("NUMA: bad reg property in memory node\n"); - break; - } + for (i = 0; !r && !of_address_to_resource(np, i, &rsrc); i++) + r = numa_add_memblk(nid, rsrc.start, rsrc.end + 1); - r = numa_add_memblk(nid, rsrc.start, rsrc.end + 1); - if (r) - break; + if (!i || r) { + of_node_put(np); + pr_err("NUMA: bad property in memory node\n"); + return r ? : -EINVAL; + } } - of_node_put(np); - return r; + return 0; } static int __init of_numa_parse_distance_map_v1(struct device_node *map) |