public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Ulf Hansson <ulf.hansson@stericsson.com>
Cc: Liam Girdwood <lrg@slimlogic.co.uk>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Mattias WALLIN <mattias.wallin@stericsson.com>,
	Jonas ABERG <jonas.aberg@stericsson.com>,
	Lee Jones <lee.jones@linaro.org>,
	Jassi Brar <jassisinghbrar@gmail.com>
Subject: Re: [PATCH] regulator: core: Keep boot_on regulators powered during init
Date: Wed, 25 Apr 2012 10:58:19 +0100	[thread overview]
Message-ID: <20120425095819.GG3195@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <4F97C5E0.1050808@stericsson.com>

[-- Attachment #1: Type: text/plain, Size: 1511 bytes --]

On Wed, Apr 25, 2012 at 11:37:36AM +0200, Ulf Hansson wrote:

> Maybe you have convinced me now :-) I will therefore start thinking
> of a patch on the mmc framework instead. I will include you if/when
> I send out the patch to the mmc-list, just for reference if that is
> ok with you?

Yes, please - there've been some issues with the way regulators are used
in MMC for a while.  There's definitely some stuff that needs to be
worked through here.

I've also added Jassi who it just occurred to me was talking about some
vaugley similar stuff relatively recently (though I'm not sure it was
MMC).  Jassi, the issue here is working out if an MMC device is powered
at boot so we can skip probing then shutting it down cleanly.

> Some final thoughts (please comment if you like):
> We already have the boot_on constraint, which to me is similar to
> what a new kind of "boot keep state" constraint would be. I think it
> would be no more odd than what boot_on already is. Maybe not a good
> argument, but still..

Yeah, the main use case for boot_on is to support things like automatic
enumeration of devices - for buses where we can enumerate devices when
they're plugged in we normally don't explicitly register them so if we
boot with the regulator disabled we need some way of allowing the device
to appear on the bus.  Don't know that that actually gets much use
though.  It is also used by regulators which for some reason can't
figure out their initial state, there's a few examples of that in
mainline.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

  reply	other threads:[~2012-04-25  9:58 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-23  9:37 [PATCH] regulator: core: Keep boot_on regulators powered during init Ulf Hansson
2012-04-23 10:18 ` Mark Brown
2012-04-23 10:52   ` Ulf Hansson
2012-04-23 11:05     ` Mark Brown
2012-04-23 12:21       ` Ulf Hansson
2012-04-23 12:25         ` Mark Brown
2012-04-23 12:45           ` Ulf Hansson
2012-04-23 18:01             ` Mark Brown
2012-04-24  8:09               ` Ulf Hansson
2012-04-24 10:56                 ` Mark Brown
2012-04-24 12:43                   ` Ulf Hansson
2012-04-25  8:02                     ` Mark Brown
2012-04-25  9:37                       ` Ulf Hansson
2012-04-25  9:58                         ` Mark Brown [this message]
2012-04-25 16:45                           ` Jassi Brar
2012-04-25 15:34               ` Jassi Brar
2012-04-25 15:44                 ` Ulf Hansson
2012-04-25 16:31                   ` Jassi Brar
2012-04-26  8:35                 ` Mark Brown
2012-04-26  9:10                   ` Jassi Brar

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=20120425095819.GG3195@opensource.wolfsonmicro.com \
    --to=broonie@opensource.wolfsonmicro.com \
    --cc=jassisinghbrar@gmail.com \
    --cc=jonas.aberg@stericsson.com \
    --cc=lee.jones@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lrg@slimlogic.co.uk \
    --cc=mattias.wallin@stericsson.com \
    --cc=ulf.hansson@stericsson.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