Skip to contentSkip to conversation

ARM: dts: exynos: fix property values of LDO15/17 for ODROID-XU3/4

PATCH New Dongjin Kim 3 participants 3 messages Oct 29, 2017 – Nov 6, 2017

Subsystems (2)
Arm (32-bit) Arm Samsung Exynos
lore
Share & tools
lore

Dongjin Kim

author
#0 lore raw

PATCH 1/1 ARM: dts: exynos: fix property values of LDO15/17 for ODROID-XU3/4

Looking at the schematic, LDO15 and LDO17 are tied as a power source of a builtin network chipset. The voltage on LDO15 is corrected to 3.3V and the name of LDO17 is corrected to "vdd_ldo17".

Signed-off-by Dongjin Kim <[email protected]>
---
 arch/arm/dts/exynos5422-odroidxu3.dts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
arch/arm/dts/exynos5422-odroidxu3.dts +3 -3
index b6f4333227..b48ca3b121 100644
@@ -124,8 +124,8 @@
 
 				ldo15_reg: LDO15 {
 					regulator-name = "vdd_ldo15";
-					regulator-min-microvolt = <3100000>;
-					regulator-max-microvolt = <3100000>;
+					regulator-min-microvolt = <3300000>;
+					regulator-max-microvolt = <3300000>;
 					regulator-always-on;
 				};
 

@@ -137,7 +137,7 @@
 				};
 
 				ldo17_reg: LDO17 {
-					regulator-name = "tsp_avdd";
+					regulator-name = "vdd_ldo17";
 					regulator-min-microvolt = <3300000>;
 					regulator-max-microvolt = <3300000>;
 					regulator-always-on;
Signature · 1 line
2.11.0
#1 lore raw

PATCH 1/1 ARM: dts: exynos: fix property values of LDO15/17 for ODROID-XU3/4

Hi Dongjin,

Quoted 28 lines

Looking at the schematic, LDO15 and LDO17 are tied as a power source of a builtin network chipset. The voltage on LDO15 is corrected to 3.3V and the name of LDO17 is corrected to "vdd_ldo17".

Signed-off-by Dongjin Kim <[email protected]>
---
 arch/arm/dts/exynos5422-odroidxu3.dts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
(quoted diff) +0 -0

b/arch/arm/dts/exynos5422-odroidxu3.dts index b6f4333227..b48ca3b121 100644 --- a/arch/arm/dts/exynos5422-odroidxu3.dts
+++ b/arch/arm/dts/exynos5422-odroidxu3.dts

(quoted diff) +2 -2
@@ -124,8 +124,8 @@
 
 				ldo15_reg: LDO15 {
 					regulator-name = "vdd_ldo15";
-					regulator-min-microvolt =
 <3100000>;
-					regulator-max-microvolt =
 <3100000>;
+					regulator-min-microvolt =
 <3300000>;
+					regulator-max-microvolt =

<3300000>; regulator-always-on;

arch/arm/dts/exynos5422-odroidxu3.dts +0 -0
 				};

arch/arm/dts/exynos5422-odroidxu3.dts line 143

Quoted 8 lines
arch/arm/dts/exynos5422-odroidxu3.dts +1 -1
@@ -137,7 +137,7 @@
 				};
 
 				ldo17_reg: LDO17 {
-					regulator-name = "tsp_avdd";
+					regulator-name = "vdd_ldo17";
 					regulator-min-microvolt =
 <3300000>; regulator-max-microvolt = <3300000>;
 					regulator-always-on;
Reviewed-by Lukasz Majewski <[email protected]>

Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de -------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171030/4ff64935/attachment.sig>

#2 lore raw

PATCH 1/1 ARM: dts: exynos: fix property values of LDO15/17 for ODROID-XU3/4

Dear Dongjin,

2017. 10. 30. 18:33에 "Lukasz Majewski" <[email protected]>님이 작성:

Hi Dongjin,

Quoted 28 lines

Looking at the schematic, LDO15 and LDO17 are tied as a power source of a builtin network chipset. The voltage on LDO15 is corrected to 3.3V and the name of LDO17 is corrected to "vdd_ldo17".

Signed-off-by Dongjin Kim <[email protected]>
---
 arch/arm/dts/exynos5422-odroidxu3.dts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
(quoted diff) +0 -0

b/arch/arm/dts/exynos5422-odroidxu3.dts index b6f4333227..b48ca3b121 100644 --- a/arch/arm/dts/exynos5422-odroidxu3.dts
+++ b/arch/arm/dts/exynos5422-odroidxu3.dts

(quoted diff) +2 -2
@@ -124,8 +124,8 @@
 
                              ldo15_reg: LDO15 {
                                      regulator-name = "vdd_ldo15";
-                                     regulator-min-microvolt =
 <3100000>;
-                                     regulator-max-microvolt =
 <3100000>;
+                                     regulator-min-microvolt =
 <3300000>;
+                                     regulator-max-microvolt =

<3300000>; regulator-always-on;
                              };

arch/arm/dts/exynos5422-odroidxu3.dts line 143

Quoted 8 lines
arch/arm/dts/exynos5422-odroidxu3.dts +1 -1
@@ -137,7 +137,7 @@
                              };
 
                              ldo17_reg: LDO17 {
-                                     regulator-name = "tsp_avdd";
+                                     regulator-name = "vdd_ldo17";
                                      regulator-min-microvolt =
 <3300000>; regulator-max-microvolt = <3300000>;
                                      regulator-always-on;
Reviewed-by Lukasz Majewski <[email protected]>

Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
https://lists.denx.de/listinfo/u-boot

applied to u-boot-samsung.

Thanks,
Minkyu Kang.