Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964939AbbGZDp7 (ORCPT ); Sat, 25 Jul 2015 23:45:59 -0400 Received: from mailgw01.mediatek.com ([218.249.47.110]:47778 "EHLO mailgw01.mediatek.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S964809AbbGZDp6 (ORCPT ); Sat, 25 Jul 2015 23:45:58 -0400 X-Listener-Flag: 11101 Message-ID: <1437882325.3040.22.camel@mhfsdcap03> Subject: Re: [PATCH v3 2/5] dt-bindings: Add a binding for Mediatek xHCI host controller From: chunfeng yun To: Mark Rutland CC: Mathias Nyman , Rob Herring , Matthias Brugger , "Felipe Balbi" , Sascha Hauer , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Roger Quadros , "linux-usb@vger.kernel.org" , "linux-mediatek@lists.infradead.org" , John Crispin , Daniel Kurtz Date: Sun, 26 Jul 2015 11:45:25 +0800 In-Reply-To: <20150722142208.GC15809@leverpostej> References: <1437573945-31586-1-git-send-email-chunfeng.yun@mediatek.com> <1437573945-31586-3-git-send-email-chunfeng.yun@mediatek.com> <20150722142208.GC15809@leverpostej> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.3-0ubuntu6 Content-Transfer-Encoding: 7bit MIME-Version: 1.0 X-MTK: N Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3764 Lines: 110 Hi, On Wed, 2015-07-22 at 15:22 +0100, Mark Rutland wrote: > On Wed, Jul 22, 2015 at 03:05:42PM +0100, Chunfeng Yun wrote: > > add a DT binding documentation of xHCI host controller for the > > MT8173 SoC from Mediatek. > > > > Signed-off-by: Chunfeng Yun > > --- > > .../devicetree/bindings/usb/mt8173-xhci.txt | 50 ++++++++++++++++++++++ > > 1 file changed, 50 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/usb/mt8173-xhci.txt > > > > diff --git a/Documentation/devicetree/bindings/usb/mt8173-xhci.txt b/Documentation/devicetree/bindings/usb/mt8173-xhci.txt > > new file mode 100644 > > index 0000000..444494d > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/usb/mt8173-xhci.txt > > @@ -0,0 +1,50 @@ > > +MT65XX xhci > > + > > +The device node for Mediatek SOC usb3.0 host controller > > + > > +Required properties: > > + - compatible : supports "mediatek,mt8173-xhci" > > + - reg : Offset and length of registers > > Your example has multiple reg entries. > > Please list what each entry is, and the order you expect them in. > Ok > > + - interrupts : Interrupt mode, number and trigger mode > > + - power-domains: to enable usb's mtcmos > > + - vusb33-supply: regulator of usb avdd3.3v > > + - clocks : must support all clocks that xhci needs > > + - clock-names: should be "sys_mac" for sys and mac clocks, and > > + "wakeup_deb_p0", "wakeup_deb_p1" for wakeup debounce control > > + clocks > > + - phys : the phys that xhci will bind, currently supports up to two > > + phys, so phy index should not greater than one. > > + - phy-names : should be "phy-X" format, X equals to 0 or 1 > > This seems somewhat pointless. > I'll describe it more exactly. > > + - usb3-lpm-capable: supports USB3 LPM > > + - mediatek,usb-wakeup: to access usb wakeup control register > > What exactly does this property imply? > There are some control registers for usb wakeup which are put in another module, here to get the node of that module, and then use regmap and syscon to operate it. > > + - mediatek,wakeup-src: 1: ip sleep wakeup mode; 2: line state wakeup > > + mode; others means don't enable wakeup source of usb > > This sounds like configuration rather than a hardware property. Why do > you think this needs to be in the DT? > Yes, it's better to put it in the DT. > > + - mediatek,u2port-num: the number should not greater than the number > > + of phys > > What exactly does this property imply? > On some platform, it only makes use of partial usb ports, so disable others to save power. > Mark. > > > + > > +Optional properties: > > + - vbus-supply : reference to the VBUS regulator; > > + > > +Example: > > +usb: usb30@11270000 { > > + compatible = "mediatek,mt8173-xhci"; > > + reg = <0 0x11270000 0 0x4000>, > > + <0 0x11280000 0 0x0800>; > > + interrupts = ; > > + power-domains = <&scpsys MT8173_POWER_DOMAIN_USB>; > > + clocks = <&topckgen CLK_TOP_USB30_SEL>, > > + <&pericfg CLK_PERI_USB0>, > > + <&pericfg CLK_PERI_USB1>; > > + clock-names = "sys_mac", > > + "wakeup_deb_p0", > > + "wakeup_deb_p1"; > > + phys = <&u3phy 0>, <&u3phy 1>; > > + phy-names = "phy-0", "phy-1"; > > + vusb33-supply = <&mt6397_vusb_reg>; > > + vbus-supply = <&usb_p1_vbus>; > > + usb3-lpm-capable; > > + mediatek,usb-wakeup = <&pericfg>; > > + mediatek,wakeup-src = <1>; > > + mediatek,u2port-num = <2>; > > + status = "okay"; > > +}; > > -- > > 1.8.1.1.dirty > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/