Date: Sat, 25 Jun 2011 20:51:38 +0200 From: Hans Petter Selasky <hselasky@c2i.net> To: Gavin Atkinson <gavin@freebsd.org> Cc: "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>, kevlo@freebsd.org, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "src-committers@freebsd.org" <src-committers@freebsd.org> Subject: Re: svn commit: r223537 - head/sys/dev/usb/wlan Message-ID: <201106252051.38956.hselasky@c2i.net> In-Reply-To: <201106252043.59444.hselasky@c2i.net> References: <201106252043.59444.hselasky@c2i.net>
next in thread | previous in thread | raw e-mail | index | archive | help
On Saturday 25 June 2011 20:43:59 Hans Petter Selasky wrote: > On Saturday 25 June 2011 20:26:19 Gavin Atkinson wrote: > > On Sat, 25 Jun 2011, Hans Petter Selasky wrote: > > > Author: hselasky > > > Date: Sat Jun 25 15:46:24 2011 > > > New Revision: 223537 > > > URL: http://svn.freebsd.org/changeset/base/223537 > > > > > > Log: > > > - Remove duplicate USB ID. > > > > Are you sure? > > > > > Modified: > > > head/sys/dev/usb/wlan/if_zyd.c > > > > > > Modified: head/sys/dev/usb/wlan/if_zyd.c > > > ======================================================================= > > > == ===== --- head/sys/dev/usb/wlan/if_zyd.c Sat Jun 25 15:42:33 > > > 2011 (r223536) +++ head/sys/dev/usb/wlan/if_zyd.c Sat Jun 25 15:46:24 > > > 2011 (r223537) @@ -229,7 +229,6 @@ static const STRUCT_USB_HOST_ID > > > zyd_devs > > > > > > ZYD_ZD1211_DEV(ZYXEL, ZYAIRG220), > > > ZYD_ZD1211_DEV(ZYXEL, G200V2), > > > /* ZYD_ZD1211B */ > > > > > > - ZYD_ZD1211B_DEV(ACCTON, SMCWUSBG_NF), > > > > > > ZYD_ZD1211B_DEV(ACCTON, SMCWUSBG), > > > ZYD_ZD1211B_DEV(ACCTON, ZD1211B), > > > ZYD_ZD1211B_DEV(ASUS, A9T_WIFI), > > > > gavin@psi:/home/gavin 110% grep SMCWUSBG > > /usr/freebsd/head/sys/dev/usb/usbdevs > > product ACCTON SMCWUSBG_NF 0x4505 SMCWUSB-G (no firmware) > > product ACCTON SMCWUSBG 0x4506 SMCWUSB-G > > > > This SMCWUSBG_NF ID was only added three months ago, in r219982. The > > other ID has been in the driver since it was first committed. > > > > Gavin > > Hi, > > vendor ACCTON 0x083a Accton Technology > > If I'm not mistaken, this ID is supposed to be handled by uath.conf, see > below. > > http://svnweb.freebsd.org/base/head/etc/devd/uath.conf?revision=207020&view > =markup > > 5 # Accton > 6 # SMCWUSB-G and SMCWUSBT-G2 > 7 notify 100 { > 8 match "system" "USB"; > 9 match "subsystem" "DEVICE"; > 10 match "type" "ATTACH"; > 11 match "vendor" "0x083a"; > 12 match "product" "(0x4505|0x4507)"; > 13 action "/usr/sbin/uathload -d /dev/$cdev"; > 14 }; > > I think not both of if_zyd and uath.conf should handle this ID. Correct me > if I'm wrong. According to the other entries, only the no-firmware ones > are supposed to be listed in uath.conf, while the firmware ones in xxx.ko. > > --HPS Hi, I see a minor bug here: %grep -r ACCTON /sys/dev/usb/wlan/if_zyd.c ZYD_ZD1211B_DEV(ACCTON, SMCWUSBG_NF), ZYD_ZD1211B_DEV(ACCTON, SMCWUSBG), ZYD_ZD1211B_DEV(ACCTON, ZD1211B), %grep -r ACCTON /sys/dev/usb/wlan/if_uath.c UATH_DEV(ACCTON, SMCWUSBG), UATH_DEV(ACCTON, SMCWUSBTG2), Which one of these drivers support the SMCWUSBG ? --HPS
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201106252051.38956.hselasky>