usb/148189: [usbdevs] [usb8] [patch] provide descriptive string for NEC EHCI 7-port USB hub

[Eugene Grosbein]

The following reply was made to PR usb/148189; it has been noted by GNATS.

From: Eugene Grosbein <eugen at grosbein.pp.ru>
To: ticso at cicely.de
Cc: Bernd Walter <ticso at cicely7.cicely.de>, FreeBSD-gnats-submit at freebsd.org
Subject: Re: usb/148189: [usbdevs] [usb8] [patch] provide descriptive string
 for NEC EHCI 7-port USB hub
Date: Tue, 29 Jun 2010 12:46:11 +0700

 On 28.06.2010 18:19, Bernd Walter wrote:
 
 >>> Description:
 >> 	Nameless NEC USB 2.0 Hub found in some monitors (f.e. my BenQ M2200HD)
 >> 	does not provide descriptive string:
 >>
 >> ugen4.2: <vendor 0x0409> at usbus4
 >> uhub5: <vendor 0x0409 product 0x005a, class 9/0, rev 2.00/1.00, addr 2> on usbus4
 >> uhub5: 4 ports with 4 removable, self powered
 >>
 >>> How-To-Repeat:
 >> 	Use USB-enabled devices with this USB controller on board.
 >>
 >>> Fix:
 >>
 >> The patch provides description to kernels compiled with "options USB_VERBOSE"
 >>
 >> --- sys/dev/usb/usbdevs.orig	2010-06-23 00:31:04.000000000 +0700
 >> +++ sys/dev/usb/usbdevs	2010-06-28 02:22:20.000000000 +0700
 >> @@ -2165,6 +2165,7 @@
 >>  product NATIONAL BEARPAW2400	0x1001	BearPaw 2400
 >>  
 >>  /* NEC products */
 >> +product NEC NUB_005A		0x005a	USB 2.0 4-Port Hub
 >>  product NEC HUB			0x55aa	hub
 >>  product NEC HUB_B		0x55ab	hub
 > 
 > Is this 7-port as in the subject, or 4-port as in the patch?
 
 Sorry, subject has mistake, it's 4-port as in patch and
 in quoted kernel log.
 
 > My 7-port NEC says:
 > ugen3.8: <product 0x0050 vendor 0x0409> at usbus3, cfg=0 md=HOST spd=HIGH (480Mbps) pwr=SAVE
 
 It would be nice to add another line then:
 
 product NEC NUB_0050		0x0050	USB 2.0 7-Port Hub