Bug 148189 - [usbdevs] [usb8] [patch] provide descriptive string for NEC EHCI 7-port USB hub
Summary: [usbdevs] [usb8] [patch] provide descriptive string for NEC EHCI 7-port USB hub
Status: Closed FIXED
Alias: None
Product: Base System
Classification: Unclassified
Component: usb (show other bugs)
Version: 8.1-PRERELEASE
Hardware: Any Any
: Normal Affects Only Me
Assignee: Remko Lodder
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-27 20:30 UTC by Eugene Grosbein
Modified: 2011-01-27 18:53 UTC (History)
0 users

See Also:


Attachments
file.diff (328 bytes, patch)
2010-06-27 20:30 UTC, Eugene Grosbein
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eugene Grosbein 2010-06-27 20:30:05 UTC
	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

Fix: The patch provides description to kernels compiled with "options USB_VERBOSE"
How-To-Repeat: 	Use USB-enabled devices with this USB controller on board.
Comment 1 Hans Petter Selasky 2010-06-27 22:33:50 UTC
On Sunday 27 June 2010 21:29:37 Eugene Grosbein wrote:
> Eugene Grosbein
> 

See USB P4 change ID #180276.

--HPS
Comment 2 Eugene Grosbein 2010-06-28 04:32:31 UTC
On 28.06.2010 04:33, Hans Petter Selasky wrote:
> On Sunday 27 June 2010 21:29:37 Eugene Grosbein wrote:
> 
> See USB P4 change ID #180276.

Thanks! Will wait the change hit SVN :-)

Eugene Grosbein
Comment 3 ticso 2010-06-28 12:19:39 UTC
On Mon, Jun 28, 2010 at 02:29:37AM +0700, Eugene Grosbein wrote:
> 
> >Number:         148189
> >Category:       usb
> >Synopsis:       [usbdevs] [usb8] [patch] provide descriptive string for NEC EHCI 7-port USB hub
> >Confidential:   no
> >Severity:       non-critical
> >Priority:       low
> >Responsible:    freebsd-usb
> >State:          open
> >Quarter:        
> >Keywords:       
> >Date-Required:
> >Class:          change-request
> >Submitter-Id:   current-users
> >Arrival-Date:   Sun Jun 27 19:30:05 UTC 2010
> >Closed-Date:
> >Last-Modified:
> >Originator:     Eugene Grosbein
> >Release:        FreeBSD 8.1-PRERELEASE i386
> >Organization:
> RDTC JSC
> >Environment:
> System: FreeBSD grosbein.pp.ru 8.1-PRERELEASE FreeBSD 8.1-PRERELEASE #2: Mon Jun 28 02:13:05 NOVST 2010 root@grosbein.pp.ru:/usr/local/obj/usr/local/src/sys/DADV i386
> 
> >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?
My 7-port NEC says:
ugen3.8: <product 0x0050 vendor 0x0409> at usbus3, cfg=0 md=HOST spd=HIGH (480Mbps) pwr=SAVE

Those are another 7-port, but I don't know the vendor:
ugen3.9: <USB 2.0 Hub [MTT] vendor 0x1a40> at usbus3, cfg=0 md=HOST spd=HIGH (480Mbps) pwr=SAVE
ugen3.11: <USB 2.0 Hub [MTT] vendor 0x1a40> at usbus3, cfg=0 md=HOST spd=HIGH (480Mbps) pwr=SAVE
It is a Sedna 13 port hub containing both of them.
I thought those were NEC chips too, but ID looks very different and
I can't easily open it because of the installation place.

-- 
B.Walter <bernd@bwct.de> http://www.bwct.de
Modbus/TCP Ethernet I/O Baugruppen, ARM basierte FreeBSD Rechner uvm.
Comment 4 Eugene Grosbein 2010-06-29 06:46:11 UTC
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
Comment 5 dfilter service freebsd_committer freebsd_triage 2010-07-13 21:09:23 UTC
Author: remko
Date: Tue Jul 13 20:09:14 2010
New Revision: 210018
URL: http://svn.freebsd.org/changeset/base/210018

Log:
  Add a 4 and 7 port USB hub from NEC.
  
  PR:		148189
  MFC after:	1 week

Modified:
  head/sys/dev/usb/usbdevs

Modified: head/sys/dev/usb/usbdevs
==============================================================================
--- head/sys/dev/usb/usbdevs	Tue Jul 13 19:58:06 2010	(r210017)
+++ head/sys/dev/usb/usbdevs	Tue Jul 13 20:09:14 2010	(r210018)
@@ -2160,6 +2160,8 @@ product NATIONAL BEARPAW1200	0x1000	Bear
 product NATIONAL BEARPAW2400	0x1001	BearPaw 2400
 
 /* NEC products */
+product NEC HUB_0050		0x0050	USB 2.0 7-Port Hub
+product NEC HUB_005A		0x005a	USB 2.0 4-Port Hub
 product NEC HUB			0x55aa	hub
 product NEC HUB_B		0x55ab	hub
 
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"
Comment 6 Remko Lodder freebsd_committer freebsd_triage 2010-07-13 21:09:26 UTC
Responsible Changed
From-To: freebsd-usb->remko

I'll take it.
Comment 7 Gavin Atkinson freebsd_committer freebsd_triage 2010-07-26 20:30:18 UTC
State Changed
From-To: open->patched

Mark patched, needs MFC
Comment 8 dfilter service freebsd_committer freebsd_triage 2010-11-19 01:35:21 UTC
Author: thompsa
Date: Fri Nov 19 01:35:14 2010
New Revision: 215487
URL: http://svn.freebsd.org/changeset/base/215487

Log:
  MFC r210018
  
   Add a 4 and 7 port USB hub from NEC.
  
  PR:		148189

Modified:
  stable/8/sys/dev/usb/usbdevs
Directory Properties:
  stable/8/sys/   (props changed)
  stable/8/sys/amd64/include/xen/   (props changed)
  stable/8/sys/cddl/contrib/opensolaris/   (props changed)
  stable/8/sys/contrib/dev/acpica/   (props changed)
  stable/8/sys/contrib/pf/   (props changed)
  stable/8/sys/dev/xen/xenpci/   (props changed)

Modified: stable/8/sys/dev/usb/usbdevs
==============================================================================
--- stable/8/sys/dev/usb/usbdevs	Fri Nov 19 01:34:18 2010	(r215486)
+++ stable/8/sys/dev/usb/usbdevs	Fri Nov 19 01:35:14 2010	(r215487)
@@ -2255,6 +2255,8 @@ product NATIONAL BEARPAW1200	0x1000	Bear
 product NATIONAL BEARPAW2400	0x1001	BearPaw 2400
 
 /* NEC products */
+product NEC HUB_0050		0x0050	USB 2.0 7-Port Hub
+product NEC HUB_005A		0x005a	USB 2.0 4-Port Hub
 product NEC HUB			0x55aa	hub
 product NEC HUB_B		0x55ab	hub
 
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"
Comment 9 Remko Lodder freebsd_committer freebsd_triage 2011-01-27 18:53:37 UTC
State Changed
From-To: patched->closed

This had been merged by Andrew.