Re: [PATCH] AM35x: Add musb support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello.

Gupta, Ajay Kumar wrote:

+/* USB 2.0 OTG module registers */
+#define USB_REVISION_REG     0x00
+#define USB_CTRL_REG         0x04
+#define USB_STAT_REG         0x08
+#define USB_EMULATION_REG    0x0c
+/* 0x10 Reserved */
 Wait... so the mode register isn't supported? Does AM35x support any
accelerated modes?

Mode register is available though with some difference due to 15Rx/Tx
eps.
I have not added CPPI4.1 DMA  specific register here..

   Ah. Then they're not compatible with DA8xx anyway...

We could make it compatible with some change as done at,
http://arago-project.org/git/people/?p=sriram/ti-psp-omap.git;a=commitdiff;h=2ecce9529bd56c32997e0b2cbe84f7727ba5b217

I'm not sure where cppi41_dma.c gets the definitions for USB_[RT]X_MODE_REG... anyway, I'm thinking of a different solution now, i.e. moving the functions that access the implementation specific acceleration

+#define USB_END_OF_INTR_REG  0x60
  Hm, I don't see the generic RNDIS EP size registers also. So, generic
RNDIS mode isn't supported? Then cppi41_dma.c won't work with this
chip... :-/

Generic RNDIS mode is suported and CPPi4,1 is already verified to be
working on
AM3517.
   I wonder how it can work as the registers cppi41_dma.c relies upon
are not the same as in DA8xx. Did you make some changes to cppi41_dma.c?

There were only few changes to make it work. You can see the changes
in our internal release tree at,
http://arago-project.org/git/people/?p=sriram/ti-psp-omap.git;a=summary

They have all the CPPI patches based out of your latest patch. All these

  My latest was take 5, this tree has take 3 as a base.

have to be reviewed/cleaned which is not happening as the base CPPI4.1 patch
itself is pending to be merged.

Any update on CPPI4.1 driver location?

  I'm pretty sure it should be in arch/arm/common/...

 Are you still waiting for Russell's reply?

Well, yes and no. I'm not hoping on Russell's reply though but haven't found time to move the code and post the new patch...

WBR, Sergei


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

[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux