From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752404AbbCZAE0 (ORCPT ); Wed, 25 Mar 2015 20:04:26 -0400 Received: from mail-gw1-out.broadcom.com ([216.31.210.62]:34265 "EHLO mail-gw1-out.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751500AbbCZAEY (ORCPT ); Wed, 25 Mar 2015 20:04:24 -0400 X-IronPort-AV: E=Sophos;i="5.11,468,1422950400"; d="scan'208";a="60600633" Message-ID: <55134D29.80708@broadcom.com> Date: Wed, 25 Mar 2015 17:04:57 -0700 From: Arun Ramamurthy User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: Kishon Vijay Abraham I , Rob Herring , Pawel Moll , Mark Rutland , "Ian Campbell" , Kumar Gala , Arnd Bergmann , CC: , , , Dmitry Torokhov , Anatol Pomazau , Jonathan Richardson , Scott Branden , Ray Jui Subject: Re: [PATCH v1 2/3] Phy: DT binding documentation for Broadcom Cygnus USB PHY driver References: <1426886727-537-1-git-send-email-arun.ramamurthy@broadcom.com> <1426886727-537-3-git-send-email-arun.ramamurthy@broadcom.com> <551333DA.6080700@ti.com> In-Reply-To: <551333DA.6080700@ti.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 15-03-25 03:16 PM, Kishon Vijay Abraham I wrote: > Hi, > > On Saturday 21 March 2015 02:55 AM, Arun Ramamurthy wrote: >> Broadcom's Cygnus chip has a USB 2.0 host controller connected to >> three separate phys. One of the phs (port 2) is also connectd to >> a usb 2.0 device controller >> >> Reviewed-by: Ray Jui >> Reviewed-by: Scott Branden >> Signed-off-by: Arun Ramamurthy >> >> --- >> .../bindings/phy/brcm,cygnus-usb-phy.txt | 65 ++++++++++++++++++++++ >> 1 file changed, 65 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/phy/brcm,cygnus-usb-phy.txt >> >> diff --git a/Documentation/devicetree/bindings/phy/brcm,cygnus-usb-phy.txt b/Documentation/devicetree/bindings/phy/brcm,cygnus-usb-phy.txt >> new file mode 100644 >> index 0000000..002bd59 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/phy/brcm,cygnus-usb-phy.txt >> @@ -0,0 +1,65 @@ >> +BROADCOM CYGNUS USB PHY >> + >> +Required Properties: >> + - compatible: brcm,cygnus-usb-phy >> + - reg : usbphy_regs - Base address of phy registers >> + usb2h_idm_regs - Base address of host idm registers >> + usb2d_idm_regs - Base address of device idm registers > > where is #phy-cells documented? I dont follow, isnt phy-cells a standard binding, what documentation is required? >> +The node that uses the phy must provide one integers, 0 for device and 1 for host > >> + >> +NOTE: port 0 and port 1 are host only and port 2 can be configured for host or device. >> + >> +Example of phy : >> + usbphy0: usbphy@0x0301c000 { >> + compatible = "brcm,cygnus-usb-phy"; >> + reg = <0x0301c000 0x2000>, >> + <0x18115000 0x1000>, >> + <0x18111000 0x1000>; >> + status = "okay"; >> + >> + #address-cells = <1>; >> + #size-cells = <0>; >> + usbphy0_0: usbphy0@0 { >> + #phy-cells = <1>; >> + reg = <0>; >> + status = "okay"; >> + phy-supply = <&vbus_p0>; >> + }; >> + >> + usbphy0_1: usbphy0@1 { >> + #phy-cells = <1>; >> + reg = <1>; >> + status = "okay"; >> + }; >> + >> + usbphy0_2: usbphy0@2 { >> + #phy-cells = <1>; >> + reg = <2>; >> + status = "okay"; >> + phy-supply = <&vbus_p2>; >> + }; >> + }; >> + >> +Example of node using the phy: >> + >> + /* This nodes declares all three ports as host */ >> + >> + ehci0: usb@0x18048000 { >> + compatible = "generic-ehci"; >> + reg = <0x18048000 0x100>; >> + interrupts = ; >> + phys = <&usbphy0_0 1 &usbphy0_1 1 &usbphy0_2 1>; >> + phy-names = "usb","usb","usb"; > > is it on purpose you use the same name for phy-names? it is wrong though. Kishon, I did use the same names on purpose. The phy-names are actually irrelevant because I used the new api I created devm_of_phy_get_by_index. I actually wasnt sure if should take out the phy-name field altogether or leave it as phy-names = "usb" for compatibility with other bindings. What are your thoughts? >> + status = "okay"; >> + }; >> + >> + /* This node declares port 2 phy >> + and configures it for device */ > > please use standard multi-line comment format. > Ok will do. > Thanks > Kishon >