Linux Kernel Mailing List wrote:ChangeSet 1.1288, 2004/01/26 16:58:21-02:00, francis.wiran@xxxxxx
@@ -616,7 +623,7 @@
/* detect controllers */
printk(DRIVER_NAME "\n");
- pci_register_driver(&cpqarray_pci_driver);
+ pci_module_init(&cpqarray_pci_driver);
cpqarray_eisa_detect();
for(i=0; i< MAX_CTLR; i++) {
You need to check the return value of pci_module_init() for errors.
Eeek! I want to fix that code in pci_module_init() so it doesn't do
this at all. Please don't copy that horrible function. Just register
the driver with a call to vio_register_driver() and drop the whole
vio_module_init() completly. I'll be doing that for pci soon, and
there's no reason you want to duplicate this broken logic (you always
want your module probe to succeed, for lots of reasons...)