From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: NeilBrown <neilb@suse.de>
Cc: "MyungJoo Ham" <myungjoo.ham@samsung.com>,
linux-kernel@vger.kernel.org,
"Randy Dunlap" <rdunlap@xenotime.net>,
"Mike Lockwood" <lockwood@android.com>,
"Arve Hjønnevåg" <arve@android.com>,
"Kyungmin Park" <kyungmin.park@samsung.com>,
"Donggeun Kim" <dg77.kim@samsung.com>, "Greg KH" <gregkh@suse.de>,
"Arnd Bergmann" <arnd@arndb.de>,
"MyungJoo Ham" <myungjoo.ham@gmail.com>,
"Linus Walleij" <linus.walleij@linaro.org>,
"Dmitry Torokhov" <dmitry.torokhov@gmail.com>,
"Morten CHRISTIANSEN" <morten.christiansen@stericsson.com>
Subject: Re: [PATCH v2 0/3] introduce External Connector Class (extcon)
Date: Thu, 15 Dec 2011 14:51:55 +0800 [thread overview]
Message-ID: <20111215065154.GA24248@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <20111215133229.20e7f0f2@notabene.brown>
On Thu, Dec 15, 2011 at 01:32:29PM +1100, NeilBrown wrote:
j> 3/ The use of extcon_get_extcon_dev() requires that the port device be
> registered before the device that wants to be notified by it. Ensuring
> correct ordering of device discovery is (I believe) not always easy -
> particularly with module loading.
This is a massive problem throughout the kernel
> Would it make sense to instead have one device register as a
> switch-provider and the other device register as a switch-listener and as
> soon as they both exist they get connected: a bit like how 'devices' and
> 'drivers' can be registered in any order.
> Maybe the same device/driver infrastructure could be reused (an extcon
> bus maybe?) but I'm not really familiar enough with it to say (Greg??).
Grant has a proposal for this which revolves around devices trying to
acquire their resources and returning a "please retry" error code if
they don't have all their dependencies. Half the problem here is that
coming up with the dependency graph (and finding ways to talk about
devices that haven't been enumerated yet) is a very hard problem.
> Are there other examples of inter-device dependencies that can be used as
> a model?
The current solution is to fiddle with initcall order so that drivers
for devices that other devices depend on enumerate first.
next prev parent reply other threads:[~2011-12-15 6:52 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-14 10:28 [PATCH v2 0/3] introduce External Connector Class (extcon) MyungJoo Ham
2011-12-14 10:28 ` [PATCH v2 1/3] Extcon (external connector): import Android's switch class and modify MyungJoo Ham
2011-12-15 1:01 ` Greg KH
2011-12-15 5:41 ` MyungJoo Ham
2011-12-15 7:18 ` Greg KH
2011-12-16 5:38 ` MyungJoo Ham
2011-12-16 18:18 ` Greg KH
2011-12-14 10:28 ` [PATCH v2 2/3] Extcon: support notification based on the state changes MyungJoo Ham
2011-12-14 10:28 ` [PATCH v2 3/3] Extcon: support multiple states at a device MyungJoo Ham
2011-12-15 2:32 ` [PATCH v2 0/3] introduce External Connector Class (extcon) NeilBrown
2011-12-15 6:36 ` MyungJoo Ham
2011-12-15 20:20 ` NeilBrown
2011-12-15 6:51 ` Mark Brown [this message]
2011-12-18 7:15 ` NeilBrown
2011-12-20 1:01 ` Mark Brown
2011-12-20 5:58 ` NeilBrown
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=20111215065154.GA24248@opensource.wolfsonmicro.com \
--to=broonie@opensource.wolfsonmicro.com \
--cc=arnd@arndb.de \
--cc=arve@android.com \
--cc=dg77.kim@samsung.com \
--cc=dmitry.torokhov@gmail.com \
--cc=gregkh@suse.de \
--cc=kyungmin.park@samsung.com \
--cc=linus.walleij@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lockwood@android.com \
--cc=morten.christiansen@stericsson.com \
--cc=myungjoo.ham@gmail.com \
--cc=myungjoo.ham@samsung.com \
--cc=neilb@suse.de \
--cc=rdunlap@xenotime.net \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.