All of lore.kernel.org
 help / color / mirror / Atom feed
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

  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.