Re: [RFC][PATCH 0/4] PCI legacy I/O port free driver

[Andi Kleen]

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/