Re: duplicate udc driver for managing tdi core ?

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

 



On 2/15/2011 11:25 PM, Matthieu CASTET wrote:
> Greg KH a écrit :
>> On Sat, Feb 12, 2011 at 03:05:21PM +0100, matthieu castet wrote:
>>> Hi,
>>>
>>> grepping for tdi (arc/chipidea/mips otg ehci like controller)
>>> registers in driver/usb/gadget show interesting things : it seems we
>>> have 4 drivers for handling the same controller.
>>> - fsl_udc_core [1]
>>> - ci13xxx_udc [2]
>>> - mv_udc [3]
>>> - langwell [4]
>>>
>>> fsl is the historic driver (freesale and tegra will be using it).
>>> ci13xxx was added by mips and msm chip use it
>>> mv_udc/langwell is newer.
>>>
>>> I didn't look at the driver in details, there may be tiny difference
>>> between all this controllers.
>>>
>>> Shouldn't we try to merge them and rename to a clear name in order
>>> people don't rewrite again the same driver ?
>>
>> Sure, if you think it is possible, please do so.
>>
> But without help of the current maintainers it will be difficult. I have 
> no way to test on their platform.
> 
I can help you in testing it on MSM
> Also how to select which driver should be the base of the merge :
> - fsl_udc_core is the historic driver and seems to have tested pretty 
> well  (and was used on big endian arch)
> - langwell_udc seems the most active one.
> 

ci13xxx_udc is independent of platform/pci bus. We can hook up drivers
easily with this driver. Please see drivers/usb/gadget/ci13xxx_msm.c. It
can also interact with OTG. We have quirks/events in place for SoC
specific workarounds.

Thanks,
Pavan

-- 
Sent by a consultant of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.
--
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