diff mbox

[2/4] arm64: dts: salvator-x: enable usb2_phy of channel 0

Message ID 1464065860-3948-3-git-send-email-yoshihiro.shimoda.uh@renesas.com (mailing list archive)
State Superseded
Headers show

Commit Message

Yoshihiro Shimoda May 24, 2016, 4:57 a.m. UTC
This patch also adds a regulator node for USB2.0 to handle VBUS on/off
by the phy-rcar-gen3-usb2 driver.

Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
---
 arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 24 ++++++++++++++++++++++
 1 file changed, 24 insertions(+)

Comments

Geert Uytterhoeven May 24, 2016, 7:13 a.m. UTC | #1
Hi Shimoda-san,

On Tue, May 24, 2016 at 6:57 AM, Yoshihiro Shimoda
<yoshihiro.shimoda.uh@renesas.com> wrote:
> This patch also adds a regulator node for USB2.0 to handle VBUS on/off
> by the phy-rcar-gen3-usb2 driver.
>
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> ---
>  arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 24 ++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> index c09ead3..21e7e09 100644
> --- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> @@ -112,6 +112,17 @@
>                           1800000 0>;
>         };
>
> +       vbus0_usb2_phy0: regulator@5 {

"make dtbs W=1":

Warning (unit_address_vs_reg): Node /regulator@5 has a unit name, but
no reg property

regulator-vbus0-usb2-phy0? (oops, what a long name)

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
Yoshihiro Shimoda May 24, 2016, 7:45 a.m. UTC | #2
Hi Geert-san,

> From: linux-renesas-soc-owner@vger.kernel.org [mailto:linux-renesas-soc-owner@vger.kernel.org] On Behalf Of Geert

> Uytterhoeven

> Sent: Tuesday, May 24, 2016 4:13 PM

> 

> Hi Shimoda-san,

> 

> On Tue, May 24, 2016 at 6:57 AM, Yoshihiro Shimoda

> <yoshihiro.shimoda.uh@renesas.com> wrote:

> > This patch also adds a regulator node for USB2.0 to handle VBUS on/off

> > by the phy-rcar-gen3-usb2 driver.

> >

> > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>

> > ---

> >  arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 24 ++++++++++++++++++++++

> >  1 file changed, 24 insertions(+)

> >

> > diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts

> > index c09ead3..21e7e09 100644

> > --- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts

> > +++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts

> > @@ -112,6 +112,17 @@

> >                           1800000 0>;

> >         };

> >

> > +       vbus0_usb2_phy0: regulator@5 {

> 

> "make dtbs W=1":

> 

> Warning (unit_address_vs_reg): Node /regulator@5 has a unit name, but

> no reg property


Thank you for the review!
I missed your related patch...

> regulator-vbus0-usb2-phy0? (oops, what a long name)


Hmm, I think the "-phy0" is not needed.
So, I will fix the name as "regulator-vbus0-usb2".

Best regards,
Yoshihiro Shimoda

> Gr{oetje,eeting}s,

> 

>                         Geert

> 

> --

> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

> 

> In personal conversations with technical people, I call myself a hacker. But

> when I'm talking to journalists I just say "programmer" or something like that.

>                                 -- Linus Torvalds
diff mbox

Patch

diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
index c09ead3..21e7e09 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
+++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
@@ -112,6 +112,17 @@ 
 			  1800000 0>;
 	};
 
+	vbus0_usb2_phy0: regulator@5 {
+		compatible = "regulator-fixed";
+
+		regulator-name = "USB20_VBUS0";
+		regulator-min-microvolt = <5000000>;
+		regulator-max-microvolt = <5000000>;
+
+		gpio = <&gpio6 16 GPIO_ACTIVE_HIGH>;
+		enable-active-high;
+	};
+
 	audio_clkout: audio_clkout {
 		/*
 		 * This is same as <&rcar_sound 0>
@@ -349,6 +360,11 @@ 
 		function = "audio_clk";
 	};
 
+	usb0_pins: usb0 {
+		groups = "usb0";
+		function = "usb0";
+	};
+
 	usb1_pins: usb1 {
 		groups = "usb1";
 		function = "usb1";
@@ -541,6 +557,14 @@ 
 	status = "okay";
 };
 
+&usb2_phy0 {
+	pinctrl-0 = <&usb0_pins>;
+	pinctrl-names = "default";
+
+	vbus-supply = <&vbus0_usb2_phy0>;
+	status = "okay";
+};
+
 &usb2_phy1 {
 	pinctrl-0 = <&usb1_pins>;
 	pinctrl-names = "default";