From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Marek Vasut <marek.vasut@gmail.com>
Cc: Liam Girdwood <lrg@slimlogic.co.uk>,
linux-kernel@vger.kernel.org,
Igor Grinberg <grinberg@compulab.co.il>,
Dmitry Torokhov <dmitry.torokhov@gmail.com>,
Alan Cox <alan@lxorguk.ukuu.org.uk>,
Linus Walleij <linus.ml.walleij@gmail.com>
Subject: Re: [PATCH] regulator: Add option for machine drivers to enable the dummy regulator
Date: Wed, 6 Oct 2010 06:08:25 +0100 [thread overview]
Message-ID: <20101006050825.GA17998@sirena.org.uk> (raw)
In-Reply-To: <201010060609.17412.marek.vasut@gmail.com>
On Wed, Oct 06, 2010 at 06:09:17AM +0200, Marek Vasut wrote:
> This still solves nothing ... This leaves boards broken and the granularity is
We've already got a compatibility option which allows this to be enabled
with a config change, and the actual code update required has been
reduced to a single line. This does not seem terribly onerous.
> too big. Consider there are boards that use regulator for MMC, but not for
> ADS7846 for example.
You can't have it both ways. Either drivers just ignore missing
regulators and just run as though everything is fine (which is what
you've been requesting enthusiastically) or they trust that they can get
accurate information about what is and isn't mapped. All this is doing
is providing a way to enable the former behaviour in the core.
next prev parent reply other threads:[~2010-10-06 5:08 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-06 2:18 [PATCH] regulator: Add option for machine drivers to enable the dummy regulator Mark Brown
2010-10-06 4:09 ` Marek Vasut
2010-10-06 5:08 ` Mark Brown [this message]
2010-10-10 10:14 ` Liam Girdwood
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=20101006050825.GA17998@sirena.org.uk \
--to=broonie@opensource.wolfsonmicro.com \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=dmitry.torokhov@gmail.com \
--cc=grinberg@compulab.co.il \
--cc=linus.ml.walleij@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=lrg@slimlogic.co.uk \
--cc=marek.vasut@gmail.com \
/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