Re: [PATCH] phy-rcar-gen2-usb: add device tree support

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

 




Hello.

On 02/27/2014 06:56 PM, Mark Rutland wrote:

Add support of the device tree probing for the Renesas R-Car generation 2 SoCs
documenting the device tree binding as necessary.

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

---
This patch is against the 'next' branch of Felipe Balbi's 'usb.git' repo.

  Documentation/devicetree/bindings/usb/rcar-gen2-phy.txt |   29 +++++++
  drivers/usb/phy/phy-rcar-gen2-usb.c                     |   64 ++++++++++++++--
  2 files changed, 85 insertions(+), 8 deletions(-)

Index: usb/Documentation/devicetree/bindings/usb/rcar-gen2-phy.txt
===================================================================
--- /dev/null
+++ usb/Documentation/devicetree/bindings/usb/rcar-gen2-phy.txt
@@ -0,0 +1,29 @@
+* Renesas R-Car generation 2 USB PHY
+
+This file provides information on what the device node for the R-Car generation
+2 USB PHY contains.
+
+Required properties:
+- compatible: "renesas,usb-phy-r8a7790" if the device is a part of R8A7790 SoC.
+	      "renesas,usb-phy-r8a7791" if the device is a part of R8A7791 SoC.

Is the r8a7791's USB PHY known to be different to that of the r8a7790?

Not at all, according to the preliminary manuals. The problem is the SH-Mobile maintainers don't trust the manuals (translated from Japanese, they are sometimes indeed of a questionable quality).

If this is just to possibly handle the two differently in future, why
not have "renesas,usb-phy-r8a7790" as a fallback in the compatible list?
That was you only need it in the driver for now.

   The SH-Mobile maintainers are against that approach.

+- reg: offset and length of the register block.
+- clocks: clock phandle and specifier pair.
+- clock-names: string, clock input name, must be "usbhs".
+
+Optional properties:
+- renesas,channel0-pci: boolean, specify when USB channel 0 should be connected
+			to PCI EHCI/OHCI; otherwise, it will be connected to the
+			USBHS controller.
+- renesas,channel2-pci: boolean, specify when USB channel 2 should be connected
+			to PCI EHCI/OHCI; otherwise, it will be connected to the
+			USBSS controller (xHCI).

When would you want this to connect to PCI, and when would you not? Why
is this not a run-time decision?

I think it's mostly connector type thing: A (host) vs B (device), USB 2.0 vs 3.0. With B connector you certainly want USBHS controller which is includes the function controller (testing has shown that it also has the host controller but the manuals don't quite say that). With A connector, user would probably prefer PCI EHCI/OHCI, unless you have a USB 3.0 port in which case the preference would probably be xHCI. So it appears to be the board specific thing. Some runtime switching (e.g. downgrading from xHCI to PCI EHCI/OHCI) might be considered too but I don't quite have a picture how that would work yet.

+
+Example (Lager board):
+
+	usb-phy@e6590100 {
+		compatible = "renesas,usb-phy-r8a7790";
+		reg = <0 0xe6590100 0 0x100>;
+		clocks = <&mstp7_clks R8A7790_CLK_HSUSB>;
+		clock-names = "usbhs";
+		renesas,channel2-pci;
+	};

We're not using the generic phy bindings?

   It's not a driver/phy/ but driver/usb/phy/ code, so I guess we're not.

How is the linkage to the host controller expressed?

Looking at Documentation/devicetree/bindings/usb/, it's via "usb-phy" property in the host device nodes.

[...]

-	clk = devm_clk_get(dev, "usbhs");
+	if (np)
+		clk = of_clk_get_by_name(np, "usbhs");
+	else
+		clk = clk_get(dev, "usbhs");

Doesn't clk_get (and hence devm_clk_get) call of_clk_get_by_name?

   Indeed, stupid me. I just didn't expect it, so didn't even look there.

Cheers,
Mark.

WBR, Sergei


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