Re: [PATCH] USB: OHCI: make ohci-da8xx a separate driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 7/2/2013 10:50 PM, Kevin Hilman wrote:
> 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.

Okay, I haven't seen the patch as well, but I agree with Kevin that
EXPORT_SYMBOL from platform code is a bad idea and wont help the
multi-platform build.

Right clean-up for this most probably requires creation of a PHY driver
to handle the USB 2.0 and USB 1.1 phy specifics on this chip. Its best
to start a mail thread on USB list for guidance. You can keep me in loop
too.

Thanks,
Sekhar
--
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




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux