Subject: CVS commit: src/sys/dev
To: None <source-changes@NetBSD.org>
From: Izumi Tsutsui <tsutsui@netbsd.org>
List: source-changes
Date: 03/21/2007 12:17:32
Module Name:	src
Committed By:	tsutsui
Date:		Wed Mar 21 12:17:32 UTC 2007

Modified Files:
	src/sys/dev/cardbus: if_re_cardbus.c if_rtk_cardbus.c
	src/sys/dev/ic: rtl8169.c rtl81x9.c rtl81x9reg.h rtl81x9var.h
	src/sys/dev/pci: if_re_pci.c if_rtk_pci.c

Log Message:
Cleanup handling of quikrs on each RealTek chip:
- replace rtk_type member in rtk_softc which has chip types
  with new rtk_quirk that represents quirks on each chip:
  - RTKQ_8129           doesn't have internal MII (used in rtk(4))
  - RTKQ_8139CPLUS      has different register layout (for re(4))
  - RTKQ_8169NONS       (original 8169) requires some settings on init
  - RTKQ_PCIE           requires different settings in setmulti
  so that we don't have to check each hwrev values or types everywhere
  and newer variants will also work without changes if they don't
  have other quirks
  (sc_rev is unchenged for now for reference to the Realtek's driver)
- don't check hwrev register in re_pci_match() but check
  only PCI_VENDER(), PCI_PRODUCT() and PCI_REVISION()
  so that we no longer have to map pci space there
- add a new HWREV value for another 8168 variant
- try to map PCI mem space more properly
- remove (probably unneeded) ifp->if_baudrate initialization

Tested on a newer 8168 variant by Dennis den Brok on tech-kern,
and also tested on 8139 and 8169C on macppc, and 8139C+ on landisk
by me.


To generate a diff of this commit:
cvs rdiff -r1.12 -r1.13 src/sys/dev/cardbus/if_re_cardbus.c
cvs rdiff -r1.31 -r1.32 src/sys/dev/cardbus/if_rtk_cardbus.c
cvs rdiff -r1.83 -r1.84 src/sys/dev/ic/rtl8169.c
cvs rdiff -r1.71 -r1.72 src/sys/dev/ic/rtl81x9.c
cvs rdiff -r1.26 -r1.27 src/sys/dev/ic/rtl81x9reg.h
cvs rdiff -r1.38 -r1.39 src/sys/dev/ic/rtl81x9var.h
cvs rdiff -r1.26 -r1.27 src/sys/dev/pci/if_re_pci.c
cvs rdiff -r1.31 -r1.32 src/sys/dev/pci/if_rtk_pci.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.