Re: [PATCH] of: Fix cpu node iterator to not ignore disabled cpu nodes

2018-11-01 Thread Christian Zigotzky
On 01 November 2018 at 12:08PM, Michael Ellerman wrote: Rob Herring writes: In most cases, nodes with 'status = "disabled";' are treated as if the node is not present though it is a common bug to forget to check that. However, cpu nodes are different in that "disabled" simply means offline

Re: [PATCH] of: Fix cpu node iterator to not ignore disabled cpu nodes

2018-11-01 Thread Christian Zigotzky
On 01 November 2018 at 12:08PM, Michael Ellerman wrote: Rob Herring writes: In most cases, nodes with 'status = "disabled";' are treated as if the node is not present though it is a common bug to forget to check that. However, cpu nodes are different in that "disabled" simply means offline

Re: [PATCH] of: Fix cpu node iterator to not ignore disabled cpu nodes

2018-11-01 Thread Michael Ellerman
Rob Herring writes: > In most cases, nodes with 'status = "disabled";' are treated as if the > node is not present though it is a common bug to forget to check that. > However, cpu nodes are different in that "disabled" simply means offline > and the OS can bring the CPU core online. Commit

Re: [PATCH] of: Fix cpu node iterator to not ignore disabled cpu nodes

2018-11-01 Thread Michael Ellerman
Rob Herring writes: > In most cases, nodes with 'status = "disabled";' are treated as if the > node is not present though it is a common bug to forget to check that. > However, cpu nodes are different in that "disabled" simply means offline > and the OS can bring the CPU core online. Commit

Re: [PATCH] of: Fix cpu node iterator to not ignore disabled cpu nodes

2018-10-31 Thread Christian Zigotzky
Thank you. I will test this patch tomorrow. — Christian Sent from my iPhone > On 31. Oct 2018, at 15:42, Rob Herring wrote: > > In most cases, nodes with 'status = "disabled";' are treated as if the > node is not present though it is a common bug to forget to check that. > However, cpu nodes

Re: [PATCH] of: Fix cpu node iterator to not ignore disabled cpu nodes

2018-10-31 Thread Christian Zigotzky
Thank you. I will test this patch tomorrow. — Christian Sent from my iPhone > On 31. Oct 2018, at 15:42, Rob Herring wrote: > > In most cases, nodes with 'status = "disabled";' are treated as if the > node is not present though it is a common bug to forget to check that. > However, cpu nodes

[PATCH] of: Fix cpu node iterator to not ignore disabled cpu nodes

2018-10-31 Thread Rob Herring
In most cases, nodes with 'status = "disabled";' are treated as if the node is not present though it is a common bug to forget to check that. However, cpu nodes are different in that "disabled" simply means offline and the OS can bring the CPU core online. Commit f1f207e43b8a ("of: Add cpu node

[PATCH] of: Fix cpu node iterator to not ignore disabled cpu nodes

2018-10-31 Thread Rob Herring
In most cases, nodes with 'status = "disabled";' are treated as if the node is not present though it is a common bug to forget to check that. However, cpu nodes are different in that "disabled" simply means offline and the OS can bring the CPU core online. Commit f1f207e43b8a ("of: Add cpu node