On Wed, Mar 19, 2014 at 04:50:58PM +0000, Lorenzo Pieralisi wrote:
if (parse_dt_topology()) reset_cpu_topology();
If you want to leave ret there I do not care, I flag what I notice.
Oh, right. I was confused because you said it was unused when it clearly had a use. I find that ugly since it looks like the test is the wrong way around but whatever...
parse_cluster() return value issue I flagged up must be fixed though.
Already done along with the other stylistic stuff.