On Friday 07 June 2013 11:33:33 Manjunath Goudar wrote: > > + ohci_setup(hcd); > s3c2410_start_hc(dev, hcd); > > - ohci_hcd_init(hcd_to_ohci(hcd)); > - I'm not sure about this part: s3c2410_start_hc is where the clock gets enabled, presumable we are not supposed to touch the ohci registers before it gets called, so ohci_init() being called implicitly by ohci_setup() seems like a bad idea. Arnd -- 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