Randy.Dunlap <[EMAIL PROTECTED]> : [...] > static struct pci_device_id r1000_pci_tbl[] __devinitdata = { > { 0x10ec, 0x8169, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0 }, > { 0x10ec, 0x8167, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0 }, > { 0x10ec, 0x8168, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0 }, > { 0x10ec, 0x8136, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0 }, > {0,} > }; > > Any reason not to include all of those?
Nothing worrying: - 0x8167 and 0x8168 use a different PCI region; - some phy differences. They appear when the r1000 driver is compared to the previous r8169 driver from realtek. I'll pack it with other changes. > Conversely, any reason to use the RealTek r1000 driver? Feel free to read it and make your own mind. -- Ueimor - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html