Subject: Problem Report: kern/22120
To: None <current-users@NetBSD.org>
From: Lars Heidieker <lars@cubeportal.de>
List: current-users
Date: 08/31/2003 11:13:21
Hi,

in sys/dev/pci/if_fxp_pci.c the PCI_PRODUCT_INTEL_PRO_100_VM_6 version 
should be called:
  "Intel PRO/100 VM Network Controller with 82562ET/EZ PHY"

and in fxp_pci_attach(parent, self, aux)
....
	case PCI_PRODUCT_INTEL_PRO_100_VE_0:
	case PCI_PRODUCT_INTEL_PRO_100_VE_1:
	case PCI_PRODUCT_INTEL_PRO_100_VM_0:
	case PCI_PRODUCT_INTEL_PRO_100_VM_1:
	case PCI_PRODUCT_INTEL_82562EH_HPNA_0:
	case PCI_PRODUCT_INTEL_82562EH_HPNA_1:
	case PCI_PRODUCT_INTEL_82562EH_HPNA_2:
	case PCI_PRODUCT_INTEL_PRO_100_VM_2:
	case PCI_PRODUCT_INTEL_PRO_100_VM_3:
	case PCI_PRODUCT_INTEL_PRO_100_VM_4:
	case PCI_PRODUCT_INTEL_PRO_100_VM_5:
	case PCI_PRODUCT_INTEL_PRO_100_VM_6
:
		aprint_normal(": %s, rev %d\n", fpp->fpp_name, sc->sc_rev);

		/*
		 * ICH3 chips apparently have problems with the enhanced
		 * features, so just treat them as an i82557.  It also
		 * has the resume bug that the ICH2 has.
		 */
		sc->sc_rev = 1;
		sc->sc_flags |= FXPF_HAS_RESUME_BUG;
		break;

should have the FXPF_HAS_RESUME_BUG removed as this breaks at least the 
case PCI_PRODUCT_INTEL_PRO_100_VM_6 interface if it is a 82562EZ
if there is any chip that needs the pacth this case should be broken 
into two.

Lars