From nobody@FreeBSD.ORG  Sun May 21 19:44:49 2000
Return-Path: <nobody@FreeBSD.ORG>
Received: by hub.freebsd.org (Postfix, from userid 32767)
	id 44E0837B54B; Sun, 21 May 2000 19:44:49 -0700 (PDT)
Message-Id: <20000522024449.44E0837B54B@hub.freebsd.org>
Date: Sun, 21 May 2000 19:44:49 -0700 (PDT)
From: adoal@iname.com
Sender: nobody@FreeBSD.ORG
To: freebsd-gnats-submit@FreeBSD.org
Subject: add support to Accton EN1217 network adapter
X-Send-Pr-Version: www-1.0

>Number:         18735
>Category:       kern
>Synopsis:       add support to Accton EN1217 network adapter
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    asmodai
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Sun May 21 19:50:14 PDT 2000
>Closed-Date:    Wed Mar 14 05:36:15 PST 2001
>Last-Modified:  Wed Mar 14 05:36:34 PST 2001
>Originator:     Adoal Xu
>Release:        4.0-RELEASE
>Organization:
Lunaris Studio
>Environment:
FreeBSD mail.lunaris.org 4.0-RELEASE FreeBSD 4.0-RELEASE #0: Mon Mar 20 22:50:22 GMT 2000    root@monster.cdrom.com:/usr/src/sys/compile/GENERIC  i386
>Description:
Accton's EN1217 network adapter is just based on Macronix MX98715AEC chip, so 'dc0' should work with it. But since Accton change it's PCI id, dc0 cannot probe it. So with EN1217's PCI id added, everything will be ok. It works just the standard MX98715 way, :)
DC_VENDORID_ACCTON 0x1113
DC_DEVICEID_AE310 0x1217
>How-To-Repeat:
just buy a piece of EN1217 (aka Adico AE310TX) and try it, :)
>Fix:
--- if_dc.c.orig	Mon May 22 10:30:03 2000
+++ if_dc.c	Mon May 22 10:35:45 2000
@@ -193,6 +193,8 @@
 		"82c168 PNIC 10/100BaseTX" },
 	{ DC_VENDORID_LO, DC_DEVICEID_82C168,
 		"82c169 PNIC 10/100BaseTX" },
+	{ DC_VENDORID_ACCTON, DC_DEVICEID_EN1217,
+		"Accton EN1217 10/100BaseTX" },
 	{ 0, 0, NULL }
 };
 
@@ -1548,6 +1550,7 @@
 		sc->dc_flags |= DC_TX_POLL|DC_TX_USE_TX_INTR;
 		break;
 	case DC_DEVICEID_987x5:
+	case DC_DEVICEID_EN1217:
 		sc->dc_type = DC_TYPE_987x5;
 		sc->dc_flags |= DC_TX_POLL|DC_TX_USE_TX_INTR;
 		break;
--- if_dcreg.h.orig	Mon May 22 10:29:55 2000
+++ if_dcreg.h	Mon May 22 10:34:42 2000
@@ -793,6 +793,16 @@
 #define DC_REVISION_88141	0x10
 
 /*
+ * Accton vendor ID.
+ */
+#define	DC_VENDORID_ACCTON		0x1113
+
+/*
+ * Accton device IDs.
+ */
+#define	DC_DEVICEID_EN1217	0x1217
+
+/*
  * PCI low memory base and low I/O base register, and
  * other PCI registers.
  */


>Release-Note:
>Audit-Trail:
State-Changed-From-To: open->suspended 
State-Changed-By: asmodai 
State-Changed-When: Sun Jun 11 04:56:22 PDT 2000 
State-Changed-Why:  
Set to suspended so it will stay here until the appropriate MFC's 
have been done. 


Responsible-Changed-From-To: freebsd-bugs->asmodai 
Responsible-Changed-By: asmodai 
Responsible-Changed-When: Sun Jun 11 04:56:22 PDT 2000 
Responsible-Changed-Why:  
I committed this fix. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=18735 
State-Changed-From-To: suspended->closed 
State-Changed-By: asmodai 
State-Changed-When: Wed Mar 14 05:36:15 PST 2001 
State-Changed-Why:  
This got MFC'd and will be in 4.3-RELEASE. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=18735 
>Unformatted:
