From: Jan Kardell <jan.kardell@telliq.com>
To: Robert Nelson <robertcnelson@gmail.com>,
tony@atomide.com, devicetree@vger.kernel.org
Cc: linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
Vagrant Cascadian <vagrant@debian.org>
Subject: Re: [PATCH] ARM: dts: am335x-boneblack: set dcdc1 regulator for 1.35v ddr3
Date: Sun, 12 Oct 2014 15:08:24 +0200 [thread overview]
Message-ID: <543A7D48.1080808@telliq.com> (raw)
In-Reply-To: <1412973219-3674-1-git-send-email-robertcnelson@gmail.com>
Robert Nelson skrev:
> The TPS65217C used on the boneblack defaults to 1.5v on startup for dcdc1_reg.
> While 1.35v ddr3 memory is actually used. This was discovered by a user during
> a schematic review of his beaglebone-black clone, a u-boot patch will also also
> be submitted.
Isn't it dual voltage, 1,35 or 1,5 V?
And what happens when u-boot voltage does not match the kernel? Found this:
If the SDRAM is powered up and initialized for the 1.5V operating
voltage range, voltage
can be reduced to the 1.35V operation range provided that:
Just prior to reducing the 1.5V operating voltages, no further
commands are issued,
other than NOPs or COMMAND INHIBITs, and all banks are in the precharge
state.
The 1.35V operating voltages are stable prior to issuing new commands,
other than
NOPs or COMMAND INHIBITs.
The DLL is reset and relocked after the 1.35V operating voltages are
stable and prior to
any READ command.
The ZQ calibration is performed. tZQinit must be satisfied after the
1.35V operating
voltages are stable and prior to any READ command.
Actually I'm not sure what this means in practice, but maybe this is a
problem?
//Jan
>
> Signed-off-by: Robert Nelson <robertcnelson@gmail.com>
> CC: Tony Lindgren <tony@atomide.com>
> CC: Vagrant Cascadian <vagrant@debian.org>
> ---
> arch/arm/boot/dts/am335x-boneblack.dts | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/arch/arm/boot/dts/am335x-boneblack.dts b/arch/arm/boot/dts/am335x-boneblack.dts
> index 901739f..e7c8999 100644
> --- a/arch/arm/boot/dts/am335x-boneblack.dts
> +++ b/arch/arm/boot/dts/am335x-boneblack.dts
> @@ -15,6 +15,13 @@
> compatible = "ti,am335x-bone-black", "ti,am335x-bone", "ti,am33xx";
> };
>
> +&dcdc1_reg {
> + /* TPS65217C defaults to 1.5v on powerup while 1.35v DDR3 is used */
> + regulator-min-microvolt = <1350000>;
> + regulator-max-microvolt = <1350000>;
> + regulator-boot-on;
> +};
> +
> &ldo3_reg {
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <1800000>;
--
"I have always wished for my computer to be as easy to use as my telephone; my wish has come true because I can no longer figure out how to use my telephone."
- Bjarne Stroustrup
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: jan.kardell@telliq.com (Jan Kardell)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: dts: am335x-boneblack: set dcdc1 regulator for 1.35v ddr3
Date: Sun, 12 Oct 2014 15:08:24 +0200 [thread overview]
Message-ID: <543A7D48.1080808@telliq.com> (raw)
In-Reply-To: <1412973219-3674-1-git-send-email-robertcnelson@gmail.com>
Robert Nelson skrev:
> The TPS65217C used on the boneblack defaults to 1.5v on startup for dcdc1_reg.
> While 1.35v ddr3 memory is actually used. This was discovered by a user during
> a schematic review of his beaglebone-black clone, a u-boot patch will also also
> be submitted.
Isn't it dual voltage, 1,35 or 1,5 V?
And what happens when u-boot voltage does not match the kernel? Found this:
If the SDRAM is powered up and initialized for the 1.5V operating
voltage range, voltage
can be reduced to the 1.35V operation range provided that:
? Just prior to reducing the 1.5V operating voltages, no further
commands are issued,
other than NOPs or COMMAND INHIBITs, and all banks are in the precharge
state.
? The 1.35V operating voltages are stable prior to issuing new commands,
other than
NOPs or COMMAND INHIBITs.
? The DLL is reset and relocked after the 1.35V operating voltages are
stable and prior to
any READ command.
? The ZQ calibration is performed. tZQinit must be satisfied after the
1.35V operating
voltages are stable and prior to any READ command.
Actually I'm not sure what this means in practice, but maybe this is a
problem?
//Jan
>
> Signed-off-by: Robert Nelson <robertcnelson@gmail.com>
> CC: Tony Lindgren <tony@atomide.com>
> CC: Vagrant Cascadian <vagrant@debian.org>
> ---
> arch/arm/boot/dts/am335x-boneblack.dts | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/arch/arm/boot/dts/am335x-boneblack.dts b/arch/arm/boot/dts/am335x-boneblack.dts
> index 901739f..e7c8999 100644
> --- a/arch/arm/boot/dts/am335x-boneblack.dts
> +++ b/arch/arm/boot/dts/am335x-boneblack.dts
> @@ -15,6 +15,13 @@
> compatible = "ti,am335x-bone-black", "ti,am335x-bone", "ti,am33xx";
> };
>
> +&dcdc1_reg {
> + /* TPS65217C defaults to 1.5v on powerup while 1.35v DDR3 is used */
> + regulator-min-microvolt = <1350000>;
> + regulator-max-microvolt = <1350000>;
> + regulator-boot-on;
> +};
> +
> &ldo3_reg {
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <1800000>;
--
"I have always wished for my computer to be as easy to use as my telephone; my wish has come true because I can no longer figure out how to use my telephone."
- Bjarne Stroustrup
next prev parent reply other threads:[~2014-10-12 13:08 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-10 20:33 [PATCH] ARM: dts: am335x-boneblack: set dcdc1 regulator for 1.35v ddr3 Robert Nelson
2014-10-10 20:33 ` Robert Nelson
2014-10-12 13:08 ` Jan Kardell [this message]
2014-10-12 13:08 ` Jan Kardell
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=543A7D48.1080808@telliq.com \
--to=jan.kardell@telliq.com \
--cc=devicetree@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=robertcnelson@gmail.com \
--cc=tony@atomide.com \
--cc=vagrant@debian.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.