Hello, On Tuesday, June 21, 2011 2:36 PM Praveen Paneri wrote: > On Tue, Jun 21, 2011 at 4:21 PM, PRAVEEN PANERI <p.paneri@xxxxxxxxxxx> > wrote: > Hi, > > > > Hello, > > > > On Tuesday, June 21, 2011 9:04 AM p.paneri@xxxxxxxxxxx wrote: > > > > > From: Praveen Paneri > > > > > > These patches modify DWC OTG driver for ARM and add it for ARM based > SoCs. > > > Currently it has been tested for Samsung's S5P6440. The development is > > > still > > > under process so these are not the best set of patches. They are just > for > > > review > > > and not for merge. Intent of these patches is to show required > > > modifications > > > to make dwc_otg patches work on ARM. > > > > What are the differences between DWC OTG hardware and OTG hardware found > > in s3c64xx/s5pc100/s5pv210 and Exynos4 SoCs series? I've did a brief look > > at register definitions and most of them looks same. Can this driver be > > merged with s3c-hsotg.c driver? > Yes they are same because they all use the same OTG IP. This driver > provides > functionalities of both host and gadget while s3c-hsotg.c is only a > gadget driver. I'm perfectly aware that s3c-hsotg is just for UDC functionality. I just wanted to ask if this driver can replace s3c-hsotg.c driver completely? > If you think this can be merged in s3c-hsotg to make that work in > smiler fashion > then go ahead but will that work on platforms other than ARM? You can always create so called variants of the driver that exports different functionality depending on the platform data the driver has been bound to. Usually it is used to distinguish different models of the hardware that has a lot in common. For example please refer to driver/media/video/s5p-fimc/fimc-core.c and fimc_driver_ids array in the bottom of the code. > > > > > Following features are tested for basic functions on SMDK6440 > > > board after modifications: > > > OTG HOST: HID and mass-storage > > > OTG DEVICE: mass-storage and adb > > > > Does it mean that HID or mass-storage (both host or gadget drivers) needs > > modifications to get them working with this DWC OTG driver? > > Why would we need any other driver if DWC OTG driver is working both > as host and gadget? You got me wrong. I wanted to ask if one needs to tweak in mass storage driver to get it working with DWC OTG driver? I'm asking because I've already seen a lot of scary code on some public Android trees, where mass storage was hacked to workaround bugs (or flaws) in the OTG driver. Have you tried your DWC OTG driver with some more demanding gadget drivers? One of such is g_multi which exports 3 usb functions (RNDIS, ACM serial and Mass Storage) in a composite device. Best regards -- Marek Szyprowski Samsung Poland R&D Center -- 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
