On Mon, May 06, 2019 at 06:46:31PM -0500, Chris Brandt wrote:
> Enable USB Host support for both the Type-C connector on the CPU board
> and the Type-A plug on the sub board.
>
> Both boards are also capable of USB Device operation as well after the
> appropriate Device Tree modifications.
>
> Signed-off-by: Chris Brandt <chris.brandt@xxxxxxxxxxx>
> ---
> arch/arm/boot/dts/r7s9210-rza2mevb.dts | 37 ++++++++++++++++++++++++++++++++++
> 1 file changed, 37 insertions(+)
>
> diff --git a/arch/arm/boot/dts/r7s9210-rza2mevb.dts b/arch/arm/boot/dts/r7s9210-rza2mevb.dts
> index 1eba37db7cdc..79d9d4b4779f 100644
> --- a/arch/arm/boot/dts/r7s9210-rza2mevb.dts
> +++ b/arch/arm/boot/dts/r7s9210-rza2mevb.dts
> @@ -100,6 +100,18 @@
> pinmux = <RZA2_PINMUX(PORT5, 4, 3)>, /* SD1_CD */
> <RZA2_PINMUX(PORT5, 5, 3)>; /* SD1_WP */
> };
> +
> + usb0_pins: usb0 {
> + pinmux = <RZA2_PINMUX(PORT5, 2, 3)>, /* VBUSIN0 */
> + <RZA2_PINMUX(PORTC, 6, 1)>, /* VBUSEN0 */
> + <RZA2_PINMUX(PORTC, 7, 1)>; /* OVRCUR0 */
> + };
> +
> + usb1_pins: usb1 {
> + pinmux = <RZA2_PINMUX(PORTC, 0, 1)>, /* VBUSIN1 */
> + <RZA2_PINMUX(PORTC, 5, 1)>, /* VBUSEN1 */
> + <RZA2_PINMUX(PORT7, 5, 5)>; /* OVRCUR1 */
> + };
> };
>
> /* High resolution System tick timers */
> @@ -154,3 +166,28 @@
> bus-width = <4>;
> status = "okay";
> };
> +
> +/* USB-0 as Host */
> +/* NOTE: Requires JP3 to be fitted */
> +&usb2_phy0 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&usb0_pins>;
> + renesas,uses_usb_x1;
> + dr_mode = "host";
> + status = "okay";
> +};
Hi Chris,
Please add a space between the usb2_phy0 and ehci0 nodes.
Likewise below between the usb2_phy1 and ehci1 nodes.
Otherwise this patch looks good to me.
> +&ehci0 {
> + status = "okay";
> +};
> +
> +/* USB-1 as Host */
> +&usb2_phy1 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&usb1_pins>;
> + renesas,uses_usb_x1;
> + dr_mode = "host";
> + status = "okay";
> +};
> +&ehci1 {
> + status = "okay";
> +};
> --
> 2.16.1
>
