From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752544AbcHIJnE (ORCPT ); Tue, 9 Aug 2016 05:43:04 -0400 Received: from foss.arm.com ([217.140.101.70]:40709 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752185AbcHIJnC (ORCPT ); Tue, 9 Aug 2016 05:43:02 -0400 Date: Tue, 9 Aug 2016 10:42:38 +0100 From: Juri Lelli To: linux-kernel@vger.kernel.org Cc: linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, peterz@infradead.org, vincent.guittot@linaro.org, robh+dt@kernel.org, mark.rutland@arm.com, linux@arm.linux.org.uk, sudeep.holla@arm.com, lorenzo.pieralisi@arm.com, catalin.marinas@arm.com, will.deacon@arm.com, morten.rasmussen@arm.com, dietmar.eggemann@arm.com, broonie@kernel.org Subject: Re: [PATCH v6 4/8] arm64: parse cpu capacity-dmips-mhz from DT Message-ID: <20160809094238.GA23236@e106622-lin> References: <1468932048-31635-1-git-send-email-juri.lelli@arm.com> <1468932048-31635-5-git-send-email-juri.lelli@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1468932048-31635-5-git-send-email-juri.lelli@arm.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, this patch needs the fix below, since platforms can boot without a cpu-map. On 19/07/16 13:40, Juri Lelli wrote: [...] > @@ -185,6 +328,8 @@ static int __init parse_dt_topology(void) > if (ret != 0) > goto out_map; > > + normalize_cpu_capacity(); > + > /* > * Check that all cores are in the topology; the SMP code will > * only mark cores described in the DT as possible. > -- --->8--- arch/arm64/kernel/topology.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/arch/arm64/kernel/topology.c b/arch/arm64/kernel/topology.c index 56bbd6d41daf..288f511a5f29 100644 --- a/arch/arm64/kernel/topology.c +++ b/arch/arm64/kernel/topology.c @@ -394,8 +394,10 @@ static int __init parse_dt_topology(void) * cluster with restricted subnodes. */ map = of_get_child_by_name(cn, "cpu-map"); - if (!map) + if (!map) { + cap_parsing_failed = true; goto out; + } ret = parse_cluster(map, 0); if (ret != 0) -- 2.7.0