From: balbi@ti.com (Felipe Balbi)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 2/2] arm: dts: add support for AM437x StarterKit
Date: Wed, 18 Jun 2014 18:23:27 -0500 [thread overview]
Message-ID: <20140618232327.GC9855@saruman.home> (raw)
In-Reply-To: <20140618231923.GA9855@saruman.home>
Hi,
(side note)
On Wed, Jun 18, 2014 at 06:19:23PM -0500, Felipe Balbi wrote:
> > >>>+ interrupt-controller;
> > >>>+ #interrupt-cells = <2>;
> > >>>+
> > >>>+ dcdc1: regulator-dcdc1 {
> > >>>+ compatible = "ti,tps65218-dcdc1";
> > >>>+ /* VDD_CORE limits min of OPP50 and max of OPP100 */
> > >>>+ regulator-name = "vdd_core";
> > >>>+ regulator-min-microvolt = <912000>;
> > >>>+ regulator-max-microvolt = <1144000>;
> > >>>+ regulator-boot-on;
> > >>>+ regulator-always-on;
> > >>>+ };
> > >>>+
> > >>>+ dcdc2: regulator-dcdc2 {
> > >>>+ compatible = "ti,tps65218-dcdc2";
> > >>>+ /* VDD_MPU limits min of OPP50 and max of OPP_NITRO */
> > >>>+ regulator-name = "vdd_mpu";
> > >>>+ regulator-min-microvolt = <912000>;
> > >>>+ regulator-max-microvolt = <1378000>;
> > >>>+ regulator-boot-on;
> > >>>+ regulator-always-on;
> > >>>+ };
> > >>>+
> > >>>+ dcdc3: regulator-dcdc3 {
> > >>>+ compatible = "ti,tps65218-dcdc3";
> > >>>+ regulator-name = "vdds_ddr";
> > >>no voltage ?
> > >
> > >has no users in kernel. Also, it comes out with default, and correct,
> > >voltage.
> >
> > Device tree is description of hardware, not just who uses what in OS
> > of interest.
>
> then we should start "fixing" every single DTS in the kernel because I'm
> sure even LEDs and keypads are tied to some sort of supply, as is the
> case with this board. No driver will use it though.
>
> Also, I couldn't find anywhere which voltage is dcdc3 outputing and
> there's no shunt anywhere (other than a 0 ohm resistor) to measure
> voltage. So I kinda have my hands tied here.
>
> (goes review schematics)
>
> actually there's 0.05 ohm sense resistor. Hopefully it's easy to access.
1.35v
--
balbi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140618/de3bf90a/attachment.sig>
next prev parent reply other threads:[~2014-06-18 23:23 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-18 15:43 [PATCH v2 0/2] arm: dts: add support for am437x sk Felipe Balbi
2014-06-18 15:43 ` [PATCH v2 1/2] arm: dts: am4372: let boards enable RTC and Watchdog Felipe Balbi
2014-06-18 15:55 ` Nishanth Menon
2014-06-18 19:25 ` Felipe Balbi
2014-06-18 21:43 ` Nishanth Menon
2014-06-18 21:51 ` Felipe Balbi
2014-06-18 21:55 ` Nishanth Menon
2014-06-18 15:43 ` [PATCH v2 2/2] arm: dts: add support for AM437x StarterKit Felipe Balbi
2014-06-18 16:14 ` Nishanth Menon
2014-06-18 19:31 ` Felipe Balbi
2014-06-18 21:54 ` Nishanth Menon
2014-06-18 23:19 ` Felipe Balbi
2014-06-18 23:23 ` Felipe Balbi [this message]
2014-06-19 2:26 ` Nishanth Menon
2014-06-19 3:05 ` Felipe Balbi
2014-06-19 3:17 ` Nishanth Menon
2014-06-19 4:02 ` Felipe Balbi
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=20140618232327.GC9855@saruman.home \
--to=balbi@ti.com \
--cc=linux-arm-kernel@lists.infradead.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).