Re: [RFC PATCH] usb-serial/option: modem port types

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

 



On Mon, 2009-03-09 at 19:38 +0100, Matthias Urlichs wrote:
> Hi,
> 
> oliver@xxxxxxxxxx:
> > It seems to me most of this belongs into the generic layer.
> 
> I agree.
> 
> > Also, using strings here really is arcane, as this will be parsed by
> > software.
> > 
> Both numbers and strings are magic, and the computational difference
> between atoi() and strcmp() is negligible.
> 
> One comment about the implementation: Please use a table-driven
> approach. We need that for some other enhancements that are planned for
> the option driver anyway (for instance: URB sizes need to be adapted to
> the device).

You mean like adding something to the usb device table driver data
pointer?

Dan

--
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