On 20 June 2013 09:40, Jingoo Han <[email protected]> wrote: > 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 <[email protected]> > > Cc: Arnd Bergmann <[email protected]> > > Cc: Jingoo Han <[email protected]> > > Cc: Kukjin Kim <[email protected]> > > Cc: Greg KH <[email protected]> > > Cc: Alan Stern <[email protected]> > > Cc: [email protected] > > --- > > 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 <[email protected]> > > > Also, I agree on Alan's opinion. > > +#define DRIVER_DESC "OHCI exynos driver" > Please, replace 'exynos' with 'EXYNOS'. > > Thank you very much. I fixed his comment in V3,when I am sending V3 vesrion I will add CC to Vivek Gautam.
Thanks Manjunath Goudar Best regards, > Jingoo Han > > >
_______________________________________________ linaro-dev mailing list [email protected] http://lists.linaro.org/mailman/listinfo/linaro-dev
