From owner-freebsd-hackers@FreeBSD.ORG Thu Oct 21 22:35:25 2004 Return-Path: Delivered-To: freebsd-hackers@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id DC6DB16A4CE for ; Thu, 21 Oct 2004 22:35:24 +0000 (GMT) Received: from odin.ac.hmc.edu (Odin.AC.HMC.Edu [134.173.32.75]) by mx1.FreeBSD.org (Postfix) with ESMTP id B797643D2F for ; Thu, 21 Oct 2004 22:35:24 +0000 (GMT) (envelope-from brdavis@odin.ac.hmc.edu) Received: from odin.ac.hmc.edu (localhost.localdomain [127.0.0.1]) by odin.ac.hmc.edu (8.13.0/8.13.0) with ESMTP id i9LMZPXn008536; Thu, 21 Oct 2004 15:35:25 -0700 Received: (from brdavis@localhost) by odin.ac.hmc.edu (8.13.0/8.13.0/Submit) id i9LMZPKL008535; Thu, 21 Oct 2004 15:35:25 -0700 Date: Thu, 21 Oct 2004 15:35:25 -0700 From: Brooks Davis To: Brooks Davis Message-ID: <20041021223525.GB4730@odin.ac.hmc.edu> References: <200410132110.09915.soralx@cydem.org> <200410171122.28021.soralx@cydem.org> <20041021173456.GA20517@odin.ac.hmc.edu> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="A6N2fC+uXW/VQSAv" Content-Disposition: inline In-Reply-To: <20041021173456.GA20517@odin.ac.hmc.edu> User-Agent: Mutt/1.4.1i X-Virus-Scanned: by amavisd-new X-Spam-Status: No, hits=0.3 required=8.0 tests=UPPERCASE_25_50 autolearn=no version=2.63 X-Spam-Checker-Version: SpamAssassin 2.63 (2004-01-11) on odin.ac.hmc.edu cc: freebsd-hackers@freebsd.org Subject: Re: [PATCH] Re: Linksys PCM200 X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 21 Oct 2004 22:35:25 -0000 --A6N2fC+uXW/VQSAv Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Oct 21, 2004 at 10:34:56AM -0700, Brooks Davis wrote: > On Wed, Oct 20, 2004 at 10:59:50PM -0600, soralx@cydem.org wrote: > >=20 > > [got no answer on hardware@] > >=20 > > I've tested this on 5.3-BETA7 - works OK, no more watchdog timeouts. > > So could someone review those patches and add them to the source tree? > > It's probably a good idea to update dc(4) and supported hw list also. >=20 > I'm looking at this, though I have no hardware to test with. Could some people with dc(4) devices please test the following patch? It compiles for me and is trivial, but a quick test is probalby in order before I commit it. This differs from soralx's patch in that it adds comments on the 0xab08 collisions and shortens the description string to something that will fit in one line on the console. Thanks, Brooks Index: if_dc.c =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D RCS file: /home/ncvs/src/sys/pci/if_dc.c,v retrieving revision 1.152 diff -u -p -r1.152 if_dc.c --- if_dc.c 1 Oct 2004 15:23:31 -0000 1.152 +++ if_dc.c 21 Oct 2004 22:32:39 -0000 @@ -218,6 +218,10 @@ static struct dc_type dc_devs[] =3D { "Microsoft MN-130 10/100" }, { DC_VENDORID_MICROSOFT, DC_DEVICEID_MSMN130_FAKE, "Microsoft MN-130 10/100" }, + { DC_VENDORID_LINKSYS, DC_DEVICEID_PCM200_AB08, + "Linksys PCM200 EtherFast CardBus 10/100" }, + { DC_VENDORID_LINKSYS, DC_DEVICEID_PCM200_AB09, + "Linksys PCM200 EtherFast CardBus 10/100" }, { 0, 0, NULL } }; =20 @@ -1917,10 +1921,16 @@ dc_attach(device_t dev) case DC_DEVICEID_FA511: case DC_DEVICEID_FE2500: case DC_DEVICEID_EN2242: - case DC_DEVICEID_HAWKING_PN672TX: + case DC_DEVICEID_HAWKING_PN672TX:=09 +#if 0 /* DC_DEVICEID_HAWKING_PN672TX has the same ID as the following */ + case DC_DEVICEID_FE2500MX: + case DC_DEVICEID_FNW3602T: + case DC_DEVICEID_PCM200_AB08: +#endif case DC_DEVICEID_3CSOHOB: case DC_DEVICEID_MSMN120: case DC_DEVICEID_MSMN130_FAKE: /* XXX avoid collision with PNIC*/ + case DC_DEVICEID_PCM200_AB09: sc->dc_type =3D DC_TYPE_AN985; sc->dc_flags |=3D DC_64BIT_HASH; sc->dc_flags |=3D DC_TX_USE_TX_INTR; Index: if_dcreg.h =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D RCS file: /home/ncvs/src/sys/pci/if_dcreg.h,v retrieving revision 1.42 diff -u -p -r1.42 if_dcreg.h --- if_dcreg.h 29 Jun 2004 20:38:44 -0000 1.42 +++ if_dcreg.h 21 Oct 2004 22:32:40 -0000 @@ -998,6 +998,17 @@ struct dc_softc { #define DC_DEVICEID_MSMN130_FAKE 0xFFF2 =20 /* + * Linksys vendor ID + */ +#define DC_VENDORID_LINKSYS 0x1737 + +/* + * Linksys device IDs + */ +#define DC_DEVICEID_PCM200_AB08 0xab08 +#define DC_DEVICEID_PCM200_AB09 0xab09 + +/* * PCI low memory base and low I/O base register, and * other PCI registers. */ --=20 Any statement of the form "X is the one, true Y" is FALSE. PGP fingerprint 655D 519C 26A7 82E7 2529 9BF0 5D8E 8BE9 F238 1AD4 --A6N2fC+uXW/VQSAv Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.1 (GNU/Linux) iD8DBQFBeDmsXY6L6fI4GtQRAq8CAKDSAzixBQOrWoBlWSvnyU7vRMYuswCdGWLO SDTQvy/R41NdK3iNVAJASOw= =IGaH -----END PGP SIGNATURE----- --A6N2fC+uXW/VQSAv--