On 07/02/2013 08:14 AM, Manjunath Goudar wrote: > > > On 2 July 2013 20:20, Sergei Shtylyov > <sergei.shtylyov@xxxxxxxxxxxxxxxxxx > <mailto:sergei.shtylyov@xxxxxxxxxxxxxxxxxx>> wrote: > > Hello. > > > 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 > arch/arm/mach-davinci/devices-__da8xx.c. > > > 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@xxxxxxxxxx > <mailto:manjunath.goudar@xxxxxxxxxx>> > Cc: Arnd Bergmann <arnd@xxxxxxxx <mailto:arnd@xxxxxxxx>> > Cc: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx > <mailto:stern@xxxxxxxxxxxxxxxxxxx>> > Cc: Sergei Shtylyov <sshtylyov@xxxxxxxxxxxxx > <mailto:sshtylyov@xxxxxxxxxxxxx>> > Cc: Kevin Hilman <kjh@xxxxxxxxxx <mailto:kjh@xxxxxxxxxx>> Also, please khilman@xxxxxxxxxx instead of my personal address. Thanks, Kevin -- 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