RE: [PATCH] arm64: dts: renesas: salvator-common: add dr_mode property for USB2.0 channel 0

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Hi Simon-san,

> From: Yoshihiro Shimoda, Sent: Thursday, October 12, 2017 6:24 PM
> 
> Since Salvator-X[S] have a USB2.0 dual-role channel (CN9), this patch
> adds dr_mode property for USB2.0 channel 0 (EHCI/OHCI and HS-USB)
> as "otg".
> 
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx>
> ---
>  This patch based on the renesas.git / renesas-devel-20171011-v4.14-rc4 tag.
> 
>  Using dual-role channel (or not) is related to the type of USB receptor on
>  board specification. So, I added the property into the salvator-common.dtsi.
> 
>  This property will be used by phy-rcar-gen3-usb2 driver in the future [1].
>  However, adding this property without the [1] patch will not cause any
>  side-effects.
> 
>  [1] : https://patchwork.kernel.org/patch/10001113/

Would you apply this patch for v4.15?
I got ack from the PHY maintainer:

https://www.mail-archive.com/linux-renesas-soc@xxxxxxxxxxxxxxx/msg19494.html

Best regards,
Yoshihiro Shimoda

> 
>  arch/arm64/boot/dts/renesas/salvator-common.dtsi | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
> index 3dcb26b..9bd409b 100644
> --- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi
> +++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
> @@ -282,6 +282,7 @@
>  };
> 
>  &ehci0 {
> +	dr_mode = "otg";
>  	status = "okay";
>  };
> 
> @@ -294,6 +295,7 @@
>  };
> 
>  &hsusb {
> +	dr_mode = "otg";
>  	status = "okay";
>  };
> 
> @@ -356,6 +358,7 @@
>  };
> 
>  &ohci0 {
> +	dr_mode = "otg";
>  	status = "okay";
>  };
> 
> --
> 1.9.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux