From: swarren@wwwdotorg.org (Stephen Warren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v8] reset: Add driver for gpio-controlled reset pins
Date: Wed, 17 Jul 2013 10:57:08 -0600 [thread overview]
Message-ID: <51E6CCE4.20705@wwwdotorg.org> (raw)
In-Reply-To: <20130717030246.GA4541@S2101-09.ap.freescale.net>
On 07/16/2013 09:02 PM, Shawn Guo wrote:
> On Tue, Jul 16, 2013 at 09:45:43AM -0600, Stephen Warren wrote:
>> Registering the driver earlier won't cause any bugs. However, it's not
>> the correct approach.
>>
>> Deferred probe /is/ the approach for assuring correct dependencies
>> between drivers. It works and should be used. There are not enough
>> initcall levels to play games using initcalls and solve all the issues,
>> and the ordering requirements vary board-to-board. Deferred probe at
>> runtime handles this without having to manually place all the drivers
>> into specific initcall levels, and dynamically adjusts to board
>> differences, since it all happens automatically at run-time.
>
> I do not quite follow the argument here. I agree with you that
> deferred probe is the approach to solve dependencies. But it does not
> necessarily mean that initcall can not be used to help it save some
> nasty or nested deferring. Deferred probe and initcalls are not two
> mutually exclusive mechanisms but two which can help each other.
My understanding is that deferred probe was implemented specifically to
avoid having to, or allowing, the use of initcall levels to determine
probe order.
However, if someone closely associated with the implementation of
deferred probe (e.g. Grant, or a device core maintainer) is willing to
step up and say I'm wrong, I'll drop my objection.
WARNING: multiple messages have this Message-ID (diff)
From: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
To: Shawn Guo <shawn.guo-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: Marek Vasut <marex-ynQEQJNshbs@public.gmane.org>,
Fabio Estevam
<fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org>,
Mike Turquette
<mturquette-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
Pavel Machek <pavel-+ZI9xUNit7I@public.gmane.org>,
Len Brown <len.brown-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
Sascha Hauer <s.hauer-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>,
"Rafael J. Wysocki" <rjw-KKrjLPT3xs0@public.gmane.org>,
Philipp Zabel <p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>,
devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH v8] reset: Add driver for gpio-controlled reset pins
Date: Wed, 17 Jul 2013 10:57:08 -0600 [thread overview]
Message-ID: <51E6CCE4.20705@wwwdotorg.org> (raw)
In-Reply-To: <20130717030246.GA4541-rvtDTF3kK1ictlrPMvKcciBecyulp+rMXqFh9Ls21Oc@public.gmane.org>
On 07/16/2013 09:02 PM, Shawn Guo wrote:
> On Tue, Jul 16, 2013 at 09:45:43AM -0600, Stephen Warren wrote:
>> Registering the driver earlier won't cause any bugs. However, it's not
>> the correct approach.
>>
>> Deferred probe /is/ the approach for assuring correct dependencies
>> between drivers. It works and should be used. There are not enough
>> initcall levels to play games using initcalls and solve all the issues,
>> and the ordering requirements vary board-to-board. Deferred probe at
>> runtime handles this without having to manually place all the drivers
>> into specific initcall levels, and dynamically adjusts to board
>> differences, since it all happens automatically at run-time.
>
> I do not quite follow the argument here. I agree with you that
> deferred probe is the approach to solve dependencies. But it does not
> necessarily mean that initcall can not be used to help it save some
> nasty or nested deferring. Deferred probe and initcalls are not two
> mutually exclusive mechanisms but two which can help each other.
My understanding is that deferred probe was implemented specifically to
avoid having to, or allowing, the use of initcall levels to determine
probe order.
However, if someone closely associated with the implementation of
deferred probe (e.g. Grant, or a device core maintainer) is willing to
step up and say I'm wrong, I'll drop my objection.
next prev parent reply other threads:[~2013-07-17 16:57 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-30 9:09 [PATCH v8] reset: Add driver for gpio-controlled reset pins Philipp Zabel
2013-05-30 9:09 ` Philipp Zabel
2013-05-30 10:37 ` Pavel Machek
2013-05-30 10:37 ` Pavel Machek
2013-07-16 1:50 ` Shawn Guo
2013-07-16 1:50 ` Shawn Guo
2013-07-16 3:35 ` Stephen Warren
2013-07-16 3:35 ` Stephen Warren
2013-07-16 4:10 ` Shawn Guo
2013-07-16 4:10 ` Shawn Guo
2013-07-16 9:49 ` Philipp Zabel
2013-07-16 9:49 ` Philipp Zabel
2013-07-16 12:56 ` Shawn Guo
2013-07-16 12:56 ` Shawn Guo
2013-07-16 15:45 ` Stephen Warren
2013-07-16 15:45 ` Stephen Warren
2013-07-17 3:02 ` Shawn Guo
2013-07-17 3:02 ` Shawn Guo
2013-07-17 16:57 ` Stephen Warren [this message]
2013-07-17 16:57 ` Stephen Warren
2013-07-17 21:38 ` Pavel Machek
2013-07-17 21:38 ` Pavel Machek
2013-07-17 22:24 ` Stephen Warren
2013-07-17 22:24 ` Stephen Warren
2013-07-18 11:25 ` Pavel Machek
2013-07-18 11:25 ` Pavel Machek
2013-07-18 18:45 ` Olof Johansson
2013-07-18 18:45 ` Olof Johansson
2013-07-19 3:23 ` Shawn Guo
2013-07-19 3:23 ` Shawn Guo
2013-07-19 15:45 ` Stephen Warren
2013-07-19 15:45 ` Stephen Warren
2013-07-22 7:47 ` Shawn Guo
2013-07-22 7:47 ` Shawn Guo
2013-07-26 10:26 ` Philipp Zabel
2013-07-26 10:26 ` Philipp Zabel
2013-07-18 22:55 ` Grant Likely
2013-07-18 22:55 ` Grant Likely
2013-07-18 22:50 ` Grant Likely
2013-07-18 22:50 ` Grant Likely
2013-07-16 9:59 ` Philipp Zabel
2013-07-16 9:59 ` Philipp Zabel
2013-07-16 15:48 ` Stephen Warren
2013-07-16 15:48 ` Stephen Warren
2013-07-16 6:51 ` Shawn Guo
2013-07-16 6:51 ` Shawn Guo
2013-07-16 9:51 ` Philipp Zabel
2013-07-16 9:51 ` Philipp Zabel
2013-07-16 12:15 ` Shawn Guo
2013-07-16 12:15 ` Shawn Guo
2013-07-16 15:47 ` Stephen Warren
2013-07-16 15:47 ` Stephen Warren
2013-07-17 3:43 ` Shawn Guo
2013-07-17 3:43 ` Shawn Guo
2013-07-17 7:17 ` Philipp Zabel
2013-07-17 7:17 ` Philipp Zabel
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=51E6CCE4.20705@wwwdotorg.org \
--to=swarren@wwwdotorg.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 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.