diff mbox

[1/2] ARM: shmobile: r8a7794: add USB PHY DT support

Message ID 1590842.9hQm6l1zli@wasted.cogentembedded.com (mailing list archive)
State Changes Requested
Delegated to: Simon Horman
Headers show

Commit Message

Sergei Shtylyov Sept. 12, 2015, 10:50 p.m. UTC
Define the R8A7794 generic part of the USB PHY device node. It is up to the
board file to enable the device.

Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

---
 arch/arm/boot/dts/r8a7794.dtsi |   19 +++++++++++++++++++
 1 file changed, 19 insertions(+)


--
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

Comments

Sergei Shtylyov Sept. 16, 2015, 11:15 p.m. UTC | #1
Hello.

On 09/13/2015 01:50 AM, Sergei Shtylyov wrote:

> Define the R8A7794 generic part of the USB PHY device node. It is up to the
> board file to enable the device.
>
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>
> ---
>   arch/arm/boot/dts/r8a7794.dtsi |   19 +++++++++++++++++++
>   1 file changed, 19 insertions(+)
>
> Index: renesas/arch/arm/boot/dts/r8a7794.dtsi
> ===================================================================
> --- renesas.orig/arch/arm/boot/dts/r8a7794.dtsi
> +++ renesas/arch/arm/boot/dts/r8a7794.dtsi
> @@ -690,6 +690,25 @@
>   				 0x1000 0 0 2 &gic 0 113 IRQ_TYPE_LEVEL_HIGH>;
>   	};
>
> +	usbphy: usb-phy@e6590100 {
> +		compatible = "renesas,usb-phy-r8a7794";
> +		reg = <0 0xe6590100 0 0x100>;
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +		clocks = <&mstp7_clks R8A7794_CLK_HSUSB>;
> +		clock-names = "usbhs";

    Forgot to add "power-domains" prop here. :-(

MBR, Sergei

--
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
Simon Horman Sept. 30, 2015, 8:33 a.m. UTC | #2
On Thu, Sep 17, 2015 at 02:15:58AM +0300, Sergei Shtylyov wrote:
> Hello.
> 
> On 09/13/2015 01:50 AM, Sergei Shtylyov wrote:
> 
> >Define the R8A7794 generic part of the USB PHY device node. It is up to the
> >board file to enable the device.
> >
> >Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> >
> >---
> >  arch/arm/boot/dts/r8a7794.dtsi |   19 +++++++++++++++++++
> >  1 file changed, 19 insertions(+)
> >
> >Index: renesas/arch/arm/boot/dts/r8a7794.dtsi
> >===================================================================
> >--- renesas.orig/arch/arm/boot/dts/r8a7794.dtsi
> >+++ renesas/arch/arm/boot/dts/r8a7794.dtsi
> >@@ -690,6 +690,25 @@
> >  				 0x1000 0 0 2 &gic 0 113 IRQ_TYPE_LEVEL_HIGH>;
> >  	};
> >
> >+	usbphy: usb-phy@e6590100 {
> >+		compatible = "renesas,usb-phy-r8a7794";
> >+		reg = <0 0xe6590100 0 0x100>;
> >+		#address-cells = <1>;
> >+		#size-cells = <0>;
> >+		clocks = <&mstp7_clks R8A7794_CLK_HSUSB>;
> >+		clock-names = "usbhs";
> 
>    Forgot to add "power-domains" prop here. :-(

Are you planning to repost this?
--
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
Sergei Shtylyov Sept. 30, 2015, 11:11 a.m. UTC | #3
Hello.

On 9/30/2015 11:33 AM, Simon Horman wrote:

>>> Define the R8A7794 generic part of the USB PHY device node. It is up to the
>>> board file to enable the device.
>>>
>>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>>>
>>> ---
>>>   arch/arm/boot/dts/r8a7794.dtsi |   19 +++++++++++++++++++
>>>   1 file changed, 19 insertions(+)
>>>
>>> Index: renesas/arch/arm/boot/dts/r8a7794.dtsi
>>> ===================================================================
>>> --- renesas.orig/arch/arm/boot/dts/r8a7794.dtsi
>>> +++ renesas/arch/arm/boot/dts/r8a7794.dtsi
>>> @@ -690,6 +690,25 @@
>>>   				 0x1000 0 0 2 &gic 0 113 IRQ_TYPE_LEVEL_HIGH>;
>>>   	};
>>>
>>> +	usbphy: usb-phy@e6590100 {
>>> +		compatible = "renesas,usb-phy-r8a7794";
>>> +		reg = <0 0xe6590100 0 0x100>;
>>> +		#address-cells = <1>;
>>> +		#size-cells = <0>;
>>> +		clocks = <&mstp7_clks R8A7794_CLK_HSUSB>;
>>> +		clock-names = "usbhs";
>>
>>     Forgot to add "power-domains" prop here. :-(
>
> Are you planning to repost this?

    Yes.

MBR, Sergei

--
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
diff mbox

Patch

Index: renesas/arch/arm/boot/dts/r8a7794.dtsi
===================================================================
--- renesas.orig/arch/arm/boot/dts/r8a7794.dtsi
+++ renesas/arch/arm/boot/dts/r8a7794.dtsi
@@ -690,6 +690,25 @@ 
 				 0x1000 0 0 2 &gic 0 113 IRQ_TYPE_LEVEL_HIGH>;
 	};
 
+	usbphy: usb-phy@e6590100 {
+		compatible = "renesas,usb-phy-r8a7794";
+		reg = <0 0xe6590100 0 0x100>;
+		#address-cells = <1>;
+		#size-cells = <0>;
+		clocks = <&mstp7_clks R8A7794_CLK_HSUSB>;
+		clock-names = "usbhs";
+		status = "disabled";
+
+		usb0: usb-channel@0 {
+			reg = <0>;
+			#phy-cells = <1>;
+		};
+		usb2: usb-channel@2 {
+			reg = <2>;
+			#phy-cells = <1>;
+		};
+	};
+
 	clocks {
 		#address-cells = <2>;
 		#size-cells = <2>;