Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933217AbaD1UIi (ORCPT ); Mon, 28 Apr 2014 16:08:38 -0400 Received: from fw-tnat.austin.arm.com ([217.140.110.23]:15424 "EHLO collaborate-mta1.arm.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932931AbaD1UIX (ORCPT ); Mon, 28 Apr 2014 16:08:23 -0400 Message-ID: <1398706854.20390.8.camel@hornet> Subject: Re: [PATCH 1/3] driver core & of: Mark of_nodes of added device as populated From: Pawel Moll To: Christopher Covington Cc: Mark Rutland , devicetree@vger.kernel.org, Liam Girdwood , Mike Turquette , Samuel Ortiz , Arnd Bergmann , Ian Campbell , Dmitry Eremin-Solenikov , Greg Kroah-Hartman , Mark Brown , Jean Delvare , linux-kernel@vger.kernel.org, Rob Herring , lm-sensors@lm-sensors.org, arm@kernel.org, Guenter Roeck , Kumar Gala , Grant Likely , Lee Jones , David Woodhouse , linux-arm-kernel@lists.infradead.org Date: Mon, 28 Apr 2014 18:40:54 +0100 In-Reply-To: <52FA5EB4.7080008@codeaurora.org> References: <1392137610-27842-1-git-send-email-pawel.moll@arm.com> <1392137610-27842-2-git-send-email-pawel.moll@arm.com> <52FA5EB4.7080008@codeaurora.org> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.10.4-0ubuntu1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 2014-02-11 at 12:32 -0500, Christopher Covington wrote: > Hi Pawel, > > On 02/11/2014 11:53 AM, Pawel Moll wrote: > > This patch tries to solve that issue in a generic way, > > adding a "populated" flag which is set in the device_node > > structure when a device is being created in the core. > > Later, of_platform_populate() skips such nodes (and > > its children) in a similar way to the non-available ones. > > Will there never be a case where it is useful for a parent node to be created > early, but not necessarily the child nodes? Might only skipping nodes > explicitly marked as populated be a more universal solution? That's a good point. I've assumed one would either use of_platform_device_create() on leaf nodes only (or of_platform_populate() on subtree), and kept this approach in the refreshed patch I'm about to post, but I'm happy to do what you suggested. Thanks! Pawel -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/