Re: [PATCH] usb: add sysfs configuration interface for CP2101

From: Dirk Eibach
Date: Tue Mar 04 2008 - 02:03:36 EST


greg@xxxxxxxxx wrote:
> On Mon, Mar 03, 2008 at 10:07:41AM -0500, Alan Stern wrote:
>> On Mon, 3 Mar 2008, Dirk Eibach wrote:
>>
>>> From: Dirk Eibach <eibach@xxxxxxxx>
>>>
>>> The usb configuration data for the Silabs CP2101 usb to uart bridge
>>> controller can be customized:
>>> - Vendor ID
>>> - Product ID
>>> - Power Descriptor
>>> - Release Number
>>> - Serial Number
>>> - Product Description String
>>>
>>> Silabs provides a windows-only tool to do that.
>>> Since we use linux-only machines in our production-environment, we have no
>>> proper way to customize the chips for our purpose.
>>> So I added sysfs configuration support to the linux driver.
>> Surely this sort of thing is better done in userspace, with libusb.
>
> I agree, any reason why you have not tried that? Adding files like
> "vendor_id" to an interface directory in sysfs could cause a lot of
> people to get very confused, very quickly :)

Greg, any reason why you didn't suggest using libusb in your first mail ;) ?

I have not used libusb yet, but looking at the api it seems to be just what I need. I had heard about libusb before, but never really had a look at it. So thanks for your advice, Alan. I will implement this stuff in userspace.

On the other hand it would a little bit sad, if Silabs windows tool would stay the only way to setup for all other people. I cannot see how I could spread the userspace tool the way it would be spread if it was in the kernel.
Maybe someone has an idea on that.

So, sorry for the noise guys.

Cheers
Dirk



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/