On Thursday, June 13, 2013 12:54 AM, Manjunath Goudar wrote: > > Separate the Samsung OHCI EXYNOS 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. > > V2: > -exynos_ohci_hcd structure assignment error fixed. > -Removed multiple usb_create_hcd() from prob funtion. > -platform_set_drvdata() called before exynos_ohci_phy_enable(). > -ohci_setup() removed because it is called in .reset member > of the ohci_hc_driver structure > > Signed-off-by: Manjunath Goudar <manjunath.goudar@xxxxxxxxxx> > Cc: Arnd Bergmann <arnd@xxxxxxxx> > Cc: Jingoo Han <jg1.han@xxxxxxxxxxx> > Cc: Kukjin Kim <kgene.kim@xxxxxxxxxxx> > Cc: Greg KH <greg@xxxxxxxxx> > Cc: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> > Cc: linux-usb@xxxxxxxxxxxxxxx > --- > drivers/usb/host/Kconfig | 2 +- > drivers/usb/host/Makefile | 1 + > drivers/usb/host/ohci-exynos.c | 167 +++++++++++++++++----------------------- > drivers/usb/host/ohci-hcd.c | 18 ----- > 4 files changed, 71 insertions(+), 117 deletions(-) CC'ed Vivek Gautam Hi Manjunath Goudar, It looks good. I tested this patch on Exynos4210. It works properly. I really appreciate your work. Acked-by: Jingoo Han <jg1.han@xxxxxxxxxxx> Also, I agree on Alan's opinion. > +#define DRIVER_DESC "OHCI exynos driver" Please, replace 'exynos' with 'EXYNOS'. Best regards, Jingoo Han -- 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
