|
|
Log in / Subscribe / Register

On-demand device registration

From:  Tomeu Vizoso <tomeu.vizoso@collabora.com>
To:  linux-kernel@vger.kernel.org
Subject:  [RFC 00/12] On-demand device registration
Date:  Fri, 24 Apr 2015 16:47:16 +0200
Message-ID:  <1429886848-5843-1-git-send-email-tomeu.vizoso@collabora.com>
Cc:  Thierry Reding <thierry.reding@gmail.com>, =?UTF-8?q?St=C3=A9phane=20Marchesin?= <marcheu@chromium.org>, Alexander Holler <holler@ahsoftware.de>, Olof Johansson <olof@lixom.net>, Grant Likely <grant.likely@linaro.org>, Rob Herring <robh+dt@kernel.org>, devicetree@vger.kernel.org, Mark Rutland <mark.rutland@arm.com>, Tomeu Vizoso <tomeu.vizoso@collabora.com>
Archive‑link:  Article

Hi,

while reading the thread [0] that Alexander Holler started with his series
to make probing order deterministic, it occurred to me that it should be
possible to achieve the same by probing devices as they are referenced by
other devices. 

This basically reuses the information that is already embedded in the
probe() implementations, saving us from refactoring existing drivers or
adding information to DTBs. 

The main issue I see is that the registration code path in some subsystems
may not be reentrant, so some refactoring of the locking will be needed. In
my testing I have found this problem with regulators, as the supply of a
regulator might end up being registered during the registration of the
first one. 

Something I'm not completely happy with is that I have had to move the
population of the device tree after all platform drivers have been
registered. Otherwise I don't see how I could register drivers on demand as
we don't have yet each driver's compatible strings. 

I have done my testing on a Tegra124-based Chromebook, and these patches
were enough to eliminate all the deferred probes. 

Regards,

Tomeu

[0] https://lkml.org/lkml/2014/5/12/452

Tomeu Vizoso (12):
  ARM: tegra: Register drivers before devices
  ARM: tegra: Add gpio-ranges property
  of/platform: add of_platform_device_ensure()
  gpio: Probe GPIO drivers on demand
  gpio: Probe pinctrl devices on demand
  regulator: core: Probe regulators on demand
  drm: Probe panels on demand
  drm/tegra: Probe dpaux devices on demand
  i2c: core: Probe i2c master devices on demand
  pwm: Probe PWM chip devices on demand
  backlight: Probe backlight devices on demand
  usb: phy: Probe phy devices on demand

 arch/arm/boot/dts/tegra124.dtsi     |  1 +
 arch/arm/mach-tegra/tegra.c         | 21 ++++++++-------------
 drivers/gpio/gpiolib-of.c           |  5 +++++
 drivers/gpu/drm/drm_panel.c         |  3 +++
 drivers/gpu/drm/tegra/dpaux.c       |  3 +++
 drivers/i2c/i2c-core.c              |  3 +++
 drivers/of/platform.c               | 28 ++++++++++++++++++++++++++++
 drivers/pwm/core.c                  |  3 +++
 drivers/regulator/core.c            |  2 ++
 drivers/usb/phy/phy.c               |  3 +++
 drivers/video/backlight/backlight.c |  3 +++
 include/linux/of_platform.h         |  2 ++
 12 files changed, 64 insertions(+), 13 deletions(-)

-- 
2.3.6

--
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/


Copyright © 2015, Eklektix, Inc.
Comments and public postings are copyrighted by their creators.
Linux is a registered trademark of Linus Torvalds