| Message ID | 1440160723-6636-3-git-send-email-m.szyprowski@samsung.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
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.szyprowski@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.kozlowski@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-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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"; };
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.szyprowski@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(-)