On Mon, Jan 28, 2008 at 05:09:30PM -0500, Alan Stern wrote:
> On Mon, 28 Jan 2008, Greg KH wrote:
> 
> > From: Greg Kroah-Hartman <[EMAIL PROTECTED]>
> > Subject: USB: use a real vendor and product id for root hubs
> > 
> > Turns out that a company is out there using the vendor id of 0x0000 in
> > the wild, so use a real vendor/product id for the root hubs.
> > 
> > Now that the Linux Foundation has a real vendor id, we use that, and the
> > first product id:
> >     0x1d6b is the vendor id of the Linux Foundation
> >     0x0001 is the product id for Linux root hubs
> > 
> > 
> > Signed-off-by: Greg Kroah-Hartman <[EMAIL PROTECTED]>
> > 
> > ---
> >  drivers/usb/core/hcd.c |    4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > --- a/drivers/usb/core/hcd.c
> > +++ b/drivers/usb/core/hcd.c
> > @@ -132,8 +132,8 @@ static const u8 usb2_rh_dev_descriptor [
> >     0x01,       /*  __u8  bDeviceProtocol; [ usb 2.0 single TT ]*/
> >     0x40,       /*  __u8  bMaxPacketSize0; 64 Bytes */
> >  
> > -   0x00, 0x00, /*  __le16 idVendor; */
> > -   0x00, 0x00, /*  __le16 idProduct; */
> > +   0x6b, 0x1d, /*  __le16 idVendor; Linux Foundation */
> > +   0x01, 0x00, /*  __le16 idProduct; device 0x0001 */
> >     KERNEL_VER, KERNEL_REL, /*  __le16 bcdDevice */
> >  
> >     0x03,       /*  __u8  iManufacturer; */
> 
> There's another descriptor structure (usb11_rh_dev_descriptor) just
> below this one which also needs to be changed.  You want to use a
> different idProduct for it, so that the two types of root hubs don't 
> appear to be the same.

Doh, good catch.  I made it so that id 0x0001 is for 1.1 hubs, and
0x00002 is for 2.0 hubs.  Might as well reserve 0x0003 for 3.0 hubs too
:)

thanks,

greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to