From nobody@FreeBSD.org  Fri Jun 16 22:13:42 2006
Return-Path: <nobody@FreeBSD.org>
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id F409216A474
	for <freebsd-gnats-submit@FreeBSD.org>; Fri, 16 Jun 2006 22:13:41 +0000 (UTC)
	(envelope-from nobody@FreeBSD.org)
Received: from www.freebsd.org (www.freebsd.org [216.136.204.117])
	by mx1.FreeBSD.org (Postfix) with ESMTP id A5EDB43D66
	for <freebsd-gnats-submit@FreeBSD.org>; Fri, 16 Jun 2006 22:13:34 +0000 (GMT)
	(envelope-from nobody@FreeBSD.org)
Received: from www.freebsd.org (localhost [127.0.0.1])
	by www.freebsd.org (8.13.1/8.13.1) with ESMTP id k5GMDYV4052008
	for <freebsd-gnats-submit@FreeBSD.org>; Fri, 16 Jun 2006 22:13:34 GMT
	(envelope-from nobody@www.freebsd.org)
Received: (from nobody@localhost)
	by www.freebsd.org (8.13.1/8.13.1/Submit) id k5GMDY61052006;
	Fri, 16 Jun 2006 22:13:34 GMT
	(envelope-from nobody)
Message-Id: <200606162213.k5GMDY61052006@www.freebsd.org>
Date: Fri, 16 Jun 2006 22:13:34 GMT
From: Pedro Alejandro Lpez-Valencia <palopezv@gmail.com>
To: freebsd-gnats-submit@FreeBSD.org
Subject: ste driver does not support IC Plus IP100A chipsets
X-Send-Pr-Version: www-2.3

>Number:         99064
>Category:       kern
>Synopsis:       [ste] [patch] ste driver does not support IC Plus IP100A chipsets
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    freebsd-bugs
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Fri Jun 16 22:20:19 GMT 2006
>Closed-Date:    Thu May 03 16:15:18 GMT 2007
>Last-Modified:  Thu May 03 16:15:18 GMT 2007
>Originator:     Pedro Alejandro Lpez-Valencia
>Release:        6.1-RELEASE
>Organization:
>Environment:
FreeBSD fanelia.lan 6.1-RELEASE FreeBSD 6.1-RELEASE #1: Thu Jun 15 14:12:33 COT 2006     root@:/usr/obj/usr/src/sys/LOCAL  i386

>Description:
IC Plus Corp. owns the Sundance Tech ehternet chipset business since 2004.
The IP100A chipset has appeared in several inexpensive network cards by
D-Link and Encore (I bought one of each last week, lucky me) and as the
chipset of integrated network interfaces of latest model mainboards
manufactured by ABIT, ASUS and Nvidia.

The chipset manufacturer supplies its own version of the ste driver
(<http://www.icplus.com.tw/pp-IP100A.html>), but it is based on 4.x code.
I would like to see the fix (which is trivial as seen in the patch below)
integrated in FreeBSD. Please notice that, as seen in the vendor's website
mentioned above, the ST201 chipset is now known as the IC Plus Corp. IP100
chipset.


>How-To-Repeat:
Install PCI ehternet card or use mainboard with integrated ehternet chipset (e.g, ABIT KU8/KV8//AX8/NF7-S2, ASUS P4B266/P5GD1-VM, etc.). It isn't recognized by any network driver in the distributed GENERIC nor SMP kernels.

>Fix:

*** if_ste.c	2006/06/15 22:41:48	1.1
--- if_ste.c	2006/06/15 22:56:51	1.2
***************
*** 87,92 ****
--- 87,93 ----
  static struct ste_type ste_devs[] = {
  	{ ST_VENDORID, ST_DEVICEID_ST201, "Sundance ST201 10/100BaseTX" },
  	{ DL_VENDORID, DL_DEVICEID_DL10050, "D-Link DL10050 10/100BaseTX" },
+ 	{ IC_VENDORID, IC_DEVICEID_IP100A, "IC Plus Corp. IP100A 10/100BaseTX" },
  	{ 0, 0, NULL }
  };
  
*** if_stereg.h	2006/06/15 22:41:48	1.1
--- if_stereg.h	2006/06/15 23:05:49	1.2
***************
*** 46,51 ****
--- 46,58 ----
  #define DL_DEVICEID_DL10050	0x1002
  
  /*
+  * IC Plus Corp. PCI device/vendor ID for the
+  * IP100A chip.
+  */
+ #define IC_VENDORID		0x13F0
+ #define IC_DEVICEID_IP100A	0x0200
+ 
+ /*
   * Register definitions for the Sundance Technologies ST201 PCI
   * fast ethernet controller. The register space is 128 bytes long and
   * can be accessed using either PCI I/O space or PCI memory mapping.

>Release-Note:
>Audit-Trail:

From: "=?ISO-8859-1?Q?Pedro_Alejandro_L=F3pez-Valencia?=" <palopezv@gmail.com>
To: bug-followup@FreeBSD.org, palopezv@gmail.com
Cc:  
Subject: Re: kern/99064: [ste] [patch] ste driver does not support IC Plus IP100A chipsets
Date: Thu, 1 Feb 2007 07:05:41 -0500

 ------=_Part_1342_21566466.1170331541358
 Content-Type: text/plain; charset=ISO-8859-1; format=flowed
 Content-Transfer-Encoding: 7bit
 Content-Disposition: inline
 
 Still no love in 6.2-CURRENT nor 7-CURRENT... :-(
 
 -- 
 vi veri veni versum vivus vici
 
 ------=_Part_1342_21566466.1170331541358
 Content-Type: text/html; charset=ISO-8859-1
 Content-Transfer-Encoding: 7bit
 Content-Disposition: inline
 
 Still no love in 6.2-CURRENT nor 7-CURRENT... :-(<br clear="all"><br>-- <br>vi veri veni versum vivus vici 
 
 ------=_Part_1342_21566466.1170331541358--

From: Gavin Atkinson <gavin.atkinson@ury.york.ac.uk>
To: bug-followup@FreeBSD.org, palopezv@gmail.com
Cc:  
Subject: Re: kern/99064: [ste] [patch] ste driver does not support IC Plus
	IP100A chipsets
Date: Thu, 03 May 2007 14:04:43 +0100

 This patch looks good, and is unlikely to break anything else.  It
 should be committed as-is.
 
 Gavin

From: Gavin Atkinson <gavin.atkinson@ury.york.ac.uk>
To: bug-followup@FreeBSD.org
Cc: palopezv@gmail.com
Subject: Re: kern/99064: [ste] [patch] ste driver does not support IC Plus
	IP100A chipsets
Date: Thu, 03 May 2007 14:09:02 +0100

 On Thu, 2007-05-03 at 14:04 +0100, Gavin Atkinson wrote:
 > This patch looks good, and is unlikely to break anything else.  It
 > should be committed as-is.
 
 Strike that, a similar patch was committed to src/sys/pci/if_ste.c 1.98,
 and MFC'd as 1.84.2.8.  The card will be supported in FreeBSD 6.3 and
 later.
 
 This PR can be closed.
 
 Gavin
State-Changed-From-To: open->closed 
State-Changed-By: linimon 
State-Changed-When: Thu May 3 16:14:44 UTC 2007 
State-Changed-Why:  
A similar patch has been committed and MFCed.  Thank you for the 
submission. 

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