Re: [PATCH 2/7] arm: dts: exynos: add vbus regulator to USB2 phy nodes

2015-08-22 Thread Krzysztof Kozlowski 
W dniu 21.08.2015 o 21:38, Marek Szyprowski pisze:
> Exynos USB2 PHY driver now supports VBUS regulator, so add it to all
> boards which have it available. This also fixes commit
> 7eec1266751bd3a25e35ce88686634c768fedc24 ("ARM: dts: Add Maxim 77693
> PMIC to exynos4412-trats2"), which added new regulators to Trats2 board,
> but without linking them to the consumers.
> 
> Signed-off-by: Marek Szyprowski 

Hi,

Thanks for taking care of it! Nice analysis.

1. If there would be a resend please change the "arm" to "ARM" in
subject. If not, maybe Kukjin can fix it when applying.

2. How about backporting 1st and 2nd patch to stable kernels? At least
for Trats2? Mentioned commit 7eec126675 introduces a bug (a feature
stops working) which would be nice to fix for stable releases as well.
We could also backport simpler patch, adding only "regulator-always-on".

3. What about safeout regulator #2? On Trats2 it goes to USB_VBUS of
modem. Shouldn't it be enabled?

I'll test the patchset later at work on my boards.

As for the code:
Reviewed-by: Krzysztof Kozlowski 

Best regards,
Krzysztof


> ---
>  arch/arm/boot/dts/exynos3250-monk.dts   | 1 +
>  arch/arm/boot/dts/exynos3250-rinato.dts | 1 +
>  arch/arm/boot/dts/exynos4210-trats.dts  | 2 +-
>  arch/arm/boot/dts/exynos4210-universal_c210.dts | 2 +-
>  arch/arm/boot/dts/exynos4412-trats2.dts | 1 +
>  5 files changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/exynos3250-monk.dts 
> b/arch/arm/boot/dts/exynos3250-monk.dts
> index 540a0adf2be6..35b39d2255d3 100644
> --- a/arch/arm/boot/dts/exynos3250-monk.dts
> +++ b/arch/arm/boot/dts/exynos3250-monk.dts
> @@ -161,6 +161,7 @@
>  };
>  
>  _usbphy {
> + vbus-supply = <_reg>;
>   status = "okay";
>  };
>  
> diff --git a/arch/arm/boot/dts/exynos3250-rinato.dts 
> b/arch/arm/boot/dts/exynos3250-rinato.dts
> index 41a5fafb9aa9..23623cd3ebd9 100644
> --- a/arch/arm/boot/dts/exynos3250-rinato.dts
> +++ b/arch/arm/boot/dts/exynos3250-rinato.dts
> @@ -153,6 +153,7 @@
>  
>  _usbphy {
>   status = "okay";
> + vbus-supply = <_reg>;
>  };
>  
>   {
> diff --git a/arch/arm/boot/dts/exynos4210-trats.dts 
> b/arch/arm/boot/dts/exynos4210-trats.dts
> index ba34886f8b65..01d38f2145b9 100644
> --- a/arch/arm/boot/dts/exynos4210-trats.dts
> +++ b/arch/arm/boot/dts/exynos4210-trats.dts
> @@ -251,6 +251,7 @@
>  
>  _usbphy {
>   status = "okay";
> + vbus-supply = <_sreg>;
>  };
>  
>   {
> @@ -448,7 +449,6 @@
>  
>   safe1_sreg: ESAFEOUT1 {
>regulator-name = "SAFEOUT1";
> -  regulator-always-on;
>   };
>  
>   safe2_sreg: ESAFEOUT2 {
> diff --git a/arch/arm/boot/dts/exynos4210-universal_c210.dts 
> b/arch/arm/boot/dts/exynos4210-universal_c210.dts
> index eb379526e234..2c04297825fe 100644
> --- a/arch/arm/boot/dts/exynos4210-universal_c210.dts
> +++ b/arch/arm/boot/dts/exynos4210-universal_c210.dts
> @@ -248,6 +248,7 @@
>  
>  _usbphy {
>   status = "okay";
> + vbus-supply = <_reg>;
>  };
>  
>   {
> @@ -486,7 +487,6 @@
>  
>   safeout1_reg: ESAFEOUT1 {
>   regulator-name = "SAFEOUT1";
> - regulator-always-on;
>   };
>  
>   safeout2_reg: ESAFEOUT2 {
> diff --git a/arch/arm/boot/dts/exynos4412-trats2.dts 
> b/arch/arm/boot/dts/exynos4412-trats2.dts
> index 2a1ebb76ebe0..50a5e8a85283 100644
> --- a/arch/arm/boot/dts/exynos4412-trats2.dts
> +++ b/arch/arm/boot/dts/exynos4412-trats2.dts
> @@ -391,6 +391,7 @@
>  };
>  
>  _usbphy {
> + vbus-supply = <_reg>;
>   status = "okay";
>  };
>  
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 2/7] arm: dts: exynos: add vbus regulator to USB2 phy nodes

2015-08-22 Thread Krzysztof Kozlowski 
W dniu 21.08.2015 o 21:38, Marek Szyprowski pisze:
 Exynos USB2 PHY driver now supports VBUS regulator, so add it to all
 boards which have it available. This also fixes commit
 7eec1266751bd3a25e35ce88686634c768fedc24 (ARM: dts: Add Maxim 77693
 PMIC to exynos4412-trats2), which added new regulators to Trats2 board,
 but without linking them to the consumers.
 
 Signed-off-by: Marek Szyprowski m.szyprow...@samsung.com

Hi,

Thanks for taking care of it! Nice analysis.

1. If there would be a resend please change the arm to ARM in
subject. If not, maybe Kukjin can fix it when applying.

2. How about backporting 1st and 2nd patch to stable kernels? At least
for Trats2? Mentioned commit 7eec126675 introduces a bug (a feature
stops working) which would be nice to fix for stable releases as well.
We could also backport simpler patch, adding only regulator-always-on.

3. What about safeout regulator #2? On Trats2 it goes to USB_VBUS of
modem. Shouldn't it be enabled?

I'll test the patchset later at work on my boards.

As for the code:
Reviewed-by: Krzysztof Kozlowski k.kozlow...@samsung.com

Best regards,
Krzysztof


 ---
  arch/arm/boot/dts/exynos3250-monk.dts   | 1 +
  arch/arm/boot/dts/exynos3250-rinato.dts | 1 +
  arch/arm/boot/dts/exynos4210-trats.dts  | 2 +-
  arch/arm/boot/dts/exynos4210-universal_c210.dts | 2 +-
  arch/arm/boot/dts/exynos4412-trats2.dts | 1 +
  5 files changed, 5 insertions(+), 2 deletions(-)
 
 diff --git a/arch/arm/boot/dts/exynos3250-monk.dts 
 b/arch/arm/boot/dts/exynos3250-monk.dts
 index 540a0adf2be6..35b39d2255d3 100644
 --- a/arch/arm/boot/dts/exynos3250-monk.dts
 +++ b/arch/arm/boot/dts/exynos3250-monk.dts
 @@ -161,6 +161,7 @@
  };
  
  exynos_usbphy {
 + vbus-supply = safeout_reg;
   status = okay;
  };
  
 diff --git a/arch/arm/boot/dts/exynos3250-rinato.dts 
 b/arch/arm/boot/dts/exynos3250-rinato.dts
 index 41a5fafb9aa9..23623cd3ebd9 100644
 --- a/arch/arm/boot/dts/exynos3250-rinato.dts
 +++ b/arch/arm/boot/dts/exynos3250-rinato.dts
 @@ -153,6 +153,7 @@
  
  exynos_usbphy {
   status = okay;
 + vbus-supply = safeout_reg;
  };
  
  hsotg {
 diff --git a/arch/arm/boot/dts/exynos4210-trats.dts 
 b/arch/arm/boot/dts/exynos4210-trats.dts
 index ba34886f8b65..01d38f2145b9 100644
 --- a/arch/arm/boot/dts/exynos4210-trats.dts
 +++ b/arch/arm/boot/dts/exynos4210-trats.dts
 @@ -251,6 +251,7 @@
  
  exynos_usbphy {
   status = okay;
 + vbus-supply = safe1_sreg;
  };
  
  fimd {
 @@ -448,7 +449,6 @@
  
   safe1_sreg: ESAFEOUT1 {
regulator-name = SAFEOUT1;
 -  regulator-always-on;
   };
  
   safe2_sreg: ESAFEOUT2 {
 diff --git a/arch/arm/boot/dts/exynos4210-universal_c210.dts 
 b/arch/arm/boot/dts/exynos4210-universal_c210.dts
 index eb379526e234..2c04297825fe 100644
 --- a/arch/arm/boot/dts/exynos4210-universal_c210.dts
 +++ b/arch/arm/boot/dts/exynos4210-universal_c210.dts
 @@ -248,6 +248,7 @@
  
  exynos_usbphy {
   status = okay;
 + vbus-supply = safeout1_reg;
  };
  
  fimd {
 @@ -486,7 +487,6 @@
  
   safeout1_reg: ESAFEOUT1 {
   regulator-name = SAFEOUT1;
 - regulator-always-on;
   };
  
   safeout2_reg: ESAFEOUT2 {
 diff --git a/arch/arm/boot/dts/exynos4412-trats2.dts 
 b/arch/arm/boot/dts/exynos4412-trats2.dts
 index 2a1ebb76ebe0..50a5e8a85283 100644
 --- a/arch/arm/boot/dts/exynos4412-trats2.dts
 +++ b/arch/arm/boot/dts/exynos4412-trats2.dts
 @@ -391,6 +391,7 @@
  };
  
  exynos_usbphy {
 + vbus-supply = esafeout1_reg;
   status = okay;
  };
  
 

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 2/7] arm: dts: exynos: add vbus regulator to USB2 phy nodes

2015-08-21 Thread Marek Szyprowski 
Exynos USB2 PHY driver now supports VBUS regulator, so add it to all
boards which have it available. This also fixes commit
7eec1266751bd3a25e35ce88686634c768fedc24 ("ARM: dts: Add Maxim 77693
PMIC to exynos4412-trats2"), which added new regulators to Trats2 board,
but without linking them to the consumers.

Signed-off-by: Marek Szyprowski 
---
 arch/arm/boot/dts/exynos3250-monk.dts   | 1 +
 arch/arm/boot/dts/exynos3250-rinato.dts | 1 +
 arch/arm/boot/dts/exynos4210-trats.dts  | 2 +-
 arch/arm/boot/dts/exynos4210-universal_c210.dts | 2 +-
 arch/arm/boot/dts/exynos4412-trats2.dts | 1 +
 5 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/exynos3250-monk.dts 
b/arch/arm/boot/dts/exynos3250-monk.dts
index 540a0adf2be6..35b39d2255d3 100644
--- a/arch/arm/boot/dts/exynos3250-monk.dts
+++ b/arch/arm/boot/dts/exynos3250-monk.dts
@@ -161,6 +161,7 @@
 };
 
 _usbphy {
+   vbus-supply = <_reg>;
status = "okay";
 };
 
diff --git a/arch/arm/boot/dts/exynos3250-rinato.dts 
b/arch/arm/boot/dts/exynos3250-rinato.dts
index 41a5fafb9aa9..23623cd3ebd9 100644
--- a/arch/arm/boot/dts/exynos3250-rinato.dts
+++ b/arch/arm/boot/dts/exynos3250-rinato.dts
@@ -153,6 +153,7 @@
 
 _usbphy {
status = "okay";
+   vbus-supply = <_reg>;
 };
 
  {
diff --git a/arch/arm/boot/dts/exynos4210-trats.dts 
b/arch/arm/boot/dts/exynos4210-trats.dts
index ba34886f8b65..01d38f2145b9 100644
--- a/arch/arm/boot/dts/exynos4210-trats.dts
+++ b/arch/arm/boot/dts/exynos4210-trats.dts
@@ -251,6 +251,7 @@
 
 _usbphy {
status = "okay";
+   vbus-supply = <_sreg>;
 };
 
  {
@@ -448,7 +449,6 @@
 
safe1_sreg: ESAFEOUT1 {
 regulator-name = "SAFEOUT1";
-regulator-always-on;
};
 
safe2_sreg: ESAFEOUT2 {
diff --git a/arch/arm/boot/dts/exynos4210-universal_c210.dts 
b/arch/arm/boot/dts/exynos4210-universal_c210.dts
index eb379526e234..2c04297825fe 100644
--- a/arch/arm/boot/dts/exynos4210-universal_c210.dts
+++ b/arch/arm/boot/dts/exynos4210-universal_c210.dts
@@ -248,6 +248,7 @@
 
 _usbphy {
status = "okay";
+   vbus-supply = <_reg>;
 };
 
  {
@@ -486,7 +487,6 @@
 
safeout1_reg: ESAFEOUT1 {
regulator-name = "SAFEOUT1";
-   regulator-always-on;
};
 
safeout2_reg: ESAFEOUT2 {
diff --git a/arch/arm/boot/dts/exynos4412-trats2.dts 
b/arch/arm/boot/dts/exynos4412-trats2.dts
index 2a1ebb76ebe0..50a5e8a85283 100644
--- a/arch/arm/boot/dts/exynos4412-trats2.dts
+++ b/arch/arm/boot/dts/exynos4412-trats2.dts
@@ -391,6 +391,7 @@
 };
 
 _usbphy {
+   vbus-supply = <_reg>;
status = "okay";
 };
 
-- 
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 2/7] arm: dts: exynos: add vbus regulator to USB2 phy nodes

2015-08-21 Thread Marek Szyprowski 
Exynos USB2 PHY driver now supports VBUS regulator, so add it to all
boards which have it available. This also fixes commit
7eec1266751bd3a25e35ce88686634c768fedc24 (ARM: dts: Add Maxim 77693
PMIC to exynos4412-trats2), which added new regulators to Trats2 board,
but without linking them to the consumers.

Signed-off-by: Marek Szyprowski m.szyprow...@samsung.com
---
 arch/arm/boot/dts/exynos3250-monk.dts   | 1 +
 arch/arm/boot/dts/exynos3250-rinato.dts | 1 +
 arch/arm/boot/dts/exynos4210-trats.dts  | 2 +-
 arch/arm/boot/dts/exynos4210-universal_c210.dts | 2 +-
 arch/arm/boot/dts/exynos4412-trats2.dts | 1 +
 5 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/exynos3250-monk.dts 
b/arch/arm/boot/dts/exynos3250-monk.dts
index 540a0adf2be6..35b39d2255d3 100644
--- a/arch/arm/boot/dts/exynos3250-monk.dts
+++ b/arch/arm/boot/dts/exynos3250-monk.dts
@@ -161,6 +161,7 @@
 };
 
 exynos_usbphy {
+   vbus-supply = safeout_reg;
status = okay;
 };
 
diff --git a/arch/arm/boot/dts/exynos3250-rinato.dts 
b/arch/arm/boot/dts/exynos3250-rinato.dts
index 41a5fafb9aa9..23623cd3ebd9 100644
--- a/arch/arm/boot/dts/exynos3250-rinato.dts
+++ b/arch/arm/boot/dts/exynos3250-rinato.dts
@@ -153,6 +153,7 @@
 
 exynos_usbphy {
status = okay;
+   vbus-supply = safeout_reg;
 };
 
 hsotg {
diff --git a/arch/arm/boot/dts/exynos4210-trats.dts 
b/arch/arm/boot/dts/exynos4210-trats.dts
index ba34886f8b65..01d38f2145b9 100644
--- a/arch/arm/boot/dts/exynos4210-trats.dts
+++ b/arch/arm/boot/dts/exynos4210-trats.dts
@@ -251,6 +251,7 @@
 
 exynos_usbphy {
status = okay;
+   vbus-supply = safe1_sreg;
 };
 
 fimd {
@@ -448,7 +449,6 @@
 
safe1_sreg: ESAFEOUT1 {
 regulator-name = SAFEOUT1;
-regulator-always-on;
};
 
safe2_sreg: ESAFEOUT2 {
diff --git a/arch/arm/boot/dts/exynos4210-universal_c210.dts 
b/arch/arm/boot/dts/exynos4210-universal_c210.dts
index eb379526e234..2c04297825fe 100644
--- a/arch/arm/boot/dts/exynos4210-universal_c210.dts
+++ b/arch/arm/boot/dts/exynos4210-universal_c210.dts
@@ -248,6 +248,7 @@
 
 exynos_usbphy {
status = okay;
+   vbus-supply = safeout1_reg;
 };
 
 fimd {
@@ -486,7 +487,6 @@
 
safeout1_reg: ESAFEOUT1 {
regulator-name = SAFEOUT1;
-   regulator-always-on;
};
 
safeout2_reg: ESAFEOUT2 {
diff --git a/arch/arm/boot/dts/exynos4412-trats2.dts 
b/arch/arm/boot/dts/exynos4412-trats2.dts
index 2a1ebb76ebe0..50a5e8a85283 100644
--- a/arch/arm/boot/dts/exynos4412-trats2.dts
+++ b/arch/arm/boot/dts/exynos4412-trats2.dts
@@ -391,6 +391,7 @@
 };
 
 exynos_usbphy {
+   vbus-supply = esafeout1_reg;
status = okay;
 };
 
-- 
1.9.2

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/