Re: [PATCH] powerpc/mm: simplify loop control in parse_numa_properties()

2016-10-14 Thread Reza Arbab
On Fri, Oct 14, 2016 at 11:33:12AM +1100, Michael Ellerman wrote: Reza Arbab writes: The flow of the main loop in parse_numa_properties() is overly complicated. Simplify it to be less confusing and easier to read. No functional change. I think you're right, but it's not obvious from the diff.

Re: [PATCH] powerpc/mm: simplify loop control in parse_numa_properties()

2016-10-13 Thread Michael Ellerman
Reza Arbab writes: > The flow of the main loop in parse_numa_properties() is overly > complicated. Simplify it to be less confusing and easier to read. > No functional change. I think you're right, but it's not obvious from the diff. Please explain in more detail why the old and new code is equi

[PATCH] powerpc/mm: simplify loop control in parse_numa_properties()

2016-10-13 Thread Reza Arbab
The flow of the main loop in parse_numa_properties() is overly complicated. Simplify it to be less confusing and easier to read. No functional change. Signed-off-by: Reza Arbab --- arch/powerpc/mm/numa.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/arch/powerpc