Re: [PATCH v2] usb: musb: dsps: dt binding - add resources, example

2012-11-07 Thread Benoit Cousson
Hi Felipe,

On 11/06/2012 07:22 PM, Felipe Balbi wrote:
 Hi,
 
 On Tue, Nov 06, 2012 at 05:58:57PM +0100, Benoit Cousson wrote:
 On 11/06/2012 05:44 PM, Felipe Balbi wrote:
 Hi,

 On Tue, Nov 06, 2012 at 07:26:06PM +0530, Afzal Mohammed wrote:
 OMAP2+ family of devices are now obtaining resources via DT, earlier
 it was obtained from hwmod. Update binding document accrodingly, while
 at it add example.

 Signed-off-by: Afzal Mohammed af...@ti.com

 this looks fine to me:

 Reviewed-by: Felipe Balbi ba...@ti.com

 Benoit, do you take Documentation patches too ?

 Well, ideally it should go with the driver change. But if this is a
 simple change related to generic attribute I can take it.
 
 ok, cool. Please take this via your tree.

Done. I've just applied it in my for_3.8/dts_part2 branch.

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


[PATCH v2] usb: musb: dsps: dt binding - add resources, example

2012-11-06 Thread Afzal Mohammed
OMAP2+ family of devices are now obtaining resources via DT, earlier
it was obtained from hwmod. Update binding document accrodingly, while
at it add example.

Signed-off-by: Afzal Mohammed af...@ti.com
---

v2: node name changed to usb

 .../devicetree/bindings/usb/am33xx-usb.txt  | 21 +
 1 file changed, 21 insertions(+)

diff --git a/Documentation/devicetree/bindings/usb/am33xx-usb.txt 
b/Documentation/devicetree/bindings/usb/am33xx-usb.txt
index a922505..ea840f7 100644
--- a/Documentation/devicetree/bindings/usb/am33xx-usb.txt
+++ b/Documentation/devicetree/bindings/usb/am33xx-usb.txt
@@ -1,5 +1,7 @@
 AM33XX MUSB GLUE
  - compatible : Should be ti,musb-am33xx
+ - reg : offset and length of register sets, first usbss, then for musb 
instances
+ - interrupts : usbss, musb instance interrupts in order
  - ti,hwmods : must be usb_otg_hs
  - multipoint : Should be 1 indicating the musb controller supports
multipoint. This is a MUSB configuration-specific setting.
@@ -12,3 +14,22 @@ AM33XX MUSB GLUE
represents PERIPHERAL.
  - power : Should be 250. This signifies the controller can supply upto
500mA when operating in host mode.
+
+Example:
+
+usb@4740  {
+   compatible = ti,musb-am33xx;
+   reg = 0x4740 0x1000/* usbss */
+  0x47401000 0x800 /* musb instance 0 */
+  0x47401800 0x800;   /* musb instance 1 */
+   interrupts = 17/* usbss */
+ 18/* musb instance 0 */
+ 19;  /* musb instance 1 */
+   multipoint = 1;
+   num-eps = 16;
+   ram-bits = 12;
+   port0-mode = 3;
+   port1-mode = 3;
+   power = 250;
+   ti,hwmods = usb_otg_hs;
+};
-- 
1.7.12

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


Re: [PATCH v2] usb: musb: dsps: dt binding - add resources, example

2012-11-06 Thread Felipe Balbi
Hi,

On Tue, Nov 06, 2012 at 07:26:06PM +0530, Afzal Mohammed wrote:
 OMAP2+ family of devices are now obtaining resources via DT, earlier
 it was obtained from hwmod. Update binding document accrodingly, while
 at it add example.
 
 Signed-off-by: Afzal Mohammed af...@ti.com

this looks fine to me:

Reviewed-by: Felipe Balbi ba...@ti.com

