diff mbox

ARM: dts: am335x-boneblack: set dcdc1 regulator for 1.35v ddr3

Message ID 1412973219-3674-1-git-send-email-robertcnelson@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Robert Nelson Oct. 10, 2014, 8:33 p.m. UTC
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.

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(+)

Comments

Jan Kardell Oct. 12, 2014, 1:08 p.m. UTC | #1
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>;
diff mbox

Patch

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>;