[PATCH] USB: OHCI: make ohci-da8xx a separate driver
khilman at linaro.org
Tue Jul 2 13:20:00 EDT 2013
On 07/02/2013 08:14 AM, Manjunath Goudar wrote:
> On 2 July 2013 20:20, Sergei Shtylyov
> <sergei.shtylyov at cogentembedded.com
> <mailto:sergei.shtylyov at cogentembedded.com>> wrote:
> On 02-07-2013 15:36, Manjunath Goudar wrote:
> Separate the Davinci OHCI 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.
> One preexisting error:
> "da8xx_syscfg0_base" [drivers/usb/host/ohci-da8xx.__ko] undefined!
> Fixed eventually using below modification:
> added EXPORT_SYMBOL_GPL(da8xx___syscfg0_base) in
> And you managed to get this fix into the DaVinci tree? I tried it
> long ago and it was refused by then DaVinci maintainer Kevin Hilman.
> Yes I saw your patch that is what I mentioned in patch description.
> We will wait for DaVinci maintainer response,what he will suggest.
Note that Sekhar Nori (now Cc'd) is the primary maintainer of davinci,
and I'll defer the final decision to him. However, the mach-davinci
change is not in this patch, so I'm not sure exactly how it relates
here, since that problem exists independently of this patch.
That being said, I will NAK the above EXPORT_SYMBOL change in
mach-davinci code because passing data between platform code and drivers
via global variables is still a bad idea. Some helper accessor
functions will need to be created to abstract those low-level accesses.
> Signed-off-by: Manjunath Goudar <manjunath.goudar at linaro.org
> <mailto:manjunath.goudar at linaro.org>>
> Cc: Arnd Bergmann <arnd at arndb.de <mailto:arnd at arndb.de>>
> Cc: Alan Stern <stern at rowland.harvard.edu
> <mailto:stern at rowland.harvard.edu>>
> Cc: Sergei Shtylyov <sshtylyov at ru.mvista.com
> <mailto:sshtylyov at ru.mvista.com>>
> Cc: Kevin Hilman <kjh at hilman.org <mailto:kjh at hilman.org>>
Also, please khilman at linaro.org instead of my personal address. Thanks,
More information about the linux-arm-kernel