I've been thinkig about creating two child nodes for the independent musb
controllers on the am33. I've been thinking about the following:

diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
index 8e1248f..6aa9506 100644
--- a/arch/arm/boot/dts/am33xx.dtsi
+++ b/arch/arm/boot/dts/am33xx.dtsi
@@ -326,21 +326,78 @@
                        status = "disabled";
                };
 
                usb@47400000 {
                        compatible = "ti,musb-am33xx";
-                       reg = <0x47400000 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>;
+                       reg = <0x47400000 0x1000>;      /* usbss */
+                       interrupts = <17>;              /* usbss */
                        ti,hwmods = "usb_otg_hs";
+
+                       usb0@0x47401000 {
+                               reg = <0x47401000 0x800>;       /* musb 
instance 0 */
+                               interrupts = <18>;              /* musb 
instance 0 */
+                               multipoint = <1>;
+                               num-eps = <16>;
+                               ram-bits = <12>;
+                               port-mode = <3>;
+                               power = <250>;
+
+                               phys = <&nopphy0>;
+                               dmas = <&cppi41dma 0
+                                       &cppi41dma 1
+                                       &cppi41dma 2
+                                       &cppi41dma 3
+                                       &cppi41dma 4
+                                       &cppi41dma 32
+                                       &cppi41dma 34
+                                       &cppi41dma 36
+                                       &cppi41dma 38
+                                       &cppi41dma 40>;
+                               dma-names = "rx1", "rx2", "rx3", "rx4", "rx5",
+                                           "tx1", "tx2", "tx3", "tx4", "tx5";
+                       };
+
+                       usb0@0x47401800 {
+                               reg = <0x47401800 0x800>;       /* musb 
instance 1 */
+                               interrupts = <19>;              /* musb 
instance 1 */
+                               multipoint = <1>;
+                               num-eps = <16>;
+                               ram-bits = <12>;
+                               port-mode = <3>;
+                               power = <250>;
+
+                               phys = <&nopphy1>;
+                               dmas = <&cppi41dma 0
+                                       &cppi41dma 1
+                                       &cppi41dma 2
+                                       &cppi41dma 3
+                                       &cppi41dma 4
+                                       &cppi41dma 62
+                                       &cppi41dma 64
+                                       &cppi41dma 66
+                                       &cppi41dma 68
+                                       &cppi41dma 70>;
+                               dma-names = "rx1", "rx2", "rx3", "rx4", "rx5",
+                                           "tx1", "tx2", "tx3", "tx4", "tx5";
+                       };
+               };

Please keep in mind that his is not a proper patch it should point out
what I have in mind.
This would easy the for instance the dma channel assignment. Also, it will
move back to the port-mode property instead of portX-mode one which it has
now. This is kind of confusing since it is not a root hub or anything like
that it is a complete musb controller. Plus for the phys property could
easily take the one avaible and would not require which instance it
really it is.

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

Reply via email to