Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752670Ab2KPQKE (ORCPT ); Fri, 16 Nov 2012 11:10:04 -0500 Received: from avon.wwwdotorg.org ([70.85.31.133]:57946 "EHLO avon.wwwdotorg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751665Ab2KPQKC (ORCPT ); Fri, 16 Nov 2012 11:10:02 -0500 Message-ID: <50A66553.3050705@wwwdotorg.org> Date: Fri, 16 Nov 2012 09:09:55 -0700 From: Stephen Warren User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121028 Thunderbird/16.0.2 MIME-Version: 1.0 To: Linus Walleij CC: Mark Brown , Linus Walleij , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Stephen Warren , Anmar Oueja , Felipe Balbi , Benoit Cousson , Dmitry Torokhov , Thomas Petazzoni , Mitch Bradley , Ulf Hansson , "Rafael J. Wysocki" , Kevin Hilman , Jean-Christophe PLAGNIOL-VILLARD , Rickard Andersson , Greg Kroah-Hartman , Russell King Subject: Re: [PATCH] RFC: pinctrl: grab default handler with bus notifiers References: <1352636539-6318-1-git-send-email-linus.walleij@stericsson.com> <50A15A54.3090803@wwwdotorg.org> <20121113063546.GD18224@opensource.wolfsonmicro.com> <50A53304.6040904@wwwdotorg.org> In-Reply-To: X-Enigmail-Version: 1.4.5 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2002 Lines: 45 On 11/16/2012 04:36 AM, Linus Walleij wrote: > On Thu, Nov 15, 2012 at 7:23 PM, Stephen Warren wrote: >> On 11/15/2012 07:03 AM, Linus Walleij wrote: > >>> OK I'll have to come up with a patch to the device core >>> instead... it'll be much simpler anyway and if both of you guys >>> can back it I guess Greg might be OK with it too. >> >> I did have one thought here; how will this interact with hogs? If a >> device's pinctrl configuration must be pinctrl_get()'d before the device >> is probed, then a pinctrl device with hogs will never get probed because >> it won't be registered to provide the pinctrl node parsing. > > Catch 22 :-( > > Yeah we need to come up with something there. > >> Solutions might include: >> >> a) Some special case where if the pinctrl driver only can't probe due to >> missing pinctrl from its own node, don't defer the probe, but defer the >> pinctrl_get(). >> >> b) Separate out DT node parsing from device instantiation, so that the >> driver can always parse the DT, without needing the context of a >> specific pinctrl device to do so. > > But this mechanism can't be device tree-specific, we have some > of olschool pdata users and ACPI probing around the corner. > > I will likely just cook up something like seeing if the > dev_name() for provider and consumer is the same and > in that case avoid deferral. Well, the DT parsing is all about creating the mapping table from the DT. Without DT, this doesn't need to happen, since the board file already contains the complete mapping table. So, this isn't so much a special case for DT, but rather simply a step we don't need to take for DT, if you get what I mean (and that's all true irrespective of the change we're discussing here). -- 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/