Benoit, do you take Documentation patches too ?

 ---
 
 v2: node name changed to usb
 
  .../devicetree/bindings/usb/am33xx-usb.txt  | 21 
 +
  1 file changed, 21 insertions(+)
 
 diff --git a/Documentation/devicetree/bindings/usb/am33xx-usb.txt 
 b/Documentation/devicetree/bindings/usb/am33xx-usb.txt
 index a922505..ea840f7 100644
 --- a/Documentation/devicetree/bindings/usb/am33xx-usb.txt
 +++ b/Documentation/devicetree/bindings/usb/am33xx-usb.txt
 @@ -1,5 +1,7 @@
  AM33XX MUSB GLUE
   - compatible : Should be ti,musb-am33xx
 + - reg : offset and length of register sets, first usbss, then for musb 
 instances
 + - interrupts : usbss, musb instance interrupts in order
   - ti,hwmods : must be usb_otg_hs
   - multipoint : Should be 1 indicating the musb controller supports
 multipoint. This is a MUSB configuration-specific setting.
 @@ -12,3 +14,22 @@ AM33XX MUSB GLUE
 represents PERIPHERAL.
   - power : Should be 250. This signifies the controller can supply upto
 500mA when operating in host mode.
 +
 +Example:
 +
 +usb@4740  {
 + compatible = ti,musb-am33xx;
 + reg = 0x4740 0x1000/* usbss */
 +0x47401000 0x800 /* musb instance 0 */
 +0x47401800 0x800;   /* musb instance 1 */
 + interrupts = 17/* usbss */
 +   18/* musb instance 0 */
 +   19;  /* musb instance 1 */
 + multipoint = 1;
 + num-eps = 16;
 + ram-bits = 12;
 + port0-mode = 3;
 + port1-mode = 3;
 + power = 250;
 + ti,hwmods = usb_otg_hs;
 +};
 -- 
 1.7.12
 

-- 
balbi


signature.asc
Description: Digital signature


Re: [PATCH v2] usb: musb: dsps: dt binding - add resources, example

2012-11-06 Thread Benoit Cousson
On 11/06/2012 05:44 PM, Felipe Balbi wrote:
 Hi,
 
 On Tue, Nov 06, 2012 at 07:26:06PM +0530, Afzal Mohammed wrote:
 OMAP2+ family of devices are now obtaining resources via DT, earlier
 it was obtained from hwmod. Update binding document accrodingly, while
 at it add example.

 Signed-off-by: Afzal Mohammed af...@ti.com
 
 this looks fine to me:
 
 Reviewed-by: Felipe Balbi ba...@ti.com
 
 Benoit, do you take Documentation patches too ?

Well, ideally it should go with the driver change. But if this is a
simple change related to generic attribute I can take it.

Regards,
Benoit

 
 ---

 v2: node name changed to usb

  .../devicetree/bindings/usb/am33xx-usb.txt  | 21 
 +
  1 file changed, 21 insertions(+)

 diff --git a/Documentation/devicetree/bindings/usb/am33xx-usb.txt 
 b/Documentation/devicetree/bindings/usb/am33xx-usb.txt
 index a922505..ea840f7 100644
 --- a/Documentation/devicetree/bindings/usb/am33xx-usb.txt
 +++ b/Documentation/devicetree/bindings/usb/am33xx-usb.txt
 @@ -1,5 +1,7 @@
  AM33XX MUSB GLUE
   - compatible : Should be ti,musb-am33xx
 + - reg : offset and length of register sets, first usbss, then for musb 
 instances
 + - interrupts : usbss, musb instance interrupts in order
   - ti,hwmods : must be usb_otg_hs
   - multipoint : Should be 1 indicating the musb controller supports
 multipoint. This is a MUSB configuration-specific setting.
 @@ -12,3 +14,22 @@ AM33XX MUSB GLUE
 represents PERIPHERAL.
   - power : Should be 250. This signifies the controller can supply upto
 500mA when operating in host mode.
 +
 +Example:
 +
 +usb@4740  {
 +compatible = ti,musb-am33xx;
 +reg = 0x4740 0x1000/* usbss */
 +   0x47401000 0x800 /* musb instance 0 */
 +   0x47401800 0x800;   /* musb instance 1 */
 +interrupts = 17/* usbss */
 +  18/* musb instance 0 */
 +  19;  /* musb instance 1 */
 +multipoint = 1;
 +num-eps = 16;
 +ram-bits = 12;
 +port0-mode = 3;
 +port1-mode = 3;
 +power = 250;
 +ti,hwmods = usb_otg_hs;
 +};
 -- 
 1.7.12

 

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


Re: [PATCH v2] usb: musb: dsps: dt binding - add resources, example

2012-11-06 Thread Felipe Balbi
Hi,

On Tue, Nov 06, 2012 at 05:58:57PM +0100, Benoit Cousson wrote:
 On 11/06/2012 05:44 PM, Felipe Balbi wrote:
  Hi,
  
  On Tue, Nov 06, 2012 at 07:26:06PM +0530, Afzal Mohammed wrote:
  OMAP2+ family of devices are now obtaining resources via DT, earlier
  it was obtained from hwmod. Update binding document accrodingly, while
  at it add example.
 
  Signed-off-by: Afzal Mohammed af...@ti.com
  
  this looks fine to me:
  
  Reviewed-by: Felipe Balbi ba...@ti.com
  
  Benoit, do you take Documentation patches too ?
 
 Well, ideally it should go with the driver change. But if this is a
 simple change related to generic attribute I can take it.

ok, cool. Please take this via your tree.

thanks

-- 
balbi


signature.asc
Description: Digital signature