Hi, (you did not add Greg in Cc, he's the USB maintainer. You also didn't Cc myself as maintainer of the gadget framework) On Tue, Aug 30, 2011 at 05:27:47PM +0530, Pratyush Anand wrote: > These patches are based on:http://patchwork.ozlabs.org/patch/89560/ > After not getting any reply from developers, I started to do > modifications for my platform (SPEAr1340). > I have done modifications in such a way that all the code in > driver/usb/dwc/ would be platform independent. > I have tested this code for host/device/dma/slave mode. > My fifo configuration is dedicated and dynamic. Looking at that driver code, I believe this is pretty much Synopsys reference code with a few tweaks. If you want some tip, don't do that. The reference driver, while functional, has lots of unnecessary layering which should be avoided. See how we wrote the DWC3 driver. It's very small code which does the necessary. It's now functionally complete, from now on we only need to improve on performance (for example starting transfers on Transfer Complete event if we already have requests queued). Nevertheless, the driver is small and has been tested with all transfer types. If you really want to use this driver as is, that's your call. But I would suggest to Greg that he only takes this in if there is someone dedicated to maintain it. It's a really big codebase and whenever I change something on the gadget framework I will have to patch this beast too. If there's no maintainer assigned to it, who's willing to put effort in improving this driver in the long run, I think this will become yet another burden on the community. Still, it's your call. -- balbi
Attachment:
signature.asc
Description: Digital signature
