kern/110804: re driver doesn't properly support RTL8110SC

[Mark Linimon]

The following reply was made to PR kern/110804; it has been noted by GNATS.

From: linimon at lonesome.com (Mark Linimon)
To: freebsd-gnats-submit at FreeBSD.org
Cc:  
Subject: Re: kern/110804: re driver doesn't properly support RTL8110SC
Date: Mon, 26 Mar 2007 11:43:56 -0500

 Forwarded from email to freebsd-bugs.
 
 On Mon, Mar 26, 2007 at 01:52:18PM +0200, Daan Vreeken [PA4DAN] <Danovitsch at vitsch.net> wrote:
 > 
 > I stumbled upon this two weeks ago while setting up a new (mini-ITX) server 
 > for my company. The attached patch adds the correct ID's for the Realtek 
 > chips on the Jetway triple-gigabit expansion card.
 > After patching the kernel all three devices are properly detected and work 
 > like a charm.
 > 
 > This tiny patch was sponsoren by : Vitsch Electronics
 > 
 > pciconf, dmesg or any other information is available upon request.
 > 
 > grtz,
 > -- 
 > Daan
 
 > --- if_re.c~	Mon Jan  1 02:07:02 2007
 > +++ if_re.c	Mon Jan  1 02:11:38 2007
 > @@ -182,6 +182,8 @@
 >  		"RealTek 8169S Single-chip Gigabit Ethernet" },
 >  	{ RT_VENDORID, RT_DEVICEID_8169, RL_HWREV_8169_8110SB,
 >  		"RealTek 8169SB/8110SB Single-chip Gigabit Ethernet" },
 > +	{ RT_VENDORID, RT_DEVICEID_8169, RL_HWREV_8169_8110SC,
 > +		"RealTek 8169SC/8110SC Single-chip Gigabit Ethernet" },
 >  	{ RT_VENDORID, RT_DEVICEID_8169SC, RL_HWREV_8169_8110SC,
 >  		"RealTek 8169SC/8110SC Single-chip Gigabit Ethernet" },
 >  	{ RT_VENDORID, RT_DEVICEID_8169, RL_HWREV_8110S,
 > @@ -1219,7 +1221,9 @@
 >  	 * Allocate the parent bus DMA tag appropriate for PCI.
 >  	 */
 >  #define RL_NSEG_NEW 32
 > -	error = bus_dma_tag_create(bus_get_dma_tag(dev), 1, 0,
 > +// DAAN Daan daan !!
 > +//	error = bus_dma_tag_create(bus_get_dma_tag(dev), 1, 0,
 > +	error = bus_dma_tag_create(NULL, 1, 0,
 >  	    BUS_SPACE_MAXADDR_32BIT, BUS_SPACE_MAXADDR, NULL, NULL,
 >  	    MAXBSIZE, RL_NSEG_NEW, BUS_SPACE_MAXSIZE_32BIT, 0,
 >  	    NULL, NULL, &sc->rl_parent_tag);