From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lorenzo Pieralisi Subject: Re: [RFC PATCH v4 01/18] ARM: kernel: fix arm_dt_init_cpu_maps() to skip non-cpu nodes Date: Fri, 17 May 2013 18:04:23 +0100 Message-ID: <20130517170422.GF27589@e102568-lin.cambridge.arm.com> References: <1368804061-4421-1-git-send-email-lorenzo.pieralisi@arm.com> <1368804061-4421-2-git-send-email-lorenzo.pieralisi@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Content-Disposition: inline List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: "devicetree-discuss" To: Rob Herring Cc: Nicolas Pitre , Jon Medhurst , Kukjin Kim , Russell King , Andrew Lunn , Viresh Kumar , "devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org" , Barry Song , Will Deacon , Sekhar Nori , Simon Horman , "rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org" , Amit Kucheria , Catalin Marinas , "grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org" , David Brown , Vinayak Kale , Lennert Buytenhek , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , Magnus Damm List-Id: devicetree@vger.kernel.org On Fri, May 17, 2013 at 05:31:18PM +0100, Rob Herring wrote: > On Fri, May 17, 2013 at 10:20 AM, Lorenzo Pieralisi > wrote: > > The introduction of the cpu-map topology node in the cpus node implies > > that cpus node might have children that are not cpu nodes. The DT > > parsing code needs updating otherwise it would check for cpu nodes > > properties in nodes that are not required to contain them, resulting > > in warnings that have no bearing on bindings defined in the dts source file. > > Great, so a new DT with cpu-map entries may not work with old kernels. > Please check the behavior. This should go to stable kernels. You are right, and I do not see any other solution if we want the cpu-map node to live in the cpus node; at the time we added the cpus/cpu bindings we thought that the cpus node's children would be restricted to cpu nodes, and well, we are changing that, this is one of the consequences. Yes, this patch should go to stable kernels, I will do that. Lorenzo > Rob > > > > > Signed-off-by: Lorenzo Pieralisi > > --- > > arch/arm/kernel/devtree.c | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/arch/arm/kernel/devtree.c b/arch/arm/kernel/devtree.c > > index 5af04f6..904cad5 100644 > > --- a/arch/arm/kernel/devtree.c > > +++ b/arch/arm/kernel/devtree.c > > @@ -92,6 +92,9 @@ void __init arm_dt_init_cpu_maps(void) > > for_each_child_of_node(cpus, cpu) { > > u32 hwid; > > > > + if (of_node_cmp(cpu->type, "cpu")) > > + continue; > > + > > pr_debug(" * %s...\n", cpu->full_name); > > /* > > * A device tree containing CPU nodes with missing "reg" > > -- > > 1.8.2.2 > > > > > > _______________________________________________ > > devicetree-discuss mailing list > > devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org > > https://lists.ozlabs.org/listinfo/devicetree-discuss >