From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Bill Gatliff <bgat@billgatliff.com>
Cc: linux-embedded@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: Expose regulator:set_consumer_device_supply()?
Date: Tue, 26 Apr 2011 17:15:16 +0100 [thread overview]
Message-ID: <20110426161516.GC11848@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <BANLkTimmpx5ZThXj705Yn_wRQ6bF7VzwPw@mail.gmail.com>
On Tue, Apr 26, 2011 at 10:33:29AM -0500, Bill Gatliff wrote:
> On Tue, Apr 26, 2011 at 3:33 AM, Mark Brown
> <broonie@opensource.wolfsonmicro.com> wrote:
> > You can specify the device by either dev_name() or a dev pointer. You
> > can use dev_name() at any time without the device having been
> > instantiated, it would be unusal to use a struct device.
> When a consumer e.g. i2c chip driver is trying to get a handle for its
> own regulator, the only function I see is:
>
> struct regulator* regulator_get(struct device *dev, const char *id)
There's also regulator_get_exclusive() but it's almost exactly the same
thing.
> In order for there to be a regulator with a matching device:id
> combination, someone must have previously provided a struct
> regulator_consumer_supply with the identical device pointer to
> regulator_register(). That means that I have to call
> regulator_register() AFTER I register the i2c chip driver, so that I
> have a struct device pointer to place in the regulator_consumer_supply
> list. Right?
No. As I said in the text you've quoted above you can also specify the
device mapping using the dev_name() of the device. As you will have
seen when looking through the regulator_get() implementation the
matching is actually done on the dev_name(), if the mapping is set up
with a struct device we always do matches based on the dev_name()
string, not by comparing pointers.
> > This would facilitate abuse of the API, we already have enough problems
> > with people trying to pass regulators as platform data.
> But I think you'll agree that regulators are pretty important platform data, no?
No, the set of power supplies the device has is not platform data, it's
a physical property of the device.
> What specifically is the breakage that comes from allowing consumers
> to add themselves to regulator consumer lists at a time after
> regulator_register() is complete? Why is passing a regulator pointer
> as platform data such a problem?
It means you get reams of code in drivers conditionally using the
regulator API, all of which adds needless complexity all over the tree
as people invariably make everything conditional on the regulator not
being there when they shouldn't. This then means you also end up with
no meaningful error handling, all errors just get silently eaten.
next prev parent reply other threads:[~2011-04-26 16:15 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-26 2:16 Expose regulator:set_consumer_device_supply()? Bill Gatliff
2011-04-26 2:25 ` Bill Gatliff
2011-04-26 8:34 ` Mark Brown
2011-04-26 8:33 ` Mark Brown
2011-04-26 15:33 ` Bill Gatliff
2011-04-26 16:15 ` Mark Brown [this message]
2011-04-26 21:15 ` Bill Gatliff
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=20110426161516.GC11848@opensource.wolfsonmicro.com \
--to=broonie@opensource.wolfsonmicro.com \
--cc=bgat@billgatliff.com \
--cc=linux-embedded@vger.kernel.org \
--cc=linux-kernel@vger.kernel.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 read-only IMAP folder(s) and NNTP newsgroup(s).