From: robh+dt@kernel.org (Rob Herring)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v6 22/22] of/platform: Defer probes of registered devices
Date: Wed, 21 Oct 2015 08:44:44 -0500 [thread overview]
Message-ID: <CAL_JsqLhZmc8cEC+tFJ25KLe0bQZsWmrRWGBMQake9VN6OjvdQ@mail.gmail.com> (raw)
In-Reply-To: <1445406845.701.55.camel@freescale.com>
On Wed, Oct 21, 2015 at 12:54 AM, Scott Wood <scottwood@freescale.com> wrote:
> On Mon, 2015-09-21 at 16:03 +0200, Tomeu Vizoso wrote:
>> Instead of trying to match and probe platform and AMBA devices right
>> after each is registered, delay their probes until device_initcall_sync.
>>
>> This means that devices will start probing once all built-in drivers
>> have registered, and after all platform and AMBA devices from the DT
>> have been registered already.
>>
>> This allows us to prevent deferred probes by probing dependencies on
>> demand.
>>
>> Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
>> ---
>>
>> Changes in v4:
>> - Also defer probes of AMBA devices registered from the DT as they can
>> also request resources.
>>
>> drivers/of/platform.c | 11 ++++++++---
>> 1 file changed, 8 insertions(+), 3 deletions(-)
>
> This breaks arch/powerpc/sysdev/fsl_pci.c. The PCI bus is an OF platform
> device, and it must be probed before pcibios_init() which is a
> subsys_initcall(), or else the PCI bus never gets scanned.
Thanks for the report. This is probably getting dropped, but it could
be disabled for PPC.
Any plans to fix this and make PCI hosts hotplugable? For the scanning
part, generally the host controller drivers are responsible for
scanning their bus now.
Rob
next prev parent reply other threads:[~2015-10-21 13:44 UTC|newest]
Thread overview: 70+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-21 14:02 [PATCH v6 0/22] On-demand device probing Tomeu Vizoso
2015-09-21 14:02 ` [PATCH v6 01/22] driver core: handle -EPROBE_DEFER from bus_type.match() Tomeu Vizoso
2015-10-22 1:00 ` Rafael J. Wysocki
2015-09-21 14:02 ` [PATCH v6 02/22] ARM: amba: Move reading of periphid to amba_match() Tomeu Vizoso
2015-09-21 14:02 ` [PATCH v6 03/22] of/platform: Point to struct device from device node Tomeu Vizoso
2015-09-22 0:39 ` Rob Herring
2015-09-22 6:45 ` Tomeu Vizoso
2015-10-22 1:02 ` Rafael J. Wysocki
2015-10-22 13:01 ` Tomeu Vizoso
2015-10-24 13:57 ` Rafael J. Wysocki
2015-10-27 14:48 ` Tomeu Vizoso
2015-10-27 15:43 ` Rafael J. Wysocki
2015-10-27 21:24 ` Rob Herring
2015-09-21 14:02 ` [PATCH v6 04/22] of: add function to allow probing a device from a OF node Tomeu Vizoso
2015-10-22 1:06 ` Rafael J. Wysocki
2015-10-22 13:03 ` Tomeu Vizoso
2015-10-22 23:54 ` Mark Brown
2015-10-24 14:28 ` Rafael J. Wysocki
2015-10-24 13:55 ` Rafael J. Wysocki
2015-10-24 20:09 ` Rob Herring
2015-10-26 0:13 ` Mark Brown
2015-10-26 0:15 ` Dmitry Torokhov
2015-10-26 2:48 ` Rafael J. Wysocki
2015-10-26 3:09 ` Mark Brown
2015-10-26 8:16 ` Geert Uytterhoeven
2015-10-27 14:46 ` Tomeu Vizoso
2015-09-21 14:02 ` [PATCH v6 05/22] gpio: Probe GPIO drivers on demand Tomeu Vizoso
2015-09-21 14:02 ` [PATCH v6 06/22] gpio: Probe pinctrl devices " Tomeu Vizoso
2015-09-21 14:02 ` [PATCH v6 07/22] regulator: core: Remove regulator_list Tomeu Vizoso
2015-09-21 19:38 ` Mark Brown
2015-09-22 7:21 ` Tomeu Vizoso
2015-09-21 14:02 ` [PATCH v6 08/22] regulator: core: Probe regulators on demand Tomeu Vizoso
2015-09-21 19:39 ` Mark Brown
2015-09-21 14:02 ` [PATCH v6 09/22] drm: Probe panels " Tomeu Vizoso
2015-09-21 14:02 ` [PATCH v6 10/22] drm/tegra: Probe dpaux devices " Tomeu Vizoso
2015-09-21 14:02 ` [PATCH v6 11/22] i2c: core: Probe i2c adapters and " Tomeu Vizoso
2015-09-21 14:02 ` [PATCH v6 12/22] pwm: Probe PWM chip " Tomeu Vizoso
2015-09-21 14:02 ` [PATCH v6 13/22] backlight: Probe backlight " Tomeu Vizoso
2015-09-21 14:02 ` [PATCH v6 14/22] usb: phy: Probe phy " Tomeu Vizoso
2015-09-21 14:02 ` [PATCH v6 15/22] clk: Probe clk providers " Tomeu Vizoso
2015-09-21 14:02 ` [PATCH v6 16/22] pinctrl: Probe pinctrl devices " Tomeu Vizoso
2015-09-21 14:02 ` [PATCH v6 17/22] phy: core: Probe phy providers " Tomeu Vizoso
2015-09-21 14:02 ` [PATCH v6 18/22] dma: of: Probe DMA controllers " Tomeu Vizoso
2015-09-21 14:02 ` [PATCH v6 19/22] power-supply: Probe power supplies " Tomeu Vizoso
2015-09-21 14:03 ` [PATCH v6 20/22] driver core: Allow deferring probes until late init Tomeu Vizoso
2015-09-26 18:15 ` Rob Herring
2015-09-29 8:05 ` Tomeu Vizoso
2015-09-29 16:58 ` Rob Herring
2015-09-21 14:03 ` [PATCH v6 21/22] driver core: Start processing deferred probes earlier Tomeu Vizoso
2015-10-05 23:52 ` Frank Rowand
2015-10-06 2:49 ` Rob Herring
2015-10-06 10:45 ` Mark Brown
2015-09-21 14:03 ` [PATCH v6 22/22] of/platform: Defer probes of registered devices Tomeu Vizoso
2015-10-21 5:54 ` Scott Wood
2015-10-21 13:44 ` Rob Herring [this message]
2015-10-21 22:51 ` Scott Wood
2015-10-22 13:04 ` Tomeu Vizoso
2015-10-22 21:27 ` Scott Wood
2015-10-24 13:51 ` Rafael J. Wysocki
2015-10-28 14:40 ` Tomeu Vizoso
2015-10-29 4:17 ` Rob Herring
2015-10-29 16:06 ` Scott Wood
2015-10-22 0:34 ` Michael Ellerman
2015-09-26 18:17 ` [PATCH v6 0/22] On-demand device probing Rob Herring
2015-09-26 19:22 ` Greg Kroah-Hartman
2015-09-30 10:09 ` Tomeu Vizoso
2015-09-30 10:23 ` Greg Kroah-Hartman
2015-09-29 8:08 ` Tomeu Vizoso
2015-10-13 19:57 ` Tomeu Vizoso
2015-10-13 21:21 ` Rob Herring
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAL_JsqLhZmc8cEC+tFJ25KLe0bQZsWmrRWGBMQake9VN6OjvdQ@mail.gmail.com \
--to=robh+dt@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).