Date: Mon, 10 Mar 2003 12:56:39 +0900 From: Takeshi Shibagaki <sibagaki@lsi.melco.co.jp> To: imp@bsdimp.com Cc: sibagaki@lsi.melco.co.jp, shiba@freebsd.org, src-committers@freebsd.org, cvs-src@freebsd.org, cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/dev/pccard pccarddevs Message-ID: <20030310125639Y.sibagaki@lsi.melco.co.jp> In-Reply-To: <20030219.010124.23410261.imp@bsdimp.com> References: <20030215.193910.121221310.imp@bsdimp.com> <20030219162137F.sibagaki@lsi.melco.co.jp> <20030219.010124.23410261.imp@bsdimp.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Hi, Warner-san. I'm sorry, too slow reply... >>>>> On Wed, 19 Feb 2003 01:01:24 -0700 (MST), "M. Warner Losh" <imp@bsdimp.com> said: > : In NetBSD, the card is indicated YIS Corp. YWL-11b because pccard > : driver outputs CISTPL_VER1 strings. So it may be better to rewrite > : like NetBSD in FreeBSD. M> I like this idea in general, but dispair that a few cards have really M> bad CIS. Maybe the right thing to do is to default the descrition to M> the CISTPL_VER1 strings, if available. The driver is free to override M> the description. Most drivers could simplify a little and rely on the M> CIS string to be a good descrition, while those that know the M> descrition is lacking could add their own. Yes, your idea is very good. I also think so. M> To be clear, you'd like to see a change like the following (my cisco M> 350 card serves as an example): M> OLD: M> an0: <Aironet 350 Wireless LAN Adapter> at port 0x100-0x13f irq 10 function 0 config 5 on pccard1 M> NEW: M> an0: <Cisco Systems 350 Series Wireless LAN Adapter> at port 0x100-0x13f irq 10 function 0 config 5 on pccard1 M> "Aironet 350 Wireless LAN Adapter" comes from the if_an_pccard driver, M> while "Cisco Systems 350 Series Wireless LAN Adapter" comes from the M> cis node talked about above. OK, looks nice. If you have been implemented, I hope you commit this changes. Takeshi Shibagaki ie9t-sbgk@asahi-net.or.jp To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030310125639Y.sibagaki>