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