diff mbox

[4/6] arm64: renesas: salvator-x: enable usb2_phy

Message ID 1451037839-572-5-git-send-email-yoshihiro.shimoda.uh@renesas.com (mailing list archive)
State Superseded
Delegated to: Simon Horman
Headers show

Commit Message

Yoshihiro Shimoda Dec. 25, 2015, 10:03 a.m. UTC
Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
---
 arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 34 +++++++++++++++++++++-
 1 file changed, 33 insertions(+), 1 deletion(-)

Comments

Geert Uytterhoeven Jan. 5, 2016, 12:55 p.m. UTC | #1
Hi Shimoda-san,

On Fri, Dec 25, 2015 at 11:03 AM, Yoshihiro Shimoda
<yoshihiro.shimoda.uh@renesas.com> wrote:
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> ---
>  arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 34 +++++++++++++++++++++-
>  1 file changed, 33 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> index 265d12f..7353407 100644
> --- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> @@ -122,8 +122,22 @@
>                                  "audio_clkout_a", "audio_clkout3_a";
>                 renesas,function = "audio_clk";
>         };
> -};
>
> +       usb0_pins: usb0 {
> +               renesas,groups = "usb0";
> +               renesas,function = "usb";
> +       };
> +
> +       usb1_pins: usb1 {
> +               renesas,groups = "usb1";
> +               renesas,function = "usb";
> +       };
> +
> +       usb2_pins: usb2 {
> +               renesas,groups = "usb2";
> +               renesas,function = "usb";
> +       };

sh-pfc e6060000.pfc: function 'usb' not supported
sh-pfc e6060000.pfc: invalid function usb in map table
sh-pfc e6060000.pfc: function 'usb' not supported
sh-pfc e6060000.pfc: invalid function usb in map table
sh-pfc e6060000.pfc: function 'usb' not supported
sh-pfc e6060000.pfc: invalid function usb in map table

As the usb patches for pfc-r8a7795 haven't been posted yet, it's difficult
to test/comment.

Note that the functions were called "usb%u" in a previous version, and in the
pfc-r8a7795 code in the BSP.

Can you please post the pfc-r8a7795 patches, too?

Thanks!

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
--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Yoshihiro Shimoda Jan. 7, 2016, 1:56 a.m. UTC | #2
Hi Geert-san,

> From: Geert Uytterhoeven

> Sent: Tuesday, January 05, 2016 9:56 PM

> 

> Hi Shimoda-san,

> 

> On Fri, Dec 25, 2015 at 11:03 AM, Yoshihiro Shimoda

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

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

> > ---

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

> >  1 file changed, 33 insertions(+), 1 deletion(-)

> >

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

> > index 265d12f..7353407 100644

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

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

> > @@ -122,8 +122,22 @@

> >                                  "audio_clkout_a", "audio_clkout3_a";

> >                 renesas,function = "audio_clk";

> >         };

> > -};

> >

> > +       usb0_pins: usb0 {

> > +               renesas,groups = "usb0";

> > +               renesas,function = "usb";

> > +       };

> > +

> > +       usb1_pins: usb1 {

> > +               renesas,groups = "usb1";

> > +               renesas,function = "usb";

> > +       };

> > +

> > +       usb2_pins: usb2 {

> > +               renesas,groups = "usb2";

> > +               renesas,function = "usb";

> > +       };

> 

> sh-pfc e6060000.pfc: function 'usb' not supported

> sh-pfc e6060000.pfc: invalid function usb in map table

> sh-pfc e6060000.pfc: function 'usb' not supported

> sh-pfc e6060000.pfc: invalid function usb in map table

> sh-pfc e6060000.pfc: function 'usb' not supported

> sh-pfc e6060000.pfc: invalid function usb in map table

> 

> As the usb patches for pfc-r8a7795 haven't been posted yet, it's difficult

> to test/comment.

> 

> Note that the functions were called "usb%u" in a previous version, and in the

> pfc-r8a7795 code in the BSP.

> 

> Can you please post the pfc-r8a7795 patches, too?


Thank you for the point. I overlooked these error messages because
usb host functions worked correctly...

Anyway, I will make the pfc-r8a7795 patches for it later.

Best regards,
Yoshihiro Shimoda

> Thanks!

> 

> 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 265d12f..7353407 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
+++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
@@ -122,8 +122,22 @@ 
 				 "audio_clkout_a", "audio_clkout3_a";
 		renesas,function = "audio_clk";
 	};
-};
 
+	usb0_pins: usb0 {
+		renesas,groups = "usb0";
+		renesas,function = "usb";
+	};
+
+	usb1_pins: usb1 {
+		renesas,groups = "usb1";
+		renesas,function = "usb";
+	};
+
+	usb2_pins: usb2 {
+		renesas,groups = "usb2";
+		renesas,function = "usb";
+	};
+};
 &scif1 {
 	pinctrl-0 = <&scif1_pins>;
 	pinctrl-names = "default";
@@ -249,3 +263,21 @@ 
 		interrupts = <11 IRQ_TYPE_LEVEL_LOW>;
 	};
 };
+
+&usb2_phy0 {
+	status = "okay";
+	pinctrl-0 = <&usb0_pins>;
+	pinctrl-names = "default";
+};
+
+&usb2_phy1 {
+	status = "okay";
+	pinctrl-0 = <&usb1_pins>;
+	pinctrl-names = "default";
+};
+
+&usb2_phy2 {
+	status = "okay";
+	pinctrl-0 = <&usb2_pins>;
+	pinctrl-names = "default";
+};