On Mon, 20 Jan 2003, Mark M. Hoffman wrote: > FYI: I tested this on my 645/961 vs. a recent RedHat 2.4. > Besides adding PCI ids, are there any other changes I > should make to accomodate Ky?sti's recent work? Functionally it doesn't matter, but you should avoid all that pci_find_device in sis645_setup(), a call to pci_module_init has done the work for you. You can take i2c-amdXX as a sample that got accepted in Linus's 2.5 tree. As with any PCI device, the driver should support multiple instances of a device. Yes, even if it is unlikely. Do it right for i2c-sis645 and we get to copy it for other busses. -- Ky?sti M?lkki kmalkki at cc.hut.fi