From: Maxime Ripard <maxime.ripard@free-electrons.com>
To: Chen-Yu Tsai <wens@csie.org>
Cc: Quentin Schulz <quentin.schulz@free-electrons.com>,
Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>,
Mark Rutland <mark.rutland@arm.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Kumar Gala <galak@codeaurora.org>,
linux@armlinux.org.uk, devicetree <devicetree@vger.kernel.org>,
linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
linux-kernel <linux-kernel@vger.kernel.org>,
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
Hans De Goede <hdegoede@redhat.com>
Subject: Re: [PATCH] ARM: sun8i: Add Parrot Board DTS
Date: Mon, 20 Jun 2016 20:35:43 +0200 [thread overview]
Message-ID: <20160620183543.GM26668@lukather> (raw)
In-Reply-To: <CAGb2v66_iUMd5V=9Np4Kqqv+yqq2HbjoPhC3Z6cpHFrMzkddvw@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 2686 bytes --]
On Tue, Jun 21, 2016 at 12:30:25AM +0800, Chen-Yu Tsai wrote:
> >> >>> +®_aldo1 {
> >> >>> + regulator-always-on;
> >> >>> + regulator-min-microvolt = <3000000>;
> >> >>> + regulator-max-microvolt = <3000000>;
> >> >>> + regulator-name = "aldo1";
> >> >>
> >> >> What is this for exactly? Would turning it off render the system inoperable?
> >> >> How was it referenced in the fex file?
> >> >>
> >> >> If this is for WiFi I/O VCC, then you should specify it in mmc1 with
> >> >> vqmmc-supply.
> >> >>
> >> >
> >> > In the fex, aldo1 is one of the three power inputs for the WiFi (the
> >> > others being dldo1 and dldo2) and in the schematics it is linked to
> >> > both VCC-USB and VCC-IO-WIFI.
> >> >
> >> > I tried to turn it off and, indeed, the system becomes inoperable.
> >> >
> >> > I'll add vqmmc-supply in mmc1 with aldo1 regulator. However, I am
> >> > wondering what to put in vmmc-supply for mmc1 since the WiFi module has
> >> > three power inputs: dldo1, dldo2 and aldo1. In the fex, they are
> >> > referenced as, respectively, module_power1, module_power2 and
> >> > module_power3 and in the schematics dldo1 and dldo2 are named VCC-WIFI
> >> > while aldo1 is used for VCC-IO-WIFI (if it can help in any way).
> >> >
> >> > VCC-WIFI is connected to pin VBAT of the Broadcom AP6212 WiFi chip.
> >> > VCC-IO-WIFI is connected to pin VDDIO of the chip.
> >>
> >> VCC-IO-WIFI is vqmmc, and VCC-WIFI is vmmc.
> >
> > Isn't vmmc supposed to be always powered at 3.3v, and vqmmc the one
> > used to drop to 1.8V with the UHS cards?
>
> For eMMC, vmmc and vqmmc correspond to the pins on the eMMC chip.
> vmmc provides power to the internals, while vqmmc provides power to the
> I/O buffers on both sides. With SD this is a bit less clear. IIUC the SD
> card itself supplies I/O voltage, dropping it to 1.8V if necessary, from
> VDD (vmmc). vqmmc only affects the host side, and whatever external pull-ups
> that might exist.
Ok.
> > In which case, VCC-IO-WIFI is vmmc, and VCC-WIFI is just one of the
> > wifi chip power rail, to be handled by a power sequence.
>
> For the WiFi chip, there is a separate IO-VCC rail, which is VCC-IO-WIFI.
> This, from what I understand of the datasheet, is for I/O signaling, and
> thus vqmmc. VCC-WIFI is what powers the chip, which is what vmmc is.
Ok. but there's still the issue of the two regulators that needs to be
kept in sync. Before, that, I'd rather stice to not tying them to the
MMC bus, and putting a comment on top.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
next prev parent reply other threads:[~2016-06-20 18:35 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-13 10:15 [PATCH] ARM: sun8i: Add Parrot Board DTS Quentin Schulz
2016-06-13 10:15 ` Quentin Schulz
[not found] ` <1465812931-22824-2-git-send-email-quentin.schulz-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2016-06-13 12:43 ` kbuild test robot
2016-06-13 13:04 ` Chen-Yu Tsai
2016-06-14 12:59 ` Quentin Schulz
2016-06-14 13:19 ` Chen-Yu Tsai
2016-06-15 7:52 ` Hans de Goede
2016-06-20 15:44 ` Maxime Ripard
2016-06-20 16:30 ` Chen-Yu Tsai
2016-06-20 16:55 ` Hans de Goede
2016-06-20 18:35 ` Maxime Ripard [this message]
2016-06-21 9:11 ` Chen-Yu Tsai
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=20160620183543.GM26668@lukather \
--to=maxime.ripard@free-electrons.com \
--cc=devicetree@vger.kernel.org \
--cc=galak@codeaurora.org \
--cc=hdegoede@redhat.com \
--cc=ijc+devicetree@hellion.org.uk \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=mark.rutland@arm.com \
--cc=pawel.moll@arm.com \
--cc=quentin.schulz@free-electrons.com \
--cc=robh+dt@kernel.org \
--cc=thomas.petazzoni@free-electrons.com \
--cc=wens@csie.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 NNTP newsgroup(s).