Skip to contentSkip to conversation

usb: renesas_usbhs: Add device tree bindings documentation

PATCH Superseded Yoshihiro Shimoda 2 participants 3 messages Aug 1, 2014 – Aug 4, 2014 1 message not in the archive

Subsystems (3)
Device tree bindings USB SuperH

v1 of 3 Superseded Latest is v3 →

lore
Share & tools
lore

Yoshihiro Shimoda

author
#0 lore raw

PATCH 1/2 usb: renesas_usbhs: Add device tree bindings documentation

Document the device tree bindings for the Renesas USBHS controller.

Signed-off-by Yoshihiro Shimoda <[email protected]>
---
 .../devicetree/bindings/usb/renesas_usbhs.txt      |   24 ++++++++++++++++++++
 1 file changed, 24 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/usb/renesas_usbhs.txt
Documentation/devicetree/bindings/usb/renesas_usbhs.txt new file +24 -0
new file mode 100644
index 0000000..e9faa65
@@ -0,0 +1,24 @@
+Renesas Electronics USBHS driver
+
+Required properties:
+  - compatible: Must contain one of the following:
+	- "renesas,usbhs-r8a7790"
+	- "renesas,usbhs-r8a7791"
+  - reg: Base address and length of the register for the USBHS
+  - interrupts: Interrupt specifier for the USBHS
+  - clocks: A list of phandle + clock specifier pairs
+
+Optional properties:
+  - buswait_bwait: Integer to use BUSWAIT register
+  - enable-gpio: A gpio specifier to check GPIO determining if USB function
+		 should be enabled
+  - phys: phandle + phy specifier pair
+  - phy-names: must be "usb"
+
+Example:
+	usbhs: usb@e6590000 {
+		compatible = "renesas,usbhs-r8a7790";
+		reg = <0 0xe6590000 0 0x100>;
+		interrupts = <0 107 IRQ_TYPE_LEVEL_HIGH>;
+		clocks = <&mstp7_clks R8A7790_CLK_HSUSB>;
+	};
Signature · 1 line
1.7.9.5

A message that is not in the archive (sent off-list or not yet received): [email protected]

#2 lore raw

Hello.

Quoted 16 lines · Yoshihiro Shimoda

Document the device tree bindings for the Renesas USBHS controller.

Signed-off-by Yoshihiro Shimoda <[email protected]>
---
  .../devicetree/bindings/usb/renesas_usbhs.txt      |   24 ++++++++++++++++++++
  1 file changed, 24 insertions(+)

create mode 100644 Documentation/devicetree/bindings/usb/renesas_usbhs.txt

Documentation/devicetree/bindings/usb/renesas_usbhs.txt new file +2 -0
new file mode 100644
index 0000000..e9faa65
@@ -0,0 +1,24 @@
+Renesas Electronics USBHS driver
+

Documentation/devicetree/bindings/usb/renesas_usbhs.txt line 12

Quoted 10 lines
Documentation/devicetree/bindings/usb/renesas_usbhs.txt new file +10 -0
+Required properties:
+  - compatible: Must contain one of the following:
+	- "renesas,usbhs-r8a7790"
+	- "renesas,usbhs-r8a7791"
+  - reg: Base address and length of the register for the USBHS
+  - interrupts: Interrupt specifier for the USBHS
+  - clocks: A list of phandle + clock specifier pairs
+
+Optional properties:
+  - buswait_bwait: Integer to use BUSWAIT register

Should be named "renesas,buswait-bwait" or probably just "renesas,buswait".

Documentation/devicetree/bindings/usb/renesas_usbhs.txt line 13

Quoted 1 line
Documentation/devicetree/bindings/usb/renesas_usbhs.txt new file +1 -0
+  - enable-gpio: A gpio specifier to check GPIO determining if USB function

Should also be prefixed with "renesas,".

[...]

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html

author
#3 lore raw

Hello,

Quoted 24 lines · Sergei Shtylyov

Hello.

Quoted 20 lines · Yoshihiro Shimoda

Document the device tree bindings for the Renesas USBHS controller.

Signed-off-by Yoshihiro Shimoda <[email protected]>
---
  .../devicetree/bindings/usb/renesas_usbhs.txt      |   24 ++++++++++++++++++++
  1 file changed, 24 insertions(+)

create mode 100644 Documentation/devicetree/bindings/usb/renesas_usbhs.txt

Documentation/devicetree/bindings/usb/renesas_usbhs.txt new file +6 -0
new file mode 100644
index 0000000..e9faa65
@@ -0,0 +1,24 @@
+Renesas Electronics USBHS driver
+
+Required properties:
+  - compatible: Must contain one of the following:
+	- "renesas,usbhs-r8a7790"
+	- "renesas,usbhs-r8a7791"
Quoted 8 lines

Documentation/devicetree/bindings/usb/renesas_usbhs.txt line 12

Quoted 6 lines
Documentation/devicetree/bindings/usb/renesas_usbhs.txt new file +6 -0
+  - reg: Base address and length of the register for the USBHS
+  - interrupts: Interrupt specifier for the USBHS
+  - clocks: A list of phandle + clock specifier pairs
+
+Optional properties:
+  - buswait_bwait: Integer to use BUSWAIT register

Should be named "renesas,buswait-bwait" or probably just "renesas,buswait".

Thank you for the point. I will fix this name to "renesas,buswait".

Quoted 3 lines

Documentation/devicetree/bindings/usb/renesas_usbhs.txt line 13

Quoted 1 line
Documentation/devicetree/bindings/usb/renesas_usbhs.txt new file +1 -0
+  - enable-gpio: A gpio specifier to check GPIO determining if USB function

Should also be prefixed with "renesas,".

I will fix this name to "renesas,enable-gpio".

Best regards,
Yoshihiro Shimoda

Quoted 4 lines

[...]

WBR, Sergei