On Wed, May 23, 2007 at 08:10:54PM -0700, Jesse Barnes wrote: > > +/* Give unknown D-Link network adapters a proper class */ > > +static void __devinit quirk_dlink_unknown(struct pci_dev *dev) > > +{ > > + if (dev->class = PCI_CLASS_UNKNOWN) > > Err, == of course. Obviously I didn't test this. :)
Actually, it should be something like this (also untested). Ivan. --- a/drivers/pci/quirks.c +++ b/drivers/pci/quirks.c @@ -1690,6 +1690,14 @@ static void __devinit quirk_p64h2_1k_io(struct pci_dev *dev) } DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x1460, quirk_p64h2_1k_io); +/* Give unknown D-Link network adapters a proper class */ +static void __devinit quirk_dlink_unknown(struct pci_dev *dev) +{ + if ((dev->class >> 8) == PCI_CLASS_NOT_DEFINED) + dev->class = PCI_CLASS_NETWORK_ETHERNET << 8; +} +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_DLINK, 0x4901, quirk_dlink_unknown); + /* Fix the IOBL_ADR for 1k I/O space granularity on the Intel P64H2 * The IOBL_ADR gets re-written to 4k boundaries in pci_setup_bridge() * in drivers/pci/setup-bus.c - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/