On Fri, Jun 02, 2006 at 12:03:06AM -0300, Luiz Fernando N.Capitulino wrote: > > Hi folks. > > This patch series is my first attempt to port the USB-Serial layer to the > Serial Core API. Currently USB-Serial uses the TTY layer directly, duplicating > code and solutions from the Serial Core implementation. > > The final (ported) USB-Serial code is simpler and cleaner. Now I'd like to > know > whether I'm doing it right or not. > > Note that this is a work in progress though. I've only ported the USB-Serial > core and one of its drivers, the pl2303 one. > > Most of my questions and design decisions are adressed in the patches, please > refer to them for details.
Nice first cut at this. But please try to also convert 2 other drivers at the same time to make sure that the model is right. I'd suggest the io_edgeport and the funsoft drivers. io_edgeport because it is very complex in that it doesn't share a single bulk in/out pair for every port, but multiplexes them all through one pipe. And funsoft because we want to still be able to write usb-serial drivers that are this simple. I agree with most of Pete's comments and like the general idea of moving the usb-serial core to be more like libata (provider of helper functions, not getting in the middle of everything). I'll wait for your next cut to provide specific code comments. thanks, greg k-h _______________________________________________ linux-usb-devel@lists.sourceforge.net To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
