From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Matt Sealey <matt@genesi-usa.com>
Cc: Linux ARM Kernel Mailing List
<linux-arm-kernel@lists.infradead.org>,
Steev Klimaszewski <steev@genesi-usa.com>,
Shawn Guo <shawn.guo@linaro.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] efikamx: reintroduce Genesi Efika MX Smarttop via device tree
Date: Thu, 9 Aug 2012 15:17:28 +0100 [thread overview]
Message-ID: <20120809141727.GW24328@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <CAKGA1bkT2GDXXXAgjG4ySfTxwO=4O7o30BLzLHCMZKDrR318xQ@mail.gmail.com>
On Thu, Aug 09, 2012 at 08:40:36AM -0500, Matt Sealey wrote:
> The reason they're set like that is legacy - that's how they're set up
> in a kernel
> (pre-DT) that we know works. Most of those ranges are directly from the Babbage
> reference and stay like that in the Babbage DT too - so there's another broken
> one nobody noticed. I know what those voltages should be, but we're
> leaving that for another patch that restricts the range of voltages
> (it works right
> now, since there are no consumers, nothing CHANGES the voltages as
> configured at U-Boot time, and anything not boot-on is just not listed
> in the DT anyway, but some of them really need to stay on)
Oh dear. Well, no reason to propagate the breakage - if nothing else it
might well explode if we start doing more aggressive power saving with
the regulators (like having an option for dropping down to the lower end
of the voltage ranges in late init which I keep contemplating, it'd
explode with the boards doing this).
> There are few consumers because the primary ones out there are the display
> controllers and USB hubs and some other things. MMC should be a consumer
> but since on one board we share two MMC slots with one regulator we don't
> want anyone to change the voltage (it breaks spec anyway, since we can't
> provide more than 3.15V with FSL's PMIC and it should be 3.3V by default)
> and since you can't coordinate between MMC hosts on what the lowest voltage
> both cards can support actually is.. having someone change it would be
> bad.
It's not a problem to have fixed voltages, the problem is the
combination of specifying voltage ranges in conjunction with not having
anything there that wants to change the voltage. Especially with things
like the audio supply, it's clear what that's for and it'd normally get
upset with the voltage changing.
next prev parent reply other threads:[~2012-08-09 14:17 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-07 21:46 [PATCH] efikamx: reintroduce Genesi Efika MX Smarttop via device tree Matt Sealey
2012-08-08 15:15 ` Shawn Guo
2012-08-08 16:55 ` Matt Sealey
2012-08-08 17:19 ` Fabio Estevam
2012-08-09 14:29 ` Matt Sealey
2012-08-10 1:41 ` Shawn Guo
2012-08-10 13:36 ` Matt Sealey
2012-08-10 14:04 ` Shawn Guo
2012-08-10 14:26 ` Matt Sealey
2012-08-10 14:40 ` Shawn Guo
2012-08-10 14:42 ` Matt Sealey
2012-08-09 10:19 ` Mark Brown
2012-08-09 13:40 ` Matt Sealey
2012-08-09 14:17 ` Mark Brown [this message]
2012-08-13 15:42 ` Matt Sealey
2012-08-13 17:38 ` Mark Brown
2012-08-13 22:05 ` Matt Sealey
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=20120809141727.GW24328@opensource.wolfsonmicro.com \
--to=broonie@opensource.wolfsonmicro.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=matt@genesi-usa.com \
--cc=shawn.guo@linaro.org \
--cc=steev@genesi-usa.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;
as well as URLs for NNTP newsgroup(s).