diff mbox

[v5,2/2] ARM: dts: sun9i: optimus: Switch to phy core regulator bindings for usb phys

Message ID 1426305437-15208-3-git-send-email-wens@csie.org (mailing list archive)
State New, archived
Headers show

Commit Message

Chen-Yu Tsai March 14, 2015, 3:57 a.m. UTC
Since the phy core already supports specifying a regulator to handle
during power up/down, it was decided to drop the regulator support
in the sun9i usb phy driver.

This patch switches the DT to the core bindings. This and the phy driver
would be in the same release and should not be a problem as far as DT
stability goes.

Signed-off-by: Chen-Yu Tsai <wens@csie.org>
---
 arch/arm/boot/dts/sun9i-a80-optimus.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Maxime Ripard March 17, 2015, 8:38 a.m. UTC | #1
On Sat, Mar 14, 2015 at 11:57:17AM +0800, Chen-Yu Tsai wrote:
> Since the phy core already supports specifying a regulator to handle
> during power up/down, it was decided to drop the regulator support
> in the sun9i usb phy driver.
> 
> This patch switches the DT to the core bindings. This and the phy driver
> would be in the same release and should not be a problem as far as DT
> stability goes.
> 
> Signed-off-by: Chen-Yu Tsai <wens@csie.org>

It looks fine for me. I'll wait for Kishon approval before merging
this patch.

Thanks!
Maxime
Kishon Vijay Abraham I March 18, 2015, 2:52 p.m. UTC | #2
On Saturday 14 March 2015 09:27 AM, Chen-Yu Tsai wrote:
> Since the phy core already supports specifying a regulator to handle
> during power up/down, it was decided to drop the regulator support
> in the sun9i usb phy driver.
>
> This patch switches the DT to the core bindings. This and the phy driver
> would be in the same release and should not be a problem as far as DT
> stability goes.
>
> Signed-off-by: Chen-Yu Tsai <wens@csie.org>

looks good..

Acked-by: Kishon Vijay Abraham I <kishon@ti.com>

> ---
>   arch/arm/boot/dts/sun9i-a80-optimus.dts | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/sun9i-a80-optimus.dts b/arch/arm/boot/dts/sun9i-a80-optimus.dts
> index e53f2656c2d0..c6ca116c59d2 100644
> --- a/arch/arm/boot/dts/sun9i-a80-optimus.dts
> +++ b/arch/arm/boot/dts/sun9i-a80-optimus.dts
> @@ -203,7 +203,7 @@
>   };
>
>   &usbphy1 {
> -	vbus-supply = <&reg_usb1_vbus>;
> +	phy-supply = <&reg_usb1_vbus>;
>   	status = "okay";
>   };
>
> @@ -212,6 +212,6 @@
>   };
>
>   &usbphy3 {
> -	vbus-supply = <&reg_usb3_vbus>;
> +	phy-supply = <&reg_usb3_vbus>;
>   	status = "okay";
>   };
>
Maxime Ripard March 18, 2015, 5:37 p.m. UTC | #3
On Sat, Mar 14, 2015 at 11:57:17AM +0800, Chen-Yu Tsai wrote:
> Since the phy core already supports specifying a regulator to handle
> during power up/down, it was decided to drop the regulator support
> in the sun9i usb phy driver.
> 
> This patch switches the DT to the core bindings. This and the phy driver
> would be in the same release and should not be a problem as far as DT
> stability goes.
> 
> Signed-off-by: Chen-Yu Tsai <wens@csie.org>

Applied with Kishon's Acked By

Thanks!
Maxime
diff mbox

Patch

diff --git a/arch/arm/boot/dts/sun9i-a80-optimus.dts b/arch/arm/boot/dts/sun9i-a80-optimus.dts
index e53f2656c2d0..c6ca116c59d2 100644
--- a/arch/arm/boot/dts/sun9i-a80-optimus.dts
+++ b/arch/arm/boot/dts/sun9i-a80-optimus.dts
@@ -203,7 +203,7 @@ 
 };
 
 &usbphy1 {
-	vbus-supply = <&reg_usb1_vbus>;
+	phy-supply = <&reg_usb1_vbus>;
 	status = "okay";
 };
 
@@ -212,6 +212,6 @@ 
 };
 
 &usbphy3 {
-	vbus-supply = <&reg_usb3_vbus>;
+	phy-supply = <&reg_usb3_vbus>;
 	status = "okay";
 };