On Tue, 25 Jun 2013, Manjunath Goudar wrote:

> Separate the  TI OHCI OMAP3 host controller driver from ohci-hcd
> host code so that it can be built as a separate driver module.
> This work is part of enabling multi-platform kernels on ARM;
> it would be nice to have in 3.11.
> 
> Signed-off-by: Manjunath Goudar <[email protected]>
> Cc: Anand Gadiyar <[email protected]>
> Cc: Felipe Balbi <[email protected]>
> Cc: Arnd Bergmann <[email protected]>
> Cc: Greg KH <[email protected]>
> Cc: Alan Stern <[email protected]>
> Cc: [email protected]
> 
> V2:
>  -ohci_setup() removed because it is called in .reset member
>   of the ohci_hc_driver structure.
>  -The improper multi-line commenting style written in proper way.
>   ('*' characters aligned in vertically).
> 
> V3:
>  -RemoteWakeupConnected setting has been removed.

You weren't supposed to remove it!  I told you to remove this one line:

        writel(OHCI_CTRL_RWC, &ohci->regs->control);

The preceding lines:

        /*
         * RemoteWakeupConnected has to be set explicitly before
         * calling ohci_run. The reset value of RWC is 0.
         */
        ohci->hc_control = OHCI_CTRL_RWC;

should remain.

Alan Stern

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

Reply via email to