diff mbox

arm64: dts: rockchip: correct vqmmc voltage for rk3399 platforms

Message ID 1508211484-210856-1-git-send-email-shawn.lin@rock-chips.com (mailing list archive)
State New, archived
Headers show

Commit Message

Shawn Lin Oct. 17, 2017, 3:38 a.m. UTC
The vcc_sd or vcc_sdio used for IO voltage for sdmmc and sdio
interface on rk3399 platform have a limitation that it can't be
larger than 3.0v, otherwise it has a potential risk for the chip.
Correct all of them.

Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>
---

 arch/arm64/boot/dts/rockchip/rk3399-firefly.dts   | 4 ++--
 arch/arm64/boot/dts/rockchip/rk3399-puma.dtsi     | 4 ++--
 arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi | 4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)

Comments

Klaus Goger Oct. 17, 2017, 9:16 a.m. UTC | #1
Hi Shawn,

> On 17.10.2017, at 05:38, Shawn Lin <shawn.lin@rock-chips.com> wrote:
> 
> The vcc_sd or vcc_sdio used for IO voltage for sdmmc and sdio
> interface on rk3399 platform have a limitation that it can't be
> larger than 3.0v, otherwise it has a potential risk for the chip.
> Correct all of them.

For rk3399-puma.dtsi
Tested-by: Klaus Goger <klaus.goger@theobroma-systems.com>

— 
Klaus
Heiko Stuebner Oct. 17, 2017, 5:50 p.m. UTC | #2
Am Dienstag, 17. Oktober 2017, 11:38:04 CEST schrieb Shawn Lin:
> The vcc_sd or vcc_sdio used for IO voltage for sdmmc and sdio
> interface on rk3399 platform have a limitation that it can't be
> larger than 3.0v, otherwise it has a potential risk for the chip.
> Correct all of them.
> 
> Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>

applied as fix for 4.14 with Klaus' tested tag.


Thanks
Heiko
diff mbox

Patch

diff --git a/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts b/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts
index da373dd..4f28628 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts
@@ -382,10 +382,10 @@ 
 				regulator-always-on;
 				regulator-boot-on;
 				regulator-min-microvolt = <1800000>;
-				regulator-max-microvolt = <3300000>;
+				regulator-max-microvolt = <3000000>;
 				regulator-state-mem {
 					regulator-on-in-suspend;
-					regulator-suspend-microvolt = <3300000>;
+					regulator-suspend-microvolt = <3000000>;
 				};
 			};
 
diff --git a/arch/arm64/boot/dts/rockchip/rk3399-puma.dtsi b/arch/arm64/boot/dts/rockchip/rk3399-puma.dtsi
index 53ff3d1..910628d 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-puma.dtsi
+++ b/arch/arm64/boot/dts/rockchip/rk3399-puma.dtsi
@@ -325,12 +325,12 @@ 
 			vcc_sd: LDO_REG4 {
 				regulator-name = "vcc_sd";
 				regulator-min-microvolt = <1800000>;
-				regulator-max-microvolt = <3300000>;
+				regulator-max-microvolt = <3000000>;
 				regulator-always-on;
 				regulator-boot-on;
 				regulator-state-mem {
 					regulator-on-in-suspend;
-					regulator-suspend-microvolt = <3300000>;
+					regulator-suspend-microvolt = <3000000>;
 				};
 			};
 
diff --git a/arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi b/arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi
index 6c30bb0..0f873c8 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi
+++ b/arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi
@@ -315,10 +315,10 @@ 
 				regulator-always-on;
 				regulator-boot-on;
 				regulator-min-microvolt = <1800000>;
-				regulator-max-microvolt = <3300000>;
+				regulator-max-microvolt = <3000000>;
 				regulator-state-mem {
 					regulator-on-in-suspend;
-					regulator-suspend-microvolt = <3300000>;
+					regulator-suspend-microvolt = <3000000>;
 				};
 			};