Re: CVS commit: src/sys/dev/ic

[Soren Jacobsen <snj%pobox.com@localhost>]

On 03/21 03:08, Izumi Tsutsui wrote:
> > > Modified Files:
> > >   src/sys/dev/ic: rtl8169.c rtl81x9var.h
> > > 
> > > Log Message:
> > > Access LDPS register in re_reset() only on 8169S single chip variants.
> > > From OpenBSD and FreeBSD drivers via PR kern/41009, and
> > > Realtek-supplied FreeBSD driver.
> > 
> > We should probably pull this up to 5.0..  my HP desktop falls into the
> > hole reported in the PR quite often.
> 
> Can you confirm committed version on newer chips?
> 
> I've added a flag for the LDPS as FreeBSD/OpenBSD rather than
> checking sc->sc_rev value, and I tested it only on my old PCI 8169S.

re0 at pci2 dev 0 function 0: RealTek 8168B/8111B PCIe Gigabit Ethernet
(rev. 0x02)
re0: interrupting at ioapic0 pin 17
re0: Ethernet address 00:1f:d0:2e:4b:c9
re0: using 256 tx descriptors
rgephy0 at re0 phy 7: RTL8169S/8110S/8211 1000BASE-T media interface,
rev. 2
rgephy0: 10baseT, 10baseT-FDX, 100baseTX, 100baseTX-FDX, 1000baseT,
1000baseT-FDX, auto

The motherboard documentation states this is an 8111C.

Works here on netbsd-5.  I also had to locally pull in revision 1.106 of
rtl8169.c and 1.33 of rtl81x9reg.h.