Dear Chen Peter-B29397,

quoted

V7: * Pass only necessary data to the drivers registered by imx-otg

      (memory, irq, pointer to imx-otg device). This should fix issue

pointed

      out by Lothar Wassmann.
    
    * Have single IRQ handler in imx-otg, which then calls host/gadget

IRQ

      handlers only if they're registered via imx_otg_set_irq_handler()

call,

      depending on the current state of the OTG.

Hi Marek,

If the host and device interrupt handler are called by otg interrupt
handler, then, the interrupt handlers are registered by host/device will
be called twice as they are also registered by host/device themselves.

No they won't, I don't register them in ehci-mxs or ci13xxx-mxs anymore.

Besides, what kinds of function do you want to add at this patchset?

Device and Host mode, separate so far, configurable by platform data. I'm also 
looking into the OTG, but let's get this basic stuff right and mainline first.

device-only?host-only? basic otg (id switch)? power management (As I have
seen you add wakeup interrupt handler)?

There is a stub wakeup handler right now, we don't do powermanagement yet.

Peter, thanks for reviewing these :)

Best regards,
Marek Vasut