From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756754AbbGPXlU (ORCPT ); Thu, 16 Jul 2015 19:41:20 -0400 Received: from mail-wi0-f174.google.com ([209.85.212.174]:33166 "EHLO mail-wi0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756141AbbGPXlS (ORCPT ); Thu, 16 Jul 2015 19:41:18 -0400 MIME-Version: 1.0 In-Reply-To: <20150716202312.GD11162@sirena.org.uk> References: <1435743667-11987-1-git-send-email-tomeu.vizoso@collabora.com> <1435743667-11987-2-git-send-email-tomeu.vizoso@collabora.com> <20150716202312.GD11162@sirena.org.uk> Date: Fri, 17 Jul 2015 01:41:16 +0200 X-Google-Sender-Auth: Xm9dZ_m-TTCBIz14eJ8YlxTPjQs Message-ID: Subject: Re: [PATCH v2 01/12] device: property: delay device-driver matches From: "Rafael J. Wysocki" To: Mark Brown Cc: Tomeu Vizoso , Linux Kernel Mailing List , ACPI Devel Maling List , dri-devel@lists.freedesktop.org, linux-fbdev@vger.kernel.org, linux-gpio@vger.kernel.org, "devicetree@vger.kernel.org" , linux-pwm@vger.kernel.org, "Rafael J. Wysocki" , alsa-devel@alsa-project.org, Greg Kroah-Hartman Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Mark, On Thu, Jul 16, 2015 at 10:23 PM, Mark Brown wrote: > On Wed, Jul 01, 2015 at 11:40:56AM +0200, Tomeu Vizoso wrote: > >> Delay matches of platform devices until late_initcall, when we are sure >> that all built-in drivers have been registered already. This is needed >> to prevent deferred probes because of some dependencies' drivers not >> having registered yet. > > I have to say I'm still not 100% clear that special casing platform > devices makes sense here - I can see that platform devices are usually > the first devices to instantiate but there are other kinds of devices > and it's not obvious what the benefit of specifically picking out > platform devices as opposed to just deferring all devices is. Some existing devices cannot be deferred without redesigning things quite a bit. What I was talking about, though, was to use an opt-in mechanism for that which could be set for all platform devices, for example, by default, but it might be set for other bus types too if that's useful. Thanks, Rafael From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: [PATCH v2 01/12] device: property: delay device-driver matches Date: Fri, 17 Jul 2015 01:41:16 +0200 Message-ID: References: <1435743667-11987-1-git-send-email-tomeu.vizoso@collabora.com> <1435743667-11987-2-git-send-email-tomeu.vizoso@collabora.com> <20150716202312.GD11162@sirena.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <20150716202312.GD11162-GFdadSzt00ze9xe1eoZjHA@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Mark Brown Cc: Tomeu Vizoso , Linux Kernel Mailing List , ACPI Devel Maling List , dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, linux-fbdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-gpio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , linux-pwm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, "Rafael J. Wysocki" , alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org, Greg Kroah-Hartman List-Id: linux-acpi@vger.kernel.org Hi Mark, On Thu, Jul 16, 2015 at 10:23 PM, Mark Brown wrote: > On Wed, Jul 01, 2015 at 11:40:56AM +0200, Tomeu Vizoso wrote: > >> Delay matches of platform devices until late_initcall, when we are sure >> that all built-in drivers have been registered already. This is needed >> to prevent deferred probes because of some dependencies' drivers not >> having registered yet. > > I have to say I'm still not 100% clear that special casing platform > devices makes sense here - I can see that platform devices are usually > the first devices to instantiate but there are other kinds of devices > and it's not obvious what the benefit of specifically picking out > platform devices as opposed to just deferring all devices is. Some existing devices cannot be deferred without redesigning things quite a bit. What I was talking about, though, was to use an opt-in mechanism for that which could be set for all platform devices, for example, by default, but it might be set for other bus types too if that's useful. Thanks, Rafael -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Date: Thu, 16 Jul 2015 23:41:16 +0000 Subject: Re: [PATCH v2 01/12] device: property: delay device-driver matches Message-Id: List-Id: References: <1435743667-11987-1-git-send-email-tomeu.vizoso@collabora.com> <1435743667-11987-2-git-send-email-tomeu.vizoso@collabora.com> <20150716202312.GD11162@sirena.org.uk> In-Reply-To: <20150716202312.GD11162-GFdadSzt00ze9xe1eoZjHA@public.gmane.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Mark Brown Cc: Tomeu Vizoso , Linux Kernel Mailing List , ACPI Devel Maling List , dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, linux-fbdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-gpio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , linux-pwm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, "Rafael J. Wysocki" , alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org, Greg Kroah-Hartman Hi Mark, On Thu, Jul 16, 2015 at 10:23 PM, Mark Brown wrote: > On Wed, Jul 01, 2015 at 11:40:56AM +0200, Tomeu Vizoso wrote: > >> Delay matches of platform devices until late_initcall, when we are sure >> that all built-in drivers have been registered already. This is needed >> to prevent deferred probes because of some dependencies' drivers not >> having registered yet. > > I have to say I'm still not 100% clear that special casing platform > devices makes sense here - I can see that platform devices are usually > the first devices to instantiate but there are other kinds of devices > and it's not obvious what the benefit of specifically picking out > platform devices as opposed to just deferring all devices is. Some existing devices cannot be deferred without redesigning things quite a bit. What I was talking about, though, was to use an opt-in mechanism for that which could be set for all platform devices, for example, by default, but it might be set for other bus types too if that's useful. Thanks, Rafael