> And the code with modifications: > > static int __init i2c_amd756_init(void) > { > struct pci_dev *dev; > const struct pci_device_id *id; > > printk(KERN_INFO "i2c-amd756.o version %s (%s)\n", LM_VERSION, > LM_DATE); > > return pci_module_init(&amd756_driver); ^^^^^^ That return in the middle of the function might explain why you don't see the change... ;) Strange if gcc did not tell you about that. > pci_for_each_dev(dev) { > id = pci_match_device(amd756_ids, dev); > if(id) > if(amd756_probe(dev, id) >= 0) > return 0; > } > return -ENODEV; > > > } -- Jean Delvare http://www.ensicaen.ismra.fr/~delvare/