amd64/121446: PATCH 7.0-STABLE for bge ether BCM5722 on Dell T105 box

[John Baldwin]

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

From: John Baldwin <jhb at freebsd.org>
To: freebsd-amd64 at freebsd.org,
 Chris Shenton <chris at shenton.org>
Cc: FreeBSD-gnats-submit at freebsd.org
Subject: Re: amd64/121446: PATCH 7.0-STABLE for bge ether BCM5722 on Dell T105 box
Date: Fri, 7 Mar 2008 08:30:26 -0500

 On Thursday 06 March 2008 03:52:09 pm Chris Shenton wrote:
 > >Number:         121446
 > >Category:       amd64
 > >Synopsis:       PATCH 7.0-STABLE for bge ether BCM5722 on Dell T105 box
 > >Confidential:   no
 > >Severity:       serious
 > >Priority:       medium
 > >Responsible:    freebsd-amd64
 > >State:          open
 > >Quarter:
 > >Keywords:
 > >Date-Required:
 > >Class:          change-request
 > >Submitter-Id:   current-users
 > >Arrival-Date:   Thu Mar 06 21:20:03 UTC 2008
 > >Closed-Date:
 > >Last-Modified:
 > >Originator:     Chris Shenton
 > >Release:        FreeBSD 7.0-STABLE amd64
 > >Organization:
 >
 > <organization of PR author (multiple lines)>
 >
 > >Environment:
 >
 > System: FreeBSD Navaja.shenton.org 7.0-STABLE FreeBSD 7.0-STABLE #2: Sun
 > Mar 2 16:46:28 EST 2008
 > root at Navaja.shenton.org:/usr/obj/usr/src/sys/GENERIC amd64
 >
 >
 > 	<machine, os, target, libraries (multiple lines)>
 >
 > >Description:
 >
 > 	7.0-RC1 to STABLE as of today don't support the ether card built into
 > Dell's inexpensive AMD 64 bit T105 servers.  This patch makes it work.
 >
 > >How-To-Repeat:
 >
 > Boot, try network, it doesn't.
 >
 > >Fix:
 >
 > This patch works for me.  I sent this to freebsd-hardware on Dec 23
 > 2007 and Remko said he'd try and add it in but didn't see it in the
 > csup I did a couple days ago.  Someone else told me I need to submit
 > as a PR so that's what I'm doin' here.  Thanks.
 >
 > *** if_bge.c	Mon Jan 21 12:17:00 2008
 > --- if_bge.c.CHRIS2	Sun Mar  2 14:43:26 2008
 > ***************
 > *** 169,174 ****
 > --- 169,175 ----
 >   	{ BCOM_VENDORID,	BCOM_DEVICEID_BCM5715S },
 >   	{ BCOM_VENDORID,	BCOM_DEVICEID_BCM5720 },
 >   	{ BCOM_VENDORID,	BCOM_DEVICEID_BCM5721 },
 > +  	{ BCOM_VENDORID,	BCOM_DEVICEID_BCM5722 },
 >   	{ BCOM_VENDORID,	BCOM_DEVICEID_BCM5750 },
 >   	{ BCOM_VENDORID,	BCOM_DEVICEID_BCM5750M },
 >   	{ BCOM_VENDORID,	BCOM_DEVICEID_BCM5751 },
 >
 >
 >
 > *** if_bgereg.h	Tue May 22 15:22:58 2007
 > --- if_bgereg.h.CHRIS2	Sun Mar  2 14:44:20 2008
 > ***************
 > *** 2011,2016 ****
 > --- 2011,2017 ----
 >   #define	BCOM_DEVICEID_BCM5715S		0x1679
 >   #define	BCOM_DEVICEID_BCM5720		0x1658
 >   #define	BCOM_DEVICEID_BCM5721		0x1659
 > + #define       BCOM_DEVICEID_BCM5722		0x165a
 >   #define	BCOM_DEVICEID_BCM5750		0x1676
 >   #define	BCOM_DEVICEID_BCM5750M		0x167C
 >   #define	BCOM_DEVICEID_BCM5751		0x1677
 
 This is already in HEAD and I committed further fixes for the PHY to HEAD 
 yesterday.  I'll MFC in a few days.
 
 -- 
 John Baldwin