Re: [PATCH v3 2/9] staging: dwc2: update DT binding to add generic clock/phy properties

2013-11-27 Thread Matt Porter
On Tue, Nov 26, 2013 at 03:49:30PM +0530, Kishon Vijay Abraham I wrote:
 On Monday 25 November 2013 11:46 PM, Matt Porter wrote:
  dwc2/s3c-hsotg require a single clock to be specified and optionally
  a generic phy. On the s3c-hsotg driver old style USB phy support is
  present as a fallback so the generic phy properties are optional.
  
  Signed-off-by: Matt Porter matt.por...@linaro.org
  ---
   Documentation/devicetree/bindings/staging/dwc2.txt | 10 ++
   1 file changed, 10 insertions(+)
  
  diff --git a/Documentation/devicetree/bindings/staging/dwc2.txt 
  b/Documentation/devicetree/bindings/staging/dwc2.txt
  index 1a1b7cf..b8b42b6 100644
  --- a/Documentation/devicetree/bindings/staging/dwc2.txt
  +++ b/Documentation/devicetree/bindings/staging/dwc2.txt
  @@ -5,6 +5,12 @@ Required properties:
   - compatible : snps,dwc2
   - reg : Should contain 1 register range (address and length)
   - interrupts : Should contain 1 interrupt
  +- clocks: clock provider specifier
  +- clock-names: shall be otg
  +- phys: phy provider specifier
  +- phy-names: shall be device
 
 lets mention in the Documentation too that the phy properties are optional.
 apart from that..

Good point, will do so.

 Acked-by: Kishon Vijay Abraham I kis...@ti.com
  +Refer to clk/clock-bindings.txt for generic clock consumer properties
  +Refer to phy/phy-bindings.txt for generic phy consumer properties
   
   Example:
   
  @@ -12,4 +18,8 @@ Example:
   compatible = ralink,rt3050-usb, snps,dwc2;
   reg = 0x101c 4;
   interrupts = 18;
  +   clocks = usb_otg_ahb_clk;
  +   clock-names = otg;
  +   phys = usbphy;
  +   phy-names = device;
   };
  
 
--
To unsubscribe from this list: send the line unsubscribe linux-usb in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v3 2/9] staging: dwc2: update DT binding to add generic clock/phy properties

2013-11-26 Thread Kishon Vijay Abraham I
On Monday 25 November 2013 11:46 PM, Matt Porter wrote:
 dwc2/s3c-hsotg require a single clock to be specified and optionally
 a generic phy. On the s3c-hsotg driver old style USB phy support is
 present as a fallback so the generic phy properties are optional.
 
 Signed-off-by: Matt Porter matt.por...@linaro.org
 ---
  Documentation/devicetree/bindings/staging/dwc2.txt | 10 ++
  1 file changed, 10 insertions(+)
 
 diff --git a/Documentation/devicetree/bindings/staging/dwc2.txt 
 b/Documentation/devicetree/bindings/staging/dwc2.txt
 index 1a1b7cf..b8b42b6 100644
 --- a/Documentation/devicetree/bindings/staging/dwc2.txt
 +++ b/Documentation/devicetree/bindings/staging/dwc2.txt
 @@ -5,6 +5,12 @@ Required properties:
  - compatible : snps,dwc2
  - reg : Should contain 1 register range (address and length)
  - interrupts : Should contain 1 interrupt
 +- clocks: clock provider specifier
 +- clock-names: shall be otg
 +- phys: phy provider specifier
 +- phy-names: shall be device

lets mention in the Documentation too that the phy properties are optional.
apart from that..
Acked-by: Kishon Vijay Abraham I kis...@ti.com
 +Refer to clk/clock-bindings.txt for generic clock consumer properties
 +Refer to phy/phy-bindings.txt for generic phy consumer properties
  
  Example:
  
 @@ -12,4 +18,8 @@ Example:
  compatible = ralink,rt3050-usb, snps,dwc2;
  reg = 0x101c 4;
  interrupts = 18;
 + clocks = usb_otg_ahb_clk;
 + clock-names = otg;
 + phys = usbphy;
 + phy-names = device;
  };
 

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


[PATCH v3 2/9] staging: dwc2: update DT binding to add generic clock/phy properties

2013-11-25 Thread Matt Porter
dwc2/s3c-hsotg require a single clock to be specified and optionally
a generic phy. On the s3c-hsotg driver old style USB phy support is
present as a fallback so the generic phy properties are optional.

Signed-off-by: Matt Porter matt.por...@linaro.org
---
 Documentation/devicetree/bindings/staging/dwc2.txt | 10 ++
 1 file changed, 10 insertions(+)

diff --git a/Documentation/devicetree/bindings/staging/dwc2.txt 
b/Documentation/devicetree/bindings/staging/dwc2.txt
index 1a1b7cf..b8b42b6 100644
--- a/Documentation/devicetree/bindings/staging/dwc2.txt
+++ b/Documentation/devicetree/bindings/staging/dwc2.txt
@@ -5,6 +5,12 @@ Required properties:
 - compatible : snps,dwc2
 - reg : Should contain 1 register range (address and length)
 - interrupts : Should contain 1 interrupt
+- clocks: clock provider specifier
+- clock-names: shall be otg
+- phys: phy provider specifier
+- phy-names: shall be device
+Refer to clk/clock-bindings.txt for generic clock consumer properties
+Refer to phy/phy-bindings.txt for generic phy consumer properties
 
 Example:
 
@@ -12,4 +18,8 @@ Example:
 compatible = ralink,rt3050-usb, snps,dwc2;
 reg = 0x101c 4;
 interrupts = 18;
+   clocks = usb_otg_ahb_clk;
+   clock-names = otg;
+   phys = usbphy;
+   phy-names = device;
 };
-- 
1.8.4

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