Re: [RFC][PATCH 0/4] PCI legacy I/O port free driver
From: Andi Kleen
Date: Tue Feb 14 2006 - 04:31:28 EST
Kenji Kaneshige <kaneshige.kenji@xxxxxxxxxxxxxx> writes:
> I encountered a problem that some PCI devices don't work on my system
> which have huge number of PCI devices.
Is that a large IA64 system?
[...]
The basic concept looks good to me, but I would suggest you use
the Linux bitmap functions (DECLARE_BITMAP(), set_bit, test_bit etc.)
instead of open coding all that.
And for the e1000 change - instead of adding a big switch with
magic numbers that will likely bitrot it's better to use
the driver_data field in pci_device_id for such device specific flags.
-Andi
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/