linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: exynos: fix property values of LDO15/17 for ODROID-XU3/4
@ 2017-10-28 17:32 tobetter
  2017-10-30 20:55 ` Krzysztof Kozlowski
  2017-11-27 17:47 ` Krzysztof Kozlowski
  0 siblings, 2 replies; 3+ messages in thread
From: tobetter @ 2017-10-28 17:32 UTC (permalink / raw)
  To: linux-samsung-soc
  Cc: Dongjin Kim, Rob Herring, Mark Rutland, Russell King, Kukjin Kim,
	Krzysztof Kozlowski, devicetree, linux-arm-kernel, linux-kernel

From: Dongjin Kim <tobetter@gmail.com>

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 <tobetter@gmail.com>
---
 arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi b/arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi
index a183b56283f8..68780569371d 100644
--- a/arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi
+++ b/arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi
@@ -372,8 +372,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;
 			};
 
@@ -385,7 +385,7 @@
 			};
 
 			ldo17_reg: LDO17 {
-				regulator-name = "tsp_avdd";
+				regulator-name = "vdd_ldo17";
 				regulator-min-microvolt = <3300000>;
 				regulator-max-microvolt = <3300000>;
 				regulator-always-on;
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: dts: exynos: fix property values of LDO15/17 for ODROID-XU3/4
  2017-10-28 17:32 [PATCH] ARM: dts: exynos: fix property values of LDO15/17 for ODROID-XU3/4 tobetter
@ 2017-10-30 20:55 ` Krzysztof Kozlowski
  2017-11-27 17:47 ` Krzysztof Kozlowski
  1 sibling, 0 replies; 3+ messages in thread
From: Krzysztof Kozlowski @ 2017-10-30 20:55 UTC (permalink / raw)
  To: tobetter
  Cc: linux-samsung-soc, Rob Herring, Mark Rutland, Russell King,
	Kukjin Kim, devicetree, linux-arm-kernel, linux-kernel

On Sat, Oct 28, 2017 at 01:32:48PM -0400, tobetter@gmail.com wrote:
> From: Dongjin Kim <tobetter@gmail.com>
> 
> 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 <tobetter@gmail.com>
> ---
>  arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)


Looks OK, I will queue it after merge window. For my own reference:

Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: dts: exynos: fix property values of LDO15/17 for ODROID-XU3/4
  2017-10-28 17:32 [PATCH] ARM: dts: exynos: fix property values of LDO15/17 for ODROID-XU3/4 tobetter
  2017-10-30 20:55 ` Krzysztof Kozlowski
@ 2017-11-27 17:47 ` Krzysztof Kozlowski
  1 sibling, 0 replies; 3+ messages in thread
From: Krzysztof Kozlowski @ 2017-11-27 17:47 UTC (permalink / raw)
  To: tobetter
  Cc: linux-samsung-soc, Rob Herring, Mark Rutland, Russell King,
	Kukjin Kim, devicetree, linux-arm-kernel, linux-kernel

On Sat, Oct 28, 2017 at 01:32:48PM -0400, tobetter@gmail.com wrote:
> From: Dongjin Kim <tobetter@gmail.com>
> 
> 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 <tobetter@gmail.com>
> ---
>  arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 

Patch does not apply on top of v4.15-rc1. Please rebase on v4.15-rc1 (or
on my next/dt branch).

Best regards,
Krzysztof

> diff --git a/arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi b/arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi
> index a183b56283f8..68780569371d 100644
> --- a/arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi
> +++ b/arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi
> @@ -372,8 +372,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;
>  			};
>  
> @@ -385,7 +385,7 @@
>  			};
>  
>  			ldo17_reg: LDO17 {
> -				regulator-name = "tsp_avdd";
> +				regulator-name = "vdd_ldo17";
>  				regulator-min-microvolt = <3300000>;
>  				regulator-max-microvolt = <3300000>;
>  				regulator-always-on;
> -- 
> 2.11.0
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-11-27 17:47 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-28 17:32 [PATCH] ARM: dts: exynos: fix property values of LDO15/17 for ODROID-XU3/4 tobetter
2017-10-30 20:55 ` Krzysztof Kozlowski
2017-11-27 17:47 ` Krzysztof Kozlowski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).