[PATCH 2/3] of/numa: fix a memory@ dt node can only contains one memory block

From: Zhen Lei
Date: Wed May 25 2016 - 22:44:58 EST


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@xxxxxxxxxx>
---
drivers/of/of_numa.c | 30 ++++++++++++++++++++----------
1 file changed, 20 insertions(+), 10 deletions(-)

diff --git a/drivers/of/of_numa.c b/drivers/of/of_numa.c
index 21d831f..2c5f249 100644
--- a/drivers/of/of_numa.c
+++ b/drivers/of/of_numa.c
@@ -63,7 +63,7 @@ static int __init of_numa_parse_memory_nodes(void)
struct device_node *np = NULL;
struct resource rsrc;
u32 nid;
- int r = 0;
+ int i, r = 0;

for (;;) {
np = of_find_node_by_type(np, "memory");
@@ -82,17 +82,27 @@ static int __init of_numa_parse_memory_nodes(void)
/* 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; ; i++) {
+ r = of_address_to_resource(np, i, &rsrc);
+ if (r) {
+ /* reached the end of of_address */
+ if (i > 0) {
+ r = 0;
+ break;
+ }
+
+ pr_err("NUMA: bad reg property in memory node\n");
+ goto finished;
+ }
+
+ r = numa_add_memblk(nid, rsrc.start,
+ rsrc.end - rsrc.start + 1);
+ if (r)
+ goto finished;
}
-
- r = numa_add_memblk(nid, rsrc.start,
- rsrc.end - rsrc.start + 1);
- if (r)
- break;
}
+
+finished:
of_node_put(np);

return r;
--
2.5.